linux-kernel - Re: [brauner-vfs:work.f_version.wip.v1] [proc] e85c0d9e72: WARNING:lock_held_when_returning_to_user

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: <20240905-wegschauen-intellektuell-c180f82feafe@brauner>
Date: Thu, 5 Sep 2024 11:10:16 +0200
From: Christian Brauner <brauner@...nel.org>
To: Oliver Sang <oliver.sang@...el.com>
Cc: oe-lkp@...ts.linux.dev, lkp@...el.com, 
	Christian Brauner <christianvanbrauner@...il.com>, linux-kernel@...r.kernel.org, linux-fsdevel@...r.kernel.org
Subject: Re: [brauner-vfs:work.f_version.wip.v1] [proc] e85c0d9e72:
 WARNING:lock_held_when_returning_to_user_space

On Thu, Sep 05, 2024 at 10:07:17AM GMT, Oliver Sang wrote:
> hi, Christian Brauner,
> 
> On Tue, Sep 03, 2024 at 03:59:19PM +0200, Christian Brauner wrote:
> > On Tue, Sep 03, 2024 at 09:53:05PM GMT, kernel test robot wrote:
> > > 
> > > 
> > > Hello,
> > > 
> > > kernel test robot noticed "WARNING:lock_held_when_returning_to_user_space" on:
> > > 
> > > commit: e85c0d9e725529a5ed68ad0b6abc62b332654156 ("proc: wean of off f_version")
> > > https://git.kernel.org/cgit/linux/kernel/git/vfs/vfs.git work.f_version.wip.v1
> > 
> > This is once again an old branch that's dead and was never sent.
> > Can you please exclude anything that has a *.v<nr> suffix? I thought I
> > already added a commit to this effect to the repository.
> 
> got it. seems previous deny pattern has a small problem.
> 
> Philip just pushed a fix [1]
> 
> at the same time, we noticed for v<nr>, there are two different styles in 3
> repos owned by your that we are monitoring.
> style #1: _v<nr>
> style #2: .v<nr>
> 
> should we deny both or handle them differently for different repo?

I think only using the .v<nr> style is fine.

Thank you! :)