[PATCH] rseq: Delete duplicate if statement in rseq_virt_userspace_exit()

[PATCH] rseq: Delete duplicate if statement in rseq_virt_userspace_exit()

[Dan Carpenter]

This if statement is indented weirdly.  It's a duplicate and doesn't
affect runtime (beyond wasting a little time).  Delete it.

Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
---
This was introduced in commit 32034df66b5f ("rseq: Switch to TIF_RSEQ if
supported") but it doesn't cause a bug, so there is no Fixes tag.

 include/linux/rseq.h | 1 -
 1 file changed, 1 deletion(-)

diff --git a/include/linux/rseq.h b/include/linux/rseq.h
index b5e4803c4ebe..bf8a6bf315f3 100644
--- a/include/linux/rseq.h
+++ b/include/linux/rseq.h
@@ -126,7 +126,6 @@ static inline void rseq_force_update(void)
  */
 static inline void rseq_virt_userspace_exit(void)
 {
-	if (current->rseq.event.sched_switch)
 	/*
 	 * The generic optimization for deferring RSEQ updates until the next
 	 * exit relies on having a dedicated TIF_RSEQ.
-- 
2.51.0

Re: [PATCH] rseq: Delete duplicate if statement in rseq_virt_userspace_exit()

[Thomas Gleixner]

On Tue, Nov 18 2025 at 13:52, Dan Carpenter wrote:
> This if statement is indented weirdly.  It's a duplicate and doesn't
> affect runtime (beyond wasting a little time).  Delete it.

It's actually optimized out by the compiler in both cases, but yes
that's an odd one and likely a reject fixup artifact.

Thanks for detecting it!