> > Interesting, maybe a bnxt specific issue. > > It seems their model is to process TX/RX notification in the same queue, > they throw away RX events if budget == 0 > > It means commit e7b9569102995ebc26821789628eef45bd9840d8 is wrong and > must be reverted. > > Otherwise, we have a possibility of blocking a queue under netpoll pressure.
Hmm, actually a revert might not be enough, since code at lines 2030-2031 would fire and we might not call napi_complete_done() anyway. Unfortunately this driver logic is quite complex. Could you test on other NIC eventually ? Thanks.