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
| ||
|
Date: Wed, 17 Jun 2020 11:45:28 -0700 From: Eric Dumazet <edumazet@...gle.com> To: "David S . Miller" <davem@...emloft.net> Cc: netdev <netdev@...r.kernel.org>, Eric Dumazet <eric.dumazet@...il.com>, Willem de Bruijn <willemb@...gle.com>, Antoine Tenart <antoine.tenart@...tlin.com> Subject: Re: [PATCH net-next 1/5] net: tso: double TSO_HEADER_SIZE value On Wed, Jun 17, 2020 at 11:44 AM Eric Dumazet <edumazet@...gle.com> wrote: > > Transport header size could be 60 bytes, and network header > size can also be 60 bytes. Add the Ethernet header and we > are above 128 bytes. > > Since drivers using net/core/tso.c usually allocates > one DMA coherent piece of memory per TX queue, this patch > might cause issues if a driver was using too many slots. > > For 1024 slots, we would need 256 KB of physically > contiguous memory instead of 128 KB. > > Alternative fix would be to add checks in the fast path, > but this involves more work in all drivers using net/core/tso.c. > > Fixes: f9cbe9a556af ("net: define the TSO header size in net/tso.h") > Signed-off-by: Eric Dumazet <edumazet@...gle.com> > Cc: Antoine Tenart <antoine.tenart@...tlin.com> > Please disregard this patch, I will send a patch series, sorry for the noise.
Powered by blists - more mailing lists