]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
sched: Remove never used code in mm_cid_get()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Wed, 15 Oct 2025 09:19:34 +0000 (11:19 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 29 Oct 2025 13:08:59 +0000 (14:08 +0100)
[ Upstream commit 53abe3e1c154628cc74e33a1bfcd865656e433a5 ]

Clang is not happy with set but unused variable (this is visible
with `make W=1` build:

  kernel/sched/sched.h:3744:18: error: variable 'cpumask' set but not used [-Werror,-Wunused-but-set-variable]

It seems like the variable was never used along with the assignment
that does not have side effects as far as I can see.  Remove those
altogether.

Fixes: 223baf9d17f2 ("sched: Fix performance regression introduced by mm_cid")
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Tested-by: Eric Biggers <ebiggers@kernel.org>
Reviewed-by: Breno Leitao <leitao@debian.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
kernel/sched/sched.h

index a441990fe808d856a4f7ebc5b6ff8535e6bae27c..cf541c4502d92a263cbfd6fcf2f2df004d785e02 100644 (file)
@@ -3731,11 +3731,9 @@ end:
 static inline int mm_cid_get(struct rq *rq, struct mm_struct *mm)
 {
        struct mm_cid __percpu *pcpu_cid = mm->pcpu_cid;
-       struct cpumask *cpumask;
        int cid;
 
        lockdep_assert_rq_held(rq);
-       cpumask = mm_cidmask(mm);
        cid = __this_cpu_read(pcpu_cid->cid);
        if (mm_cid_is_valid(cid)) {
                mm_cid_snapshot_time(rq, mm);