Re: [PATCH net-next 01/16] qlge: Remove irq_cnt

2019-07-15 Thread Greg Kroah-Hartman
On Mon, Jul 15, 2019 at 10:40:16AM +0900, Benjamin Poirier wrote: > On 2019/06/17 16:48, Benjamin Poirier wrote: > > qlge uses an irq enable/disable refcounting scheme that is: > > * poorly implemented > > Uses a spin_lock to protect accesses to the irq_cnt atomic variable > > * buggy > > B

Re: [PATCH net-next 01/16] qlge: Remove irq_cnt

2019-07-14 Thread Benjamin Poirier
On 2019/06/17 16:48, Benjamin Poirier wrote: > qlge uses an irq enable/disable refcounting scheme that is: > * poorly implemented > Uses a spin_lock to protect accesses to the irq_cnt atomic variable > * buggy > Breaks when there is not a 1:1 sequence of irq - napi_poll, such as >

Re: [PATCH net-next 01/16] qlge: Remove irq_cnt

2019-07-05 Thread Benjamin Poirier
On 2019/06/26 13:21, Manish Chopra wrote: > > In msix mode there's no need to explicitly disable completion interrupts, > > they > > are reliably auto-masked, according to my observations. > > I tested this on two QLE8142 adapters. > > > > Do you have reason to believe this might not always be th

RE: [PATCH net-next 01/16] qlge: Remove irq_cnt

2019-06-26 Thread Manish Chopra
> In msix mode there's no need to explicitly disable completion interrupts, they > are reliably auto-masked, according to my observations. > I tested this on two QLE8142 adapters. > > Do you have reason to believe this might not always be the case? How did you check auto-masking of MSI-X interrup

Re: [PATCH net-next 01/16] qlge: Remove irq_cnt

2019-06-26 Thread Benjamin Poirier
On 2019/06/26 08:59, Manish Chopra wrote: > > -Original Message- > > From: Benjamin Poirier > > Sent: Monday, June 17, 2019 1:19 PM > > To: Manish Chopra ; GR-Linux-NIC-Dev > nic-...@marvell.com>; netdev@vger.kernel.org > > Subject: [PATCH net-next 01/16] qlge: Remove irq_cnt > > > > qlg

RE: [PATCH net-next 01/16] qlge: Remove irq_cnt

2019-06-26 Thread Manish Chopra
> -Original Message- > From: Benjamin Poirier > Sent: Monday, June 17, 2019 1:19 PM > To: Manish Chopra ; GR-Linux-NIC-Dev nic-...@marvell.com>; netdev@vger.kernel.org > Subject: [PATCH net-next 01/16] qlge: Remove irq_cnt > > qlge uses an irq enable/disable refcounting scheme that is: >

RE: [PATCH net-next 01/16] qlge: Remove irq_cnt

2019-06-17 Thread Manish Chopra
> -Original Message- > From: Benjamin Poirier > Sent: Monday, June 17, 2019 1:19 PM > To: Manish Chopra ; GR-Linux-NIC-Dev nic-...@marvell.com>; netdev@vger.kernel.org > Subject: [PATCH net-next 01/16] qlge: Remove irq_cnt > > qlge uses an irq enable/disable refcounting scheme that is: >