[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <aRym7c2crY6rmyKN@fedora>
Date: Tue, 18 Nov 2025 22:33:41 +0530
From: Ankit Khushwaha <ankitkhushwaha.linux@...il.com>
To: André Almeida <andrealmeid@...lia.com>
Cc: linux-kernel@...r.kernel.org, Ingo Molnar <mingo@...hat.com>,
Darren Hart <dvhart@...radead.org>,
Thomas Gleixner <tglx@...utronix.de>, Shuah Khan <shuah@...nel.org>,
Peter Zijlstra <peterz@...radead.org>,
linux-kselftest@...r.kernel.org,
Davidlohr Bueso <dave@...olabs.net>
Subject: Re: [PATCH] selftests/futex: Fix storing address of local variable
On Tue, Nov 18, 2025 at 01:34:59PM -0300, André Almeida wrote:
> > diff --git a/tools/testing/selftests/futex/functional/robust_list.c b/tools/testing/selftests/futex/functional/robust_list.c
> > index e7d1254e18ca..d1aab1cc5a37 100644
> > --- a/tools/testing/selftests/futex/functional/robust_list.c
> > +++ b/tools/testing/selftests/futex/functional/robust_list.c
> > @@ -510,7 +510,7 @@ TEST(test_robust_list_multiple_elements)
> > static int child_circular_list(void *arg)
> > {
> > static struct robust_list_head head;
> > - struct lock_struct a, b, c;
> > + static struct lock_struct a, b, c;
>
> I believe that the right fix here would be to drop the static from `head`
> declaration, WDYT?
Hi André,
I have tested your suggested changes.
It works fine. will send v2 patch for this.
Thanks,
-- Ankit
Powered by blists - more mailing lists