> -----Original Message----- > From: [email protected] [mailto:[email protected]] > On Behalf Of Sergei Shtylyov > Sent: Friday, September 28, 2018 2:02 AM > To: Kirsher, Jeffrey T <[email protected]>; [email protected] > Cc: Keller, Jacob E <[email protected]>; [email protected]; > [email protected]; [email protected]; [email protected]; > Venkataramanan, Anirudh <[email protected]> > Subject: Re: [net-next 1/8] ice: use [sr]q.count when checking if queue is > initialized > > Hello! > > On 9/27/2018 7:21 PM, Jeff Kirsher wrote: > > > From: Jacob Keller <[email protected]> > > > > When shutting down the controlqs, we check if they are initialized > > before we shut them down and destroy the lock. This is important, as it > > prevents attempts to access the lock of an already shutdown queue. > > > > Unfortunately, we checked rq.head and sq.head as the value to determine > > if the queue was initialized. This doesn't work, because head is not > > reset when the queue is shutdown. In some flows, the adminq will have > > already been shut down prior to calling ice_shutdown_all_ctrlqs. This > > can result in a crash due to attempting to access the already destroyed > > mutex. > > > > Fix this by using rq.count and sq.count instead. Indeed, ice_shutdown_sq > > and ice_shutdown_rq already indicate that this is the value we should be > > using to determine of the queue was initialized. > > s/of/if/?
Yes. However, unless there's a functional change required as well, I don't think it's worth fixing this. Thanks, Jake > > > Signed-off-by: Jacob Keller <[email protected]> > > Signed-off-by: Anirudh Venkataramanan > <[email protected]> > > Tested-by: Andrew Bowers <[email protected]> > > Signed-off-by: Jeff Kirsher <[email protected]> > [...] > > MBR, Sergei
