| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Thu, 16 Oct 2014 20:22:08 +0530 From: Sudip Mukherjee <sudipm.mukherjee@...il.com> To: Felipe Balbi <balbi@...com> Cc: Paul Zimmerman <paulz@...opsys.com>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org Subject: Re: [PATCH v2 2/2] usb: dwc2: gadget: sparse warning of context imbalance On Thu, Oct 16, 2014 at 08:21:55AM -0500, Felipe Balbi wrote: > HI, > > On Thu, Oct 16, 2014 at 01:11:10PM +0530, Sudip Mukherjee wrote: > > sparse was giving the following warning: > > warning: context imbalance in 's3c_hsotg_ep_enable' > > - different lock contexts for basic block > > > > we were returning ENOMEM while still holding the spinlock. > > The sparse warning was fixed by releasing the spinlock before return. > > > > This patch depends on the previous patch of the series. > > > > Signed-off-by: Sudip Mukherjee <sudip@...torindia.org> > > this should be patch one so it can be backported to stable kernels. > my v1 patch fixed only this , while reviewing that one Paul Zimmerman suggested to rewrite the return statements. so this v2 series had the rewrite and the spinlock error fix. now if this is to be made the patch one then it will be a duplicate of my v1 followed by another patch for return statements. should i do that ? thanks sudip > -- > balbi -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists