Re: [net-next PATCH v2 3/4] netdev: replace napi_reschedule with napi_schedule

[Eric Dumazet]

On Tue, Oct 3, 2023 at 8:36 PM Christian Marangi <ansuels...@gmail.com> wrote:
>
> Now that napi_schedule return a bool, we can drop napi_reschedule that
> does the same exact function. The function comes from a very old commit
> bfe13f54f502 ("ibm_emac: Convert to use napi_struct independent of struct
> net_device") and the purpose is actually deprecated in favour of
> different logic.
>
> Convert every user of napi_reschedule to napi_schedule.
>
> Signed-off-by: Christian Marangi <ansuels...@gmail.com>
> Acked-by: Jeff Johnson <quic_jjohn...@quicinc.com> # ath10k
> Acked-by: Nick Child <nnac...@linux.ibm.com> # ibm
> Acked-by: Marc Kleine-Budde <m...@pengutronix.de> # for can/dev/rx-offload.c

OK, but I suspect some users of napi_reschedule() might not be race-free...

Reviewed-by: Eric Dumazet <eduma...@google.com>