> On Sep 24, 2018, at 12:41 PM, Eric Dumazet <eduma...@google.com> wrote: > > On Mon, Sep 24, 2018 at 12:31 PM Song Liu <songliubrav...@fb.com> wrote: > >> This triggers dev_watchdog() on a simple netperf TCP_RR on bnxt (I haven't >> tested other drivers yet). >> >> I guess this is because NAPI_STATE_SCHED is set when poll_one_napi() calls >> napi->poll(). And then cleared by napi->poll(). > > Which part of napi->poll() could possibly clear NAPI_STATE_SCHED when > called by netpoll ? > > AFAIK, napi_complete_done() should exit early (before having a chance > to clear NAPI_STATE_SCHED) > because of : > > if (unlikely(n->state & (NAPIF_STATE_NPSVC | NAPIF_STATE_IN_BUSY_POLL))) > return false; > > Thanks ! You are right on this condition. But this does trigger dev_watchdog() for some reason. Thanks, Song
- [PATCH net 14/15] nfp: remove ndo_poll_controller Eric Dumazet
- Re: [PATCH net 14/15] nfp: remove ndo_poll_controller Jakub Kicinski
- [PATCH net 15/15] tun: remove ndo_poll_controller Eric Dumazet
- [PATCH net 13/15] bnxt: remove ndo_poll_controller Eric Dumazet
- Re: [PATCH net 00/15] netpoll: avoid capture effects for NA... David Miller
- Re: [PATCH net 00/15] netpoll: avoid capture effects for NA... David Miller
- Re: [PATCH net 00/15] netpoll: avoid capture effects f... Eric Dumazet
- Re: [PATCH net 00/15] netpoll: avoid capture effec... Song Liu
- Re: [PATCH net 00/15] netpoll: avoid capture e... Eric Dumazet
- Re: [PATCH net 00/15] netpoll: avoid captu... Song Liu
- Re: [PATCH net 00/15] netpoll: avoid ... Eric Dumazet
- Re: [PATCH net 00/15] netpoll: av... Eric Dumazet
- Re: [PATCH net 00/15] netpoll: av... Song Liu
- Re: [PATCH net 00/15] netpoll: av... Michael Chan
- Re: [PATCH net 00/15] netpoll: av... Eric Dumazet
- Re: [PATCH net 00/15] netpoll: av... Michael Chan
- Re: [PATCH net 00/15] netpoll: av... Song Liu
- Re: [PATCH net 00/15] netpoll: av... Michael Chan
- Re: [PATCH net 00/15] netpoll: avoid capture effec... Song Liu