Skip to content

Commit 8b9b443

Browse files
paulmckrcuurezki
authored andcommitted
rcutorture: Fix rcu_torture_one_read() pipe_count overflow comment
The "pipe_count > RCU_TORTURE_PIPE_LEN" check has a comment saying "Should not happen, but...". This is only true when testing an RCU whose grace periods are always long enough. This commit therefore fixes this comment. Reported-by: Linus Torvalds <[email protected]> Closes: https://lore.kernel.org/lkml/CAHk-=wi7rJ-eGq+xaxVfzFEgbL9tdf6Kc8Z89rCpfcQOKm74Tw@mail.gmail.com/ Signed-off-by: Paul E. McKenney <[email protected]> Signed-off-by: Uladzislau Rezki (Sony) <[email protected]>
1 parent 8d0f9a6 commit 8b9b443

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

kernel/rcu/rcutorture.c

+2-1
Original file line numberDiff line numberDiff line change
@@ -2000,7 +2000,8 @@ static bool rcu_torture_one_read(struct torture_random_state *trsp, long myid)
20002000
preempt_disable();
20012001
pipe_count = READ_ONCE(p->rtort_pipe_count);
20022002
if (pipe_count > RCU_TORTURE_PIPE_LEN) {
2003-
/* Should not happen, but... */
2003+
// Should not happen in a correct RCU implementation,
2004+
// happens quite often for torture_type=busted.
20042005
pipe_count = RCU_TORTURE_PIPE_LEN;
20052006
}
20062007
completed = cur_ops->get_gp_seq();

0 commit comments

Comments
 (0)