Commit a82adc76 authored May 17, 2021 by Pavel Begunkov Committed by Thomas Gleixner May 25, 2021

futex: Deduplicate cond_resched() invocation in futex_wake_op()

After pagefaulting in futex_wake_op() both branches do cond_resched()
before retry. Deduplicate it as compilers cannot figure it out themself.

Signed-off-by: Pavel Begunkov <asml.silence@gmail.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Reviewed-by: Davidlohr Bueso <dbueso@suse.de>
Link: https://lore.kernel.org/r/9b2588c1fd33c91fb01c4e348a3b647ab2c8baab.1621258128.git.asml.silence@gmail.com

parent f4addd54

kernel/futex.c

+2 −5

Original line number	Diff line number	Diff line
		@@ -1728,12 +1728,9 @@ futex_wake_op(u32 __user uaddr1, unsigned int flags, u32 __user uaddr2,
		return ret;
		}

		if (!(flags & FLAGS_SHARED)) {
		cond_resched();
		if (!(flags & FLAGS_SHARED))
		goto retry_private;
		}

		cond_resched();
		goto retry;
		}