Cake - FQ_codel the next generation
 help / color / mirror / Atom feed
From: Kevin Darbyshire-Bryant <kevin@darbyshire-bryant.me.uk>
To: Stephen Hemminger <stephen@networkplumber.org>
Cc: Cake List <cake@lists.bufferbloat.net>
Subject: Re: [Cake] Ingress classification
Date: Thu, 7 Feb 2019 16:28:20 +0000	[thread overview]
Message-ID: <D7FE380C-46C5-4D8D-A529-C80EC16246D8@darbyshire-bryant.me.uk> (raw)
In-Reply-To: <20190206081922.56bd9c42@hermes.lan>



> On 6 Feb 2019, at 16:19, Stephen Hemminger <stephen@networkplumber.org> wrote:
> 
>> 
>> 
>> At this stage I’m wondering if its possible to get tc/cake to classify egress based on connmarks instead of relying on DSCP but my tc filter syntax is failing me at the moment :-)
> 
> It is possible to use a tc ingress filter to remark DSCP.

Hmmm.  I wonder if this would be more efficient to re-mark the DSCP code and let cake use its normal DSCP to tin mapping instead of overriding via an skb priority edit?

My new problem is using fw marks with tc filters changing skb priority on *egress*.  If I could get that to work then I think it would become compatible with software flow offloading.

I can’t get it to work and don’t know if I’m doing something wrong (likely) with my tc incantations or if it’s just not possible.  

I think something like the following should work, assuming the cake instance handling egress on eth0 is 809a: as shown by 

# tc qdisc show dev ifb4eth0
qdisc cake 809b: root refcnt 2 bandwidth 78Mbit diffserv4 dual-dsthost nat nowash ingress no-ack-filter split-gso rtt 100.0ms ptm overhead 26 

So the following should change the skb priority field based on a firewall connmark of 0x01

tc filter add dev eth0 parent 809a: protocol all handle 0x01 fw action skbedit priority 809a:4


Cheers,

Kevin D-B

012C ACB2 28C6 C53E 9775  9123 B3A2 389B 9DE2 334A


      reply	other threads:[~2019-02-07 16:28 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-05 12:08 Kevin Darbyshire-Bryant
2019-02-05 13:38 ` John Sager
2019-02-06 12:52   ` Kevin Darbyshire-Bryant
2019-02-06 13:54     ` Toke Høiland-Jørgensen
2019-02-10 21:54       ` Kevin Darbyshire-Bryant
2019-02-10 22:18         ` Toke Høiland-Jørgensen
2019-02-06 16:19     ` Stephen Hemminger
2019-02-07 16:28       ` Kevin Darbyshire-Bryant [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://lists.bufferbloat.net/postorius/lists/cake.lists.bufferbloat.net/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=D7FE380C-46C5-4D8D-A529-C80EC16246D8@darbyshire-bryant.me.uk \
    --to=kevin@darbyshire-bryant.me.uk \
    --cc=cake@lists.bufferbloat.net \
    --cc=stephen@networkplumber.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox