lkml.org 
[lkml]   [2020]   [Oct]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 0/3] net: dsa: move skb reallocation to dsa_slave_xmit
Date


On 10/16/2020 1:02 PM, Christian Eggers wrote:

[snip]

> On Friday, 16 October 2020, 20:03:11 CEST, Jakub Kicinski wrote:
>> FWIW if you want to avoid the reallocs you may want to set
>> needed_tailroom on the netdev.
> I haven't looked for this yet. If this can really solve the tagging AND
> padding problem, I would like to do this in a follow up patch.

The comment in netdevice.h says:

* @needed_headroom: Extra headroom the hardware may need, but
not in all
* cases can this be guaranteed
* @needed_tailroom: Extra tailroom the hardware may need, but
not in all
* cases can this be guaranteed. Some cases
also use
* LL_MAX_HEADER instead to allocate the skb

and while I have never seen a reallocation occur while pushing a
descriptor status block in front of a frame on transmit after setting
the correct needed_headroom, it was not exercised in a very complicated
way either, just TCP or UDP over IPv4 or IPv6. This makes me think that
the comment is cautionary about more complicated transmit scenarios with
stacked devices, tunneling etc.

>
> Wishing a nice weekend for netdev.

Likewise!
--
Florian

\
 
 \ /
  Last update: 2020-10-17 08:03    [W:0.195 / U:0.040 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site