[PATCH] Filter: Add support for setting TCP congestion control algorithm

Trisha Biswas tbiswas at fastly.com
Fri Mar 25 16:45:20 CET 2022


Thanks for getting back, Ondrej. Adding a generic EAF_TYPE_STRING, and
merging after the redesign both sound good. Any idea what timeframe this
redesign will happen in, so that I can keep an eye on when the patch gets
merged?

Thanks!
Trisha
--

*Trisha Biswas* | Sr. Software Engineer, Network Systems
fastly.com | @fastly <https://twitter.com/fastly> | LinkedIn
<http://www.linkedin.com/company/fastly>


On Fri, Mar 25, 2022 at 7:45 AM Ondrej Zajicek <santiago at crfreenet.org>
wrote:

> On Thu, Mar 17, 2022 at 05:32:02PM -0600, Trisha Biswas wrote:
> > Add support for setting the TCP congestion control algorithm per
> > destination by
> >
> > modifying route attributes in BIRD. Previously supported kernel route
> > attributes
> >
> > in BIRD were all integer values. Subsequently, this also adds string
> > handling of
> >
> > kernel metrics.
>
> Hi
>
> We are currently redesigning attribute types in order to have direct
> matching between attribute and filter types. And considering that string
> type is not currently implemented in attributes and it is a special case
> that have a different representation in both (direct ptr vs adata), we will
> postpone this patch after the redesign and then merge relevant parts of
> your patch.
>
> Otherwise, the patch is ok and the feature is worth merginh, i would just
> change it to add generic string type EAF_TYPE_STRING instead of
> EAF_TYPE_CC_ALGO.
>
> --
> Elen sila lumenn' omentielvo
>
> Ondrej 'Santiago' Zajicek (email: santiago at crfreenet.org)
> OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, wwwkeys.pgp.net)
> "To err is human -- to blame it on a computer is even more so."
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://trubka.network.cz/pipermail/bird-users/attachments/20220325/8c6d408e/attachment.htm>


More information about the Bird-users mailing list