Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1631442 > unrolled thread

Re: [PATCH] ezchip: nps_enet: check if napi has been completed

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH] ezchip: nps_enet: check if napi has been completed Vlad Zakharov <Vladislav.Zakharov@synopsys.com> - 2017-04-26 15:00 +0200

#1631442 — Re: [PATCH] ezchip: nps_enet: check if napi has been completed

Hi David, all,

On Wed, 2017-03-29 at 14:30 -0700, David Miller wrote:
> From: Vlad Zakharov <Vladislav.Zakharov@synopsys.com>
> Date: Wed, 29 Mar 2017 13:41:46 +0300
> 
> > 
> > After a new NAPI_STATE_MISSED state was added to NAPI we can get into
> > this state and in such case we have to reschedule NAPI as some work is
> > still pending and we have to process it. napi_complete_done() function
> > returns false if we have to reschedule something (e.g. in case we were
> > in MISSED state) as current polling have not been completed yet.
> > 
> > nps_enet driver hasn't been verifying the return value of
> > napi_complete_done() and has been forcibly enabling interrupts. That is
> > not correct as we should not enable interrupts before we have processed
> > all scheduled work. As a result we were getting trapped in interrupt
> > hanlder chain as we had never been able to disabale ethernet
> > interrupts again.
> > 
> > So this patch makes nps_enet_poll() func verify return value of
> > napi_complete_done() and enable interrupts only in case all scheduled
> > work has been completed.
> > 
> > Signed-off-by: Vlad Zakharov <vzakhar@synopsys.com>
> 
> Applied.
> 
> Eric, if this is really required now, we have 148 broken drivers still.

Could you please backport this patch to stable tree (starting from 4.10) as it is pretty important and fixes nps_enet
driver?

It became actual after Eric's commit 39e6c8208d7b (net: solve a NAPI race) which has already been backported to 4.10.

Thanks.

-- 
Best regards,
Vlad Zakharov <vzakhar@synopsys.com>

[toc] | [standalone]

Back to top | Article view | linux.kernel

csiph-web