Thread (19 messages) flat view 19 messages, 4 authors, 2021-10-22

Re: [RFC net v7] net: skb_expand_head() adjust skb->truesize incorrectly

From: Jakub Kicinski <kuba@kernel.org>
Date: 2021-09-21 00:41:52

On Tue, 21 Sep 2021 00:41:15 +0300 Vasily Averin wrote:
quoted
Thanks for taking a look. I would prefer not to bake any ideas about
the skb's function into generic functions. Enumerating every destructor
callback in generic code is impossible (technically so, since the code
may reside in modules).

Let me think about it. Perhaps we can extend sock callbacks with
skb_sock_inherit, and skb_adjust_trusize? That'd transfer the onus of
handling the adjustments done on splitting to the protocols. I'll see
if that's feasible unless someone can immediately call this path
ghastly.  
This is similar to Alexey Kuznetsov's suggestion for me, 
see https://lkml.org/lkml/2021/8/27/460
Interesting, I wasn't thinking of keeping the ops pointer in every skb.
However I think we can do it later,
right now we need to fix somehow broken skb_expand_head(),
please take look at v8.
I think v8 still has the issue that Eric was explaining over and over.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help