From: Bartek Kois <bartek.kois@gmail.com>
To: Cong Wang <xiyou.wangcong@gmail.com>
Cc: Linux Kernel Network Developers <netdev@vger.kernel.org>
Subject: Re: Problem with queuing vlan tagged packets after migration from 3.16.0 to 4.9.0
Date: Fri, 4 Jan 2019 19:11:14 +0100 [thread overview]
Message-ID: <d43c0e65-fe03-8cdc-2197-c953005f25d2@gmail.com> (raw)
In-Reply-To: <CAM_iQpWvLSVVJcr_R9jhmM3LQsmA=2Nu=D+=WfyMM_aGJqGD_g@mail.gmail.com>
Hi
W dniu 03.01.2019 o 21:44, Cong Wang pisze:
> On Thu, Jan 3, 2019 at 7:25 AM Bartek Kois <bartek.kois@gmail.com> wrote:
>> Hi
>> 1. What exactly caused this change in the kernel?
>
> I don't follow VLAN changes, I guess it must be some change which
> inserts the VLAN tag before this ->ndo_start_xmit().
>
>
>> 2. I don`t understand why adding VLAN tag, which is just 4 additional
>> bytes to the passing packet make it impossible to classify.
> It is possible, you just have to specify the offset manually, as the
> iproute2 can't detect the offset of IP header in this case. So it is just
> inconvenient.
>
>> 3. This whole thing makes the QoS under Linux routers hard to configure
>> in scenarios with more than one VLAN which is pretty much every slightly
>> bigger router nowadays especially if we use IFB and hashing filters. Is
>> there any walkaround for that problem?
>>
> Just move these filters from physical device to the VLAN device instead?
>
Basically my current scenario looks like this:
- router with eth0 as WAN and eth1 as LAN with 10-20 vlans,
- around 1000-2000 ip addresses in differnets subnets behind router (on
the LAN side),
- QoS made with tc + ifb (for upload queuing) + hasing filters (for
performance reasons)
Moving this to two queuing trees (one on vlan and one on ifbx) per each
vlan makes this really hard to configure, but not impossible as long as
I can redirect single VLAN to ifb (don`t know if that is possible).
Anton suggested to use iptables+ipset but I don`t think that would be a
good idea to do that in scenario with so many queues.
Best reagrds
Bartek Kois
next prev parent reply other threads:[~2019-01-04 18:11 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-29 12:52 Problem with queuing vlan tagged packets after migration from 3.16.0 to 4.9.0 Bartek Kois
2018-12-30 18:53 ` Cong Wang
2018-12-30 21:14 ` Bartek Kois
2018-12-31 18:13 ` Bartek Kois
2019-01-01 19:33 ` Cong Wang
2019-01-01 19:46 ` Bartek Kois
2019-01-03 3:30 ` Cong Wang
2019-01-03 15:25 ` Bartek Kois
2019-01-03 20:44 ` Cong Wang
2019-01-04 18:11 ` Bartek Kois [this message]
2019-01-05 5:03 ` Cong Wang
2019-01-06 14:44 ` Jamal Hadi Salim
2019-01-10 13:45 ` Simon Horman
2019-01-12 12:12 ` Jamal Hadi Salim
2019-01-13 18:22 ` Cong Wang
2019-01-15 15:09 ` Jamal Hadi Salim
2019-01-15 18:19 ` Cong Wang
2019-01-16 14:13 ` Jamal Hadi Salim
2019-01-14 8:12 ` Simon Horman
2019-01-15 15:16 ` Jamal Hadi Salim
2019-01-18 4:32 ` Eric W. Biederman
2019-01-03 21:49 ` Anton Danilov
2019-01-04 7:07 ` Bartek Kois
2018-12-31 21:47 ` Jakub Kicinski
2018-12-31 22:12 ` Bartek Kois
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
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d43c0e65-fe03-8cdc-2197-c953005f25d2@gmail.com \
--to=bartek.kois@gmail.com \
--cc=netdev@vger.kernel.org \
--cc=xiyou.wangcong@gmail.com \
/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;
as well as URLs for NNTP newsgroup(s).