From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 169E03C7E0F for ; Mon, 30 Mar 2026 12:02:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774872134; cv=none; b=amOhO8eZdiOsV3YvMawT4ogiMKr77jXXC5yz/RHHlDv5Kup/hNg8sUbit37oB64LA22EraFUjeejtZg61yApomihC+Zq6JfPhHMigkKYtns+jYRLN0OPb0USakv3qu3ZEtE/HH0FBo+tC4D/KqWKdbbEsuIeRgLgWICHxgTttUY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774872134; c=relaxed/simple; bh=JNx/WJXVVQ79OhTAWIHreYE7NGao/vKg6SajMKIlfhs=; h=Date:Message-ID:From:To:Cc:Subject:References:MIME-Version: Content-Type; b=j04BAHI1ChRC2drfFHkPQfo7sKEAuW9sfxGl/Ty2bdQlccy97KG5UOo1bM6jiVw/5iaWNYftiFmYPYSxQvjDUTlYfJbZJXfjXG2s8RNpyg0N+P6c2Gkmpihv8MMr9nGRT4L1Ged4RtJh6gStVkpLfMXF/PfFZ5Tvv6nHXRirLr0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=i/bD99hE; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="i/bD99hE" Received: by smtp.kernel.org (Postfix) with ESMTPSA id CDE89C2BCB2; Mon, 30 Mar 2026 12:02:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1774872133; bh=JNx/WJXVVQ79OhTAWIHreYE7NGao/vKg6SajMKIlfhs=; h=Date:From:To:Cc:Subject:References:From; b=i/bD99hExFIhKmGsUfaBZWMCWDd4ihXUL2OC9ONpeYlU5nB8eOOffQMGO8m4dFlxx CokQxLt7upajPc+H5bnaWS+qL3Xc5CmsGfIxlPt73kXwWSb5NSPkhyamucAlLhVUfu /eJtlLpEe5vos/Rw3SknbXmbPw+/dM9UjIzJ263El5FOwAx3/epb9ayKzmdAQzVuw5 gbl8MmoAJ1b2DjJm0tx9FMtzlPavZuB14dMsOz2vUN9KaN8ihq0C4ij0/VBVepZ4dg PyyOcr17S5UYl3JxanL1BbejNMjsaYq3QVj+r8fpEcFsQVKyVyOG6wZ4q1R7JGpyfg Q7++2upUPLwtg== Date: Mon, 30 Mar 2026 14:02:09 +0200 Message-ID: <20260330120117.194814100@kernel.org> User-Agent: quilt/0.68 From: Thomas Gleixner To: LKML Cc: Mathieu Desnoyers , =?UTF-8?q?Andr=C3=A8=20Almeida?= , Sebastian Andrzej Siewior , Carlos O'Donell , Peter Zijlstra , Florian Weimer , Rich Felker , Torvald Riegel , Darren Hart , Ingo Molnar , Davidlohr Bueso , Arnd Bergmann , "Liam R . Howlett" , Uros Bizjak , =?UTF-8?q?Thomas=20Wei=C3=9Fschuh?= Subject: [patch V3 02/14] futex: Make futex_mm_init() void References: <20260330114212.927686587@kernel.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Nothing fails there. Mop up the leftovers of the early version of this, which did an allocation. While at it clean up the stubs and the #ifdef comments to make the header file readable. Signed-off-by: Thomas Gleixner --- include/linux/futex.h | 28 +++++++++++----------------- kernel/fork.c | 8 ++------ kernel/futex/core.c | 3 +-- 3 files changed, 14 insertions(+), 25 deletions(-) --- a/include/linux/futex.h +++ b/include/linux/futex.h @@ -81,22 +81,20 @@ int futex_hash_prctl(unsigned long arg2, #ifdef CONFIG_FUTEX_PRIVATE_HASH int futex_hash_allocate_default(void); void futex_hash_free(struct mm_struct *mm); -int futex_mm_init(struct mm_struct *mm); - -#else /* !CONFIG_FUTEX_PRIVATE_HASH */ +void futex_mm_init(struct mm_struct *mm); +#else /* CONFIG_FUTEX_PRIVATE_HASH */ static inline int futex_hash_allocate_default(void) { return 0; } static inline int futex_hash_free(struct mm_struct *mm) { return 0; } -static inline int futex_mm_init(struct mm_struct *mm) { return 0; } -#endif /* CONFIG_FUTEX_PRIVATE_HASH */ +static inline void futex_mm_init(struct mm_struct *mm) { } +#endif /* !CONFIG_FUTEX_PRIVATE_HASH */ -#else /* !CONFIG_FUTEX */ +#else /* CONFIG_FUTEX */ static inline void futex_init_task(struct task_struct *tsk) { } static inline void futex_exit_recursive(struct task_struct *tsk) { } static inline void futex_exit_release(struct task_struct *tsk) { } static inline void futex_exec_release(struct task_struct *tsk) { } -static inline long do_futex(u32 __user *uaddr, int op, u32 val, - ktime_t *timeout, u32 __user *uaddr2, - u32 val2, u32 val3) +static inline long do_futex(u32 __user *uaddr, int op, u32 val, ktime_t *timeout, + u32 __user *uaddr2, u32 val2, u32 val3) { return -EINVAL; } @@ -104,13 +102,9 @@ static inline int futex_hash_prctl(unsig { return -EINVAL; } -static inline int futex_hash_allocate_default(void) -{ - return 0; -} +static inline int futex_hash_allocate_default(void) { return 0; } static inline int futex_hash_free(struct mm_struct *mm) { return 0; } -static inline int futex_mm_init(struct mm_struct *mm) { return 0; } - -#endif +static inline void futex_mm_init(struct mm_struct *mm) { } +#endif /* !CONFIG_FUTEX */ -#endif +#endif /* _LINUX_FUTEX_H */ --- a/kernel/fork.c +++ b/kernel/fork.c @@ -1097,6 +1097,7 @@ static struct mm_struct *mm_init(struct #endif mm_init_uprobes_state(mm); hugetlb_count_init(mm); + futex_mm_init(mm); mm_flags_clear_all(mm); if (current->mm) { @@ -1109,11 +1110,8 @@ static struct mm_struct *mm_init(struct mm->def_flags = 0; } - if (futex_mm_init(mm)) - goto fail_mm_init; - if (mm_alloc_pgd(mm)) - goto fail_nopgd; + goto fail_mm_init; if (mm_alloc_id(mm)) goto fail_noid; @@ -1140,8 +1138,6 @@ static struct mm_struct *mm_init(struct mm_free_id(mm); fail_noid: mm_free_pgd(mm); -fail_nopgd: - futex_hash_free(mm); fail_mm_init: free_mm(mm); return NULL; --- a/kernel/futex/core.c +++ b/kernel/futex/core.c @@ -1717,7 +1717,7 @@ static bool futex_ref_is_dead(struct fut return atomic_long_read(&mm->futex_atomic) == 0; } -int futex_mm_init(struct mm_struct *mm) +void futex_mm_init(struct mm_struct *mm) { mutex_init(&mm->futex_hash_lock); RCU_INIT_POINTER(mm->futex_phash, NULL); @@ -1726,7 +1726,6 @@ int futex_mm_init(struct mm_struct *mm) mm->futex_ref = NULL; atomic_long_set(&mm->futex_atomic, 0); mm->futex_batches = get_state_synchronize_rcu(); - return 0; } void futex_hash_free(struct mm_struct *mm)