From: Shrikanth Hegde <sshegde@linux.ibm.com>
To: Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
linux-kernel@vger.kernel.org
Cc: "André Almeida" <andrealmeid@igalia.com>,
"Darren Hart" <dvhart@infradead.org>,
"Davidlohr Bueso" <dave@stgolabs.net>,
"Ingo Molnar" <mingo@redhat.com>,
"Juri Lelli" <juri.lelli@redhat.com>,
"Peter Zijlstra" <peterz@infradead.org>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Valentin Schneider" <vschneid@redhat.com>,
"Waiman Long" <longman@redhat.com>
Subject: Re: [PATCH v11 00/19] futex: Add support task local hash maps, FUTEX2_NUMA and FUTEX2_MPOL
Date: Thu, 10 Apr 2025 23:21:49 +0530 [thread overview]
Message-ID: <c2aa89d7-c3ab-40e2-bd65-443a75b04017@linux.ibm.com> (raw)
In-Reply-To: <20250407155742.968816-1-bigeasy@linutronix.de>
On 4/7/25 21:27, Sebastian Andrzej Siewior wrote:
> this is a follow up on
> https://lore.kernel.org/ZwVOMgBMxrw7BU9A@jlelli-thinkpadt14gen4.remote.csb
>
> and adds support for task local futex_hash_bucket.
>
> This is the local hash map series with PeterZ FUTEX2_NUMA and
> FUTEX2_MPOL plus a few fixes on top.
>
> The complete tree is at
> https://git.kernel.org/pub/scm/linux/kernel/git/bigeasy/staging.git/log/?h=futex_local_v11
> https://git.kernel.org/pub/scm/linux/kernel/git/bigeasy/staging.git futex_local_v11
>
> v10…v11: https://lore.kernel.org/all/20250312151634.2183278-1-bigeasy@linutronix.de
> - PeterZ' fixups, changes to the local hash series have been folded
> into the earlier patches so things are not added and renamed later
> and the functionality is changed.
>
> - vmalloc_huge() has been implemented on top of vmalloc_huge_node()
> and the NOMMU bots have been adjusted. akpm asked for this.
>
> - wake_up_var() has been removed from __futex_pivot_hash(). It is
> enough to wake the userspace waiter after the final put so it can
> perform the resize itself.
>
> - Changed to logic in futex_pivot_pending() so it does not block for
> the user. It waits for __futex_pivot_hash() which follows the logic
> in __futex_pivot_hash().
>
> - Updated kernel doc for __futex_hash().
I don't see any change in Documentation/*. Is it updated in some
other path?
>
> - Patches 17+ are new:
> - Wire up PR_FUTEX_HASH_SET_SLOTS in "perf bench futex"
> - Add "immutable" mode to PR_FUTEX_HASH_SET_SLOTS to avoid resizing
> the local hash any further. This avoids rcuref usage which is
> noticeable in "perf bench futex hash"
>
> Peter Zijlstra (8):
> mm: Add vmalloc_huge_node()
> futex: Move futex_queue() into futex_wait_setup()
> futex: Pull futex_hash() out of futex_q_lock()
> futex: Create hb scopes
> futex: Create futex_hash() get/put class
> futex: Create private_hash() get/put class
> futex: Implement FUTEX2_NUMA
> futex: Implement FUTEX2_MPOL
>
> Sebastian Andrzej Siewior (11):
> rcuref: Provide rcuref_is_dead().
> futex: Acquire a hash reference in futex_wait_multiple_setup().
> futex: Decrease the waiter count before the unlock operation.
> futex: Introduce futex_q_lockptr_lock().
> futex: Create helper function to initialize a hash slot.
> futex: Add basic infrastructure for local task local hash.
> futex: Allow automatic allocation of process wide futex hash.
> futex: Allow to resize the private local hash.
> tools headers: Synchronize prctl.h ABI header
> tools/perf: Allow to select the number of hash buckets.
> futex: Allow to make the private hash immutable.
>
> include/linux/futex.h | 36 +-
> include/linux/mm_types.h | 7 +-
> include/linux/mmap_lock.h | 4 +
> include/linux/rcuref.h | 22 +-
> include/linux/vmalloc.h | 9 +-
> include/uapi/linux/futex.h | 10 +-
> include/uapi/linux/prctl.h | 6 +
> init/Kconfig | 10 +
> io_uring/futex.c | 4 +-
> kernel/fork.c | 24 +
> kernel/futex/core.c | 794 ++++++++++++++++++++++---
> kernel/futex/futex.h | 73 ++-
> kernel/futex/pi.c | 300 +++++-----
> kernel/futex/requeue.c | 480 +++++++--------
> kernel/futex/waitwake.c | 201 ++++---
> kernel/sys.c | 4 +
> mm/nommu.c | 18 +-
> mm/vmalloc.c | 11 +-
> tools/include/uapi/linux/prctl.h | 44 +-
> tools/perf/bench/Build | 1 +
> tools/perf/bench/futex-hash.c | 7 +
> tools/perf/bench/futex-lock-pi.c | 5 +
> tools/perf/bench/futex-requeue.c | 6 +
> tools/perf/bench/futex-wake-parallel.c | 9 +-
> tools/perf/bench/futex-wake.c | 4 +
> tools/perf/bench/futex.c | 60 ++
> tools/perf/bench/futex.h | 5 +
> 27 files changed, 1585 insertions(+), 569 deletions(-)
> create mode 100644 tools/perf/bench/futex.c
>
next prev parent reply other threads:[~2025-04-10 17:52 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-07 15:57 [PATCH v11 00/19] futex: Add support task local hash maps, FUTEX2_NUMA and FUTEX2_MPOL Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 01/19] rcuref: Provide rcuref_is_dead() Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 02/19] mm: Add vmalloc_huge_node() Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 03/19] futex: Move futex_queue() into futex_wait_setup() Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 04/19] futex: Pull futex_hash() out of futex_q_lock() Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 05/19] futex: Create hb scopes Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 06/19] futex: Create futex_hash() get/put class Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 07/19] futex: Create private_hash() " Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 08/19] futex: Acquire a hash reference in futex_wait_multiple_setup() Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 09/19] futex: Decrease the waiter count before the unlock operation Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 10/19] futex: Introduce futex_q_lockptr_lock() Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 11/19] futex: Create helper function to initialize a hash slot Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 12/19] futex: Add basic infrastructure for local task local hash Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 13/19] futex: Allow automatic allocation of process wide futex hash Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 14/19] futex: Allow to resize the private local hash Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 15/19] futex: Implement FUTEX2_NUMA Sebastian Andrzej Siewior
2025-04-07 16:52 ` Sebastian Andrzej Siewior
2025-04-17 15:34 ` Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 16/19] futex: Implement FUTEX2_MPOL Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 17/19] tools headers: Synchronize prctl.h ABI header Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 18/19] tools/perf: Allow to select the number of hash buckets Sebastian Andrzej Siewior
2025-04-07 15:57 ` [PATCH v11 19/19] futex: Allow to make the private hash immutable Sebastian Andrzej Siewior
2025-04-10 10:56 ` Sebastian Andrzej Siewior
2025-04-10 14:52 ` Shrikanth Hegde
2025-04-10 15:28 ` Sebastian Andrzej Siewior
2025-04-10 15:48 ` Shrikanth Hegde
2025-04-07 16:00 ` [PATCH v11 00/19] futex: Add support task local hash maps, FUTEX2_NUMA and FUTEX2_MPOL Sebastian Andrzej Siewior
2025-04-08 13:51 ` André Almeida
2025-04-08 16:13 ` Sebastian Andrzej Siewior
2025-04-10 17:51 ` Shrikanth Hegde [this message]
2025-04-15 9:03 ` Sebastian Andrzej Siewior
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=c2aa89d7-c3ab-40e2-bd65-443a75b04017@linux.ibm.com \
--to=sshegde@linux.ibm.com \
--cc=andrealmeid@igalia.com \
--cc=bigeasy@linutronix.de \
--cc=dave@stgolabs.net \
--cc=dvhart@infradead.org \
--cc=juri.lelli@redhat.com \
--cc=linux-kernel@vger.kernel.org \
--cc=longman@redhat.com \
--cc=mingo@redhat.com \
--cc=peterz@infradead.org \
--cc=tglx@linutronix.de \
--cc=vschneid@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox