locktorture: Increase Hamming distance between call_rcu_chain and rcu_call_chains
authorPaul E. McKenney <paulmck@kernel.org>
Tue, 10 Oct 2023 16:52:19 +0000 (09:52 -0700)
committerNeeraj Upadhyay (AMD) <neeraj.iitr10@gmail.com>
Thu, 23 Nov 2023 06:28:18 +0000 (11:58 +0530)
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>
kernel/locking/locktorture.c

index 69d3cd2cfc3baf1062b6a785eeca2a834fca0171..415d81e6ce7072faf1edeecb0171bc486f6ca5bf 100644 (file)
@@ -124,7 +124,7 @@ struct call_rcu_chain {
        struct rcu_head crc_rh;
        bool crc_stop;
 };
-struct call_rcu_chain *call_rcu_chain;
+struct call_rcu_chain *call_rcu_chain_list;
 
 /* Forward reference. */
 static void lock_torture_cleanup(void);
@@ -1074,12 +1074,12 @@ static int call_rcu_chain_init(void)
 
        if (call_rcu_chains <= 0)
                return 0;
-       call_rcu_chain = kcalloc(call_rcu_chains, sizeof(*call_rcu_chain), GFP_KERNEL);
-       if (!call_rcu_chain)
+       call_rcu_chain_list = kcalloc(call_rcu_chains, sizeof(*call_rcu_chain_list), GFP_KERNEL);
+       if (!call_rcu_chain_list)
                return -ENOMEM;
        for (i = 0; i < call_rcu_chains; i++) {
-               call_rcu_chain[i].crc_stop = false;
-               call_rcu(&call_rcu_chain[i].crc_rh, call_rcu_chain_cb);
+               call_rcu_chain_list[i].crc_stop = false;
+               call_rcu(&call_rcu_chain_list[i].crc_rh, call_rcu_chain_cb);
        }
        return 0;
 }
@@ -1089,13 +1089,13 @@ static void call_rcu_chain_cleanup(void)
 {
        int i;
 
-       if (!call_rcu_chain)
+       if (!call_rcu_chain_list)
                return;
        for (i = 0; i < call_rcu_chains; i++)
-               smp_store_release(&call_rcu_chain[i].crc_stop, true);
+               smp_store_release(&call_rcu_chain_list[i].crc_stop, true);
        rcu_barrier();
-       kfree(call_rcu_chain);
-       call_rcu_chain = NULL;
+       kfree(call_rcu_chain_list);
+       call_rcu_chain_list = NULL;
 }
 
 static void lock_torture_cleanup(void)