linux-kernel - RE: drivers/scsi/fnic/fdls_disc.c:263 fdls_schedule_oxid_free_retry

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:
 <SJ0PR11MB589636FC2FF34B7DE8A14B2DC3C42@SJ0PR11MB5896.namprd11.prod.outlook.com>
Date: Thu, 20 Feb 2025 22:13:45 +0000
From: "Karan Tilak Kumar (kartilak)" <kartilak@...co.com>
To: Dan Carpenter <dan.carpenter@...aro.org>
CC: "oe-kbuild@...ts.linux.dev" <oe-kbuild@...ts.linux.dev>, "lkp@...el.com"
	<lkp@...el.com>, "oe-kbuild-all@...ts.linux.dev"
	<oe-kbuild-all@...ts.linux.dev>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "Martin K. Petersen"
	<martin.petersen@...cle.com>, "Sesidhar Baddela (sebaddel)"
	<sebaddel@...co.com>, "Gian Carlo Boffa (gcboffa)" <gcboffa@...co.com>,
	"Arulprabhu Ponnusamy (arulponn)" <arulponn@...co.com>, "Arun Easi (aeasi)"
	<aeasi@...co.com>
Subject: RE: drivers/scsi/fnic/fdls_disc.c:263
 fdls_schedule_oxid_free_retry_work() warn: inconsistent indenting

On Wednesday, February 19, 2025 9:50 PM, Dan Carpenter <dan.carpenter@...aro.org> wrote:
>
> On Thu, Feb 20, 2025 at 03:10:26AM +0000, Karan Tilak Kumar (kartilak) wrote:
> > On Saturday, February 15, 2025 1:03 PM, Karan Tilak Kumar (kartilak) wrote:
> > >
> > > On Friday, February 14, 2025 7:03 AM, Dan Carpenter <dan.carpenter@...aro.org> wrote:
> > > >
> > > > On Fri, Feb 14, 2025 at 05:42:25PM +0300, Dan Carpenter wrote:
> > > > >
> > > > > I guess we can't call schedule_delayed_work() without holding the
> > > > > spin_lock?  It's a strange thing.
> > > >
> > > > Actually, I see now that we're just dropping the lock to avoid a sleeping
> > > > in atomic warning with the GFP_ATOMIC allocation...  I bet you could make
> > > > the code under the lock much smaller.  Just the test_and_clear_bit() stuff
> > > > basically.
> > > >
> > > > regards,
> > > > dan carpenter
> >
> > The window can be further reduced by adding the lock/unlock around the list_add_tail. But this looks good too.
> >
>
> Could you send the patch for this.  I don't know the code that well and
> can't test it.
>
> regards,
> dan carpenter
>
>

Sure Dan. I'll test the code and send out the patches.

Regards,
Karan