Thread (1 message) 1 message, 1 author, 2023-10-27

Re: GKI: [kernel6.1] Kernel panic on __udpv4_gso_segment_list_csum in rx-udp-gro-forwarding

From: Lena Wang (王娜) <hidden>
Date: 2023-10-27 09:55:12
Also in: lkml

On Fri, 2023-10-27 at 11:12 +0200, Paolo Abeni wrote:
 	 
External email : Please do not click links or open attachments until
you have verified the sender or the content.
 On Fri, 2023-10-27 at 08:17 +0000, Lena Wang (王娜) wrote:
quoted
Sorry. I don't use netdev ML before. I do have a google partner
account
quoted
lena.wang@mediatek.corp-partner.google.com, it is justed used to
report
quoted
Google bug and don't know how to send a email. I try to use text
formal
quoted
to reply mail.
I'm sorry if the following can sound not very polite, but really you
have to move this conversation on the netdev ML or to the google
support.

Cheers,

Paolo
Hi Paolo,
Sorry, I add the netdev in cc and if it is on netdev ML. 
==========================
I have merged your patch in the load and the kernel panic happens with
your patch.  The crashed skb with cloned = 0 and users = 2. I am afraid
the patch doesn't work because the shared skb is not checked.

I am not sure if you could submit your previous patch with:
 +       if (WARN_ON_ONCE(skb_shared(skb))) {
 +               skb =
skb_share_check(skb, GFP_ATOMIC);
 +               if (!skb)

+                       goto err_linearize;
 +       }



 diff --git a/net/core/skbuff.c b/net/core/skbuff.c
 index cea28d30abb5..1b2394ebaf33 100644
 --- a/net/core/skbuff.c
 +++ b/net/core/skbuff.c
 @@ -4270,6 +4270,17 @@ struct sk_buff *skb_segment_list(struct sk_buff
*skb,

         skb_push(skb, -skb_network_offset(skb) + offset);
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help