On 20/11/2020 1:01 am, Ramsay, Lincoln wrote: > External Email > > ---------------------------------------------------------------------- > The build_skb path fails to allow for an SKB header, but the hardware > buffer it is built around won't allow for this anyway. > > Just always use the slower codepath that copies memory into an > allocated SKB. > > Signed-off-by: Lincoln Ramsay <lincoln.ram...@opengear.com> Acked-by: Igor Russkikh <irussk...@marvell.com> Yep, that could be the only way to fix this for now. Have you tried to estimate any performance drops from this? The most harm may be here on smaller packets, for stuff like UDP. Regards, Igor
- Re: [PATCH v3] aquantia: Remove the build_skb pa... Florian Westphal
- [PATCH v4] aquantia: Remove the build_skb path Ramsay, Lincoln
- Re: [PATCH v4] aquantia: Remove the build_skb pa... Florian Westphal
- Re: [PATCH v4] aquantia: Remove the build_skb pa... Ramsay, Lincoln
- Re: [PATCH v4] aquantia: Remove the build_skb pa... Jakub Kicinski
- Re: [PATCH v4] aquantia: Remove the build_skb pa... Jakub Kicinski
- Re: [PATCH v4] aquantia: Remove the build_skb pa... Ramsay, Lincoln
- Re: [PATCH v4] aquantia: Remove the build_skb pa... Jakub Kicinski
- [PATCH net v5] aquantia: Remove the build_skb pa... Ramsay, Lincoln
- Re: [PATCH v4] aquantia: Remove the build_skb pa... Ramsay, Lincoln
- Re: [EXT] [PATCH] aquantia: Remove the build_skb path Igor Russkikh
- Re: [EXT] [PATCH] aquantia: Remove the build_skb pat... Ramsay, Lincoln
- Re: [EXT] [PATCH] aquantia: Remove the build_skb... Igor Russkikh