Skip to content

Commit 69dcbbd

Browse files
paulmckrcuneeraju
authored andcommitted
locktorture: Increase Hamming distance between call_rcu_chain and rcu_call_chains
One letter difference is really not enough, so this commit changes call_rcu_chain to call_rcu_chain_list. Reported-by: Dan Carpenter <dan.carpenter@linaro.org> Signed-off-by: Paul E. McKenney <paulmck@kernel.org> Signed-off-by: Neeraj Upadhyay (AMD) <neeraj.iitr10@gmail.com>
1 parent 98b1cc8 commit 69dcbbd

File tree

1 file changed

+9
-9
lines changed

1 file changed

+9
-9
lines changed

kernel/locking/locktorture.c

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -124,7 +124,7 @@ struct call_rcu_chain {
124124
struct rcu_head crc_rh;
125125
bool crc_stop;
126126
};
127-
struct call_rcu_chain *call_rcu_chain;
127+
struct call_rcu_chain *call_rcu_chain_list;
128128

129129
/* Forward reference. */
130130
static void lock_torture_cleanup(void);
@@ -1074,12 +1074,12 @@ static int call_rcu_chain_init(void)
10741074

10751075
if (call_rcu_chains <= 0)
10761076
return 0;
1077-
call_rcu_chain = kcalloc(call_rcu_chains, sizeof(*call_rcu_chain), GFP_KERNEL);
1078-
if (!call_rcu_chain)
1077+
call_rcu_chain_list = kcalloc(call_rcu_chains, sizeof(*call_rcu_chain_list), GFP_KERNEL);
1078+
if (!call_rcu_chain_list)
10791079
return -ENOMEM;
10801080
for (i = 0; i < call_rcu_chains; i++) {
1081-
call_rcu_chain[i].crc_stop = false;
1082-
call_rcu(&call_rcu_chain[i].crc_rh, call_rcu_chain_cb);
1081+
call_rcu_chain_list[i].crc_stop = false;
1082+
call_rcu(&call_rcu_chain_list[i].crc_rh, call_rcu_chain_cb);
10831083
}
10841084
return 0;
10851085
}
@@ -1089,13 +1089,13 @@ static void call_rcu_chain_cleanup(void)
10891089
{
10901090
int i;
10911091

1092-
if (!call_rcu_chain)
1092+
if (!call_rcu_chain_list)
10931093
return;
10941094
for (i = 0; i < call_rcu_chains; i++)
1095-
smp_store_release(&call_rcu_chain[i].crc_stop, true);
1095+
smp_store_release(&call_rcu_chain_list[i].crc_stop, true);
10961096
rcu_barrier();
1097-
kfree(call_rcu_chain);
1098-
call_rcu_chain = NULL;
1097+
kfree(call_rcu_chain_list);
1098+
call_rcu_chain_list = NULL;
10991099
}
11001100

11011101
static void lock_torture_cleanup(void)

0 commit comments

Comments
 (0)