Re: [PATCH v8 5/5] test: Explicit specify forwarding ports for pasta in log rotation tests

[David Gibson <david@gibson.dropbear.id.au>]

[-- Attachment #1: Type: text/plain, Size: 1548 bytes --]

On Mon, Sep 08, 2025 at 11:53:58AM +1000, David Gibson wrote:
> On Fri, Sep 05, 2025 at 01:03:40PM +0200, Stefano Brivio wrote:
> > On Fri,  5 Sep 2025 16:43:17 +1000
> > David Gibson <david@gibson.dropbear.id.au> wrote:
> > 
> > > test/pasta_options/log_to_file uses tcp_crr in a loop to generate log
> > > messages for what it's actually testing.  For this to work, pasta needs to
> > > forward two ports.  We set a PORTS variable with the options for this, but
> > > never actually use it.  The test works, because without the options we use
> > > automatic port forwarding.
> > 
> > Oops.
> > 
> > > That makes the test more fragile than it needs to be, in a way that's not
> > > at all related to what we're actually trying to test.  Avoid this possible
> > > point of failure by explicitly specifying the forwarding.
> > > 
> > > I encountered this because Fedora has now updated to a kernel with the
> > > /proc/net lseek() bug.
> > 
> > By the way, the fix is now in mainline as 2ce3d282bd50 ("proc: fix
> > missing pde_set_flags() for net proc files"). I hoped the breakage would
> > never reach distributions but I guess the timing was unlucky enough.
> 
> Apparently so.  Oh well, good to know the fix is upstream.

Hmm... Fedora is still on 6.16.4, so possibly they explicitly
backported the bad patch?  Oops.

-- 
David Gibson (he or they)	| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you, not the other way
				| around.
http://www.ozlabs.org/~dgibson

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]