lkml.org 
[lkml]   [2021]   [Aug]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] net: drop skbs in napi->rx_list when removing the napi context.
From
Date
On 8/12/2021 3:07 PM, Eric Dumazet wrote:
> Also I object to this fix.
>
> If packets have been stored temporarily in GRO, they should be
> released at some point,
> normally at the end of a napi poll.
>
> By released, I mean that these packets should reach the upper stack,
> instead of being dropped without
> any notification.
>
> It seems a call to gro_normal_list() is missing somewhere.
>
> Can you find where ?
>
> Thanks !
> H Eric,

I think the location that should have a call to gro_normal_list() is
__netif_napi_del(). Let say, if the driver call a function that lead to
gro_normal_one(), and add a skb to the rx_list while the napi poll is
not scheduled, and the driver remove the napi context before a napi poll
could be triggered, then the added skb will be lost.

Actually, this was the first solution that I tried with syzbot (It
passed the test too).
Best regards,
Phi

\
 
 \ /
  Last update: 2021-08-12 21:19    [W:0.092 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site