From: Shan Wei <davids...@tencent.com>

For bottom halves off, __this_cpu_read is better.

Signed-off-by: Shan Wei <davids...@tencent.com>
Reviewed-by: Christoph Lameter <c...@linux.com>
---
no changes vs v4.
---
 kernel/padata.c |    5 ++---
 1 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/kernel/padata.c b/kernel/padata.c
index 89fe3d1..072f4ee 100644
--- a/kernel/padata.c
+++ b/kernel/padata.c
@@ -171,7 +171,7 @@ static struct padata_priv *padata_get_next(struct 
parallel_data *pd)
 {
        int cpu, num_cpus;
        unsigned int next_nr, next_index;
-       struct padata_parallel_queue *queue, *next_queue;
+       struct padata_parallel_queue *next_queue;
        struct padata_priv *padata;
        struct padata_list *reorder;
 
@@ -204,8 +204,7 @@ static struct padata_priv *padata_get_next(struct 
parallel_data *pd)
                goto out;
        }
 
-       queue = per_cpu_ptr(pd->pqueue, smp_processor_id());
-       if (queue->cpu_index == next_queue->cpu_index) {
+       if (__this_cpu_read(pd->pqueue->cpu_index) == next_queue->cpu_index) {
                padata = ERR_PTR(-ENODATA);
                goto out;
        }
-- 
1.7.1


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to