diff options
| author |   Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2018-04-19 09:05:50 -0700 |
|---|---|---|
| committer | Paul E. McKenney <paulmck@linux.vnet.ibm.com> | 2018-05-15 10:30:15 -0700 |
| commit | a508aa597ec2f046c00b8809f887f90cf1aaa47f (patch) | |
| tree | 9a6804bcde0ef2bac5292ad098be1f7fb61c36d7 /kernel | |
| parent | rcu: Switch __rcu_process_callbacks() to rcu_accelerate_cbs() (diff) | |
| download | linux-dev-a508aa597ec2f046c00b8809f887f90cf1aaa47f.tar.xz linux-dev-a508aa597ec2f046c00b8809f887f90cf1aaa47f.zip | |
rcu: Cleanup, don't put ->completed into an int
It is true that currently only the low-order two bits are used, so
there should be no problem given modern machines and compilers, but
good hygiene and maintainability dictates use of an unsigned long
instead of an int. This commit therefore makes this change.
Signed-off-by: Paul E. McKenney <paulmck@linux.vnet.ibm.com>
Tested-by: Nicholas Piggin <npiggin@gmail.com>
Diffstat (limited to 'kernel')
| -rw-r--r-- | kernel/rcu/tree.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/rcu/tree.c b/kernel/rcu/tree.c index fbacc486ed4c..c7b1e6b2a3da 100644 --- a/kernel/rcu/tree.c +++ b/kernel/rcu/tree.c @@ -1765,7 +1765,7 @@ out: */ static bool rcu_future_gp_cleanup(struct rcu_state *rsp, struct rcu_node *rnp) { - int c = rnp->completed; + unsigned long c = rnp->completed; bool needmore; struct rcu_data *rdp = this_cpu_ptr(rsp->rda); |