[Cake] Using firewall connmarks as tin selectors
toke at redhat.com
Thu Feb 28 06:13:38 EST 2019
Kevin Darbyshire-Bryant <kevin at darbyshire-bryant.me.uk> writes:
>> On 28 Feb 2019, at 09:54, Toke Høiland-Jørgensen <toke at redhat.com> wrote:
>>> I also equally aware that this is ‘creeping featuritis’ and doing
>>> nothing to speed cake up…
>> Yeah, this is the crux of the issue, really: it's a tradeoff between
>> ease of use and featuritis. Now in this case the actual impact is a
>> single check it might actually be acceptable
>>> actually I may have improved BESTEFFORT a little - we no longer look
>>> for matching TC Major numbers if there’s no actual choice of tin to be
>>> made :-)
>> Well, you made the besteffort case slightly faster, but every other mode
>> slightly slower... :)
> Ah, ok, tc filter really is king ;-)
Well, I don't mind doing the simplification, but it should be done as a
separate commit; I'll do that after mering your pull request.
>> If you are going to send a patch (or pull request), please leave out the
>> refactoring, and only include the feature. This makes it easier to see
>> the impact of the feature addition on its own.
> Ha ha, sending a patch to the kernel lists??? Never again!
Haha, no, I meant to this list. I'll submit upstream, along with the
> The non-refactored version is
> and I’ve created a PR
>> Also, I assume you have a companion patch for iproute2 somewhere?
> In the first email: https://github.com/ldir-EDB0/tc-adv/commits/fwmark
> - based on 4.20.0 ‘cos that’s where openwrt is at the moment.
A pull request against tc-adv would be useful :)
> When I’ve found a suitable editor I was even going to update the man
I usually just use a regular editor and copy-paste the syntax bits from
More information about the Cake