Re: [PATCH 0/2] proc: proc_setattr for /proc/$PID/net

[Willy Tarreau]

On Sun, Jul 09, 2023 at 08:22:31PM +0200, Thomas Weißschuh wrote:
> On 2023-07-09 20:04:32+0200, Willy Tarreau wrote:
> > On Sun, Jul 09, 2023 at 07:57:27PM +0200, Thomas Weißschuh wrote:
> > > On 2023-07-09 19:27:53+0200, Willy Tarreau wrote:
> > > > On Sun, Jul 09, 2023 at 07:10:58PM +0200, Thomas Weißschuh wrote:
> > > > > On 2023-07-09 11:29:47+0200, Willy Tarreau wrote:
> > > > > > On Fri, Jun 30, 2023 at 10:06:09PM +0800, Zhangjin Wu wrote:
> > > > > >> [..]
> > > > > > 
> > > > > > Now queued, thanks!
> > > > > > Willy
> > > > > 
> > > > > Don't we need an Ack from the fs maintainers for the patch to
> > > > > fs/proc/proc_net.c ?
> > > > > 
> > > > > Personally I expected this series to go in via the fs tree because of
> > > > > that patch.
> > > > 
> > > > Gasp! You're totally right, I confused it with a test only changing
> > > > the nolibc-test file, as the chmod_net test appeared as a dependency!
> > > > Let me drop it from the series and push again.
> > > 
> > > I think if this patch now also goes in via both the nolibc/rcu trees and
> > > the fs tree it would not be great.
> > >
> > > The best way forward would probably for you to rebase your tree on top
> > > of mainline after the fs tree has introduced both patches of the series
> > > into Linus' tree and then you can drop your copy of the test removal.
> > 
> > Yeah I agree.
> > 
> > > I want to keep both patches together because I expect the fs change to
> > > be backported and if it is backported on its own it will break
> > > nolibc-test in those trees.
> > 
> > OK but we can also fix the test regardless, and mark it for backport, no ?
> 
> That should work fine, too.
> Can you add the Fixes and Cc-stable tags in your tree and let the fs
> maintainers know?

OK here's what it's like now, let me know if you'd prefer any change:

  commit 8c2e51e174ed0f998b6bd90244324a4966a55efc
  Author: Thomas Weißschuh <linux@xxxxxxxxxxxxxx>
  Date:   Sat Jun 24 12:30:46 2023 +0200

    selftests/nolibc: drop test chmod_net
    
    The test relies on /proc/$PID/net to allow chmod() operations.
    It is the only file or directory in /proc/$PID/ to allow this and a bug.
    That bug will be fixed in the next patch in the series and therefore
    the test would start failing.
    
    Link: https://lore.kernel.org/lkml/d0d111ef-edae-4760-83fb-36db84278da1@xxxxxxxx/
    Fixes: b4844fa0bdb4 ("selftests/nolibc: implement a few tests for various syscalls")
    Signed-off-by: Thomas Weißschuh <linux@xxxxxxxxxxxxxx>
    Tested-by: Zhangjin Wu <falcon@xxxxxxxxxxx>
    Signed-off-by: Willy Tarreau <w@xxxxxx>

> Or do you want me to split and resend the series?

Not needed, thank you.
Willy