netdev - Re: [PATCH net-next] net_sched: restore "mpu xxx" handling

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <a59fc45e-f9c8-91fe-09a2-e47605c4c0c7@bracey.fi>
Date:   Wed, 12 Jan 2022 09:02:59 +0200
From:   Kevin Bracey <kevin@...cey.fi>
To:     Jakub Kicinski <kuba@...nel.org>
CC:     <netdev@...r.kernel.org>, Eric Dumazet <edumazet@...gle.com>,
        Jiri Pirko <jiri@...nulli.us>, Vimalkumar <j.vimal@...il.com>
Subject: Re: [PATCH net-next] net_sched: restore "mpu xxx" handling

On 12/01/2022 08:36, Kevin Bracey wrote:
>
> Indeed, There has never been any kernel handling of tc_ratespec::mpu - 
> the kernel merely stored the value.
>
> The overhead had been similarly passed to the kernel but not 
> originally acted on. Linklayer had to be added to tc_ratespec. 

Ah, I need to correct myself there. The overhead was originally acted on 
in qdisc_l2t. htb_l2t forgot to incorporate it.

So:

  * overhead - always passed via tc_ratespec, handled by kernel. HTB
    temporarily ignored it.
  * linklayer - not originally passed via tc_ratespec, but incorporated
    in table. HTB temporarily lost functionality when it stopped using
    table. Later passed via ratespec, or inferred from table analysis
    for old iproute2.
  * mpu - always passed via tc_ratespec, but ignored by kernel.
    Incorporated in table. HTB lost functionality when it stopped using
    table.

("always" meaning "since iproute2 first had the parameter").

So this is a tad different from the other two - those were making the 
kernel act on something it previously acted on. This makes it act on 
something it's always been given, but never acted on. But it restores 
iproute2+kernel system functionality with no userspace change.

Kevin