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
| ||
|
Message-ID: <96daa3fe4db8169be920d40770d3b92a30e7f200.camel@redhat.com> Date: Tue, 19 Dec 2023 12:33:58 +0100 From: Paolo Abeni <pabeni@...hat.com> To: Graeme Smecher <gsmecher@...eespeedlogic.com>, David Ahern <dsahern@...nel.org> Cc: "David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>, Jakub Kicinski <kuba@...nel.org>, netdev@...r.kernel.org, claudiu.beznea@...on.dev, nicolas.ferre@...rochip.com, mdf@...nel.org Subject: Re: [PATCH] RFC: net: ipconfig: temporarily bring interface down when changing MTU. On Fri, 2023-12-15 at 17:04 -0800, Graeme Smecher wrote: > Several network drivers (sh_eth, macb_main, nixge, sundance) only allow > the MTU to be changed when the interface is down, because their buffer > allocations are performed during ndo_open() and calculated using a > specific MTU. I think we have to address that in the relevant device drivers. This could cause significant regressions: changing the MTU may causes the device to stay down afterwards - if the device fails to re-initialize. That even for drivers that before failed gracefully. Instead, at the device level, the driver could allocate in advance the new ring, and restore the previous one if the allocation fails, always leaving the device in a consistent status. Cheers, Paolo
Powered by blists - more mailing lists