cpu = cpumask_next(cpu, mask)
if (cpu >= nr_cpu_ids)
    cpu = cpumask_first(mask)

Above block is identical to:
cpu = cpumask_next_wrap(cpu, mask)

Replace it, No change in functionality or performance.
Slightly simpler code.

Reviewed-by: Yury Norov <[email protected]>
Signed-off-by: Shrikanth Hegde <[email protected]>
---
 arch/powerpc/kernel/irq.c             | 5 +----
 arch/powerpc/mm/book3s64/hash_utils.c | 4 +---
 2 files changed, 2 insertions(+), 7 deletions(-)

diff --git a/arch/powerpc/kernel/irq.c b/arch/powerpc/kernel/irq.c
index a0e8b998c9b5..f69de08ad347 100644
--- a/arch/powerpc/kernel/irq.c
+++ b/arch/powerpc/kernel/irq.c
@@ -370,10 +370,7 @@ int irq_choose_cpu(const struct cpumask *mask)
 do_round_robin:
                raw_spin_lock_irqsave(&irq_rover_lock, flags);
 
-               irq_rover = cpumask_next(irq_rover, cpu_online_mask);
-               if (irq_rover >= nr_cpu_ids)
-                       irq_rover = cpumask_first(cpu_online_mask);
-
+               irq_rover = cpumask_next_wrap(irq_rover, cpu_online_mask);
                cpuid = irq_rover;
 
                raw_spin_unlock_irqrestore(&irq_rover_lock, flags);
diff --git a/arch/powerpc/mm/book3s64/hash_utils.c 
b/arch/powerpc/mm/book3s64/hash_utils.c
index 9dc5889d6ecb..e4fcf929cb33 100644
--- a/arch/powerpc/mm/book3s64/hash_utils.c
+++ b/arch/powerpc/mm/book3s64/hash_utils.c
@@ -1299,9 +1299,7 @@ static void stress_hpt_timer_fn(struct timer_list *timer)
        if (!firmware_has_feature(FW_FEATURE_LPAR))
                tlbiel_all();
 
-       next_cpu = cpumask_next(raw_smp_processor_id(), cpu_online_mask);
-       if (next_cpu >= nr_cpu_ids)
-               next_cpu = cpumask_first(cpu_online_mask);
+       next_cpu = cpumask_next_wrap(raw_smp_processor_id(), cpu_online_mask);
        stress_hpt_timer.expires = jiffies + msecs_to_jiffies(10);
        add_timer_on(&stress_hpt_timer, next_cpu);
 }
-- 
2.47.3


Reply via email to