Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Thomas Gleixner <tglx@linutronix.de>
To:     Peter Zijlstra <peterz@infradead.org>,
        =?utf-8?Q?Andr=C3=A9?= Almeida <andrealmeid@collabora.com>
Cc:     Ingo Molnar <mingo@redhat.com>, Darren Hart <dvhart@infradead.org>,
        linux-kernel@vger.kernel.org, Steven Rostedt <rostedt@goodmis.org>,
        Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
        kernel@collabora.com, krisman@collabora.com,
        pgriffais@valvesoftware.com, z.figura12@gmail.com,
        joel@joelfernandes.org, malteskarupke@fastmail.fm,
        linux-api@vger.kernel.org, fweimer@redhat.com,
        libc-alpha@sourceware.org, linux-kselftest@vger.kernel.org,
        shuah@kernel.org, acme@kernel.org, corbet@lwn.net,
        Peter Oskolkov <posk@posk.io>
Subject: Re: [PATCH v3 00/13] Add futex2 syscalls
In-Reply-To: <YJKQLkHuTH3EWJoR@hirez.programming.kicks-ass.net>
References: <20210427231248.220501-1-andrealmeid@collabora.com> <YJKQLkHuTH3EWJoR@hirez.programming.kicks-ass.net>
Date:   Wed, 05 May 2021 16:23:52 +0200
Message-ID: <87bl9pi7if.ffs@nanos.tec.linutronix.de>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Wed, May 05 2021 at 14:31, Peter Zijlstra wrote:

> On Tue, Apr 27, 2021 at 08:12:35PM -0300, Andr=C3=A9 Almeida wrote:
>> Hi,
>>=20
>> This patch series introduces the futex2 syscalls.
>
> I still utterly detest that this adds a second hash-table for no
> descernable reason.
>
> The new syscall interface does not depend on that in any way, you
> previously implemented the multi-wait thing in the current futex code.
>
> Like I said last time; I'm okay with the new interface, but I don't see
> why you need to reimplement the insides, that's all pointless code
> duplication.

The real question is whether we really need to model all of this along
the existing futex functionality. I wouldn't mind a new infrastructure
which addresses all the other known issues of futexes and makes the
overall design less horrible than what we have now.

But that needs input from futex users (libraries and other horrible
wrappers) to figure out what they really need, hate, like or do not care
about.

Without that we are bound to pile more crap on the existing pile of
horrors forever.

Thanks,

        tglx