[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240510095342.GB1736038@kernel.org>
Date: Fri, 10 May 2024 10:53:42 +0100
From: Simon Horman <horms@...nel.org>
To: Eric Dumazet <edumazet@...gle.com>
Cc: "David S . Miller" <davem@...emloft.net>,
Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
netdev@...r.kernel.org, eric.dumazet@...il.com,
Steve Glendinning <steve.glendinning@...well.net>,
UNGLinuxDriver@...rochip.com
Subject: Re: [PATCH v3 net-next] net: usb: smsc95xx: stop lying about
skb->truesize
On Thu, May 09, 2024 at 08:33:13AM +0000, Eric Dumazet wrote:
> Some usb drivers try to set small skb->truesize and break
> core networking stacks.
>
> In this patch, I removed one of the skb->truesize override.
>
> I also replaced one skb_clone() by an allocation of a fresh
> and small skb, to get minimally sized skbs, like we did
> in commit 1e2c61172342 ("net: cdc_ncm: reduce skb truesize
> in rx path") and 4ce62d5b2f7a ("net: usb: ax88179_178a:
> stop lying about skb->truesize")
>
> v3: also fix a sparse error ( https://lore.kernel.org/oe-kbuild-all/202405091310.KvncIecx-lkp@intel.com/ )
> v2: leave the skb_trim() game because smsc95xx_rx_csum_offload()
> needs the csum part. (Jakub)
> While we are it, use get_unaligned() in smsc95xx_rx_csum_offload().
>
> Fixes: 2f7ca802bdae ("net: Add SMSC LAN9500 USB2.0 10/100 ethernet adapter driver")
> Signed-off-by: Eric Dumazet <edumazet@...gle.com>
> Cc: Steve Glendinning <steve.glendinning@...well.net>
> Cc: UNGLinuxDriver@...rochip.com
Reviewed-by: Simon Horman <horms@...nel.org>
...
Powered by blists - more mailing lists