Commit d7b381bb authored by Mike Travis's avatar Mike Travis
Browse files

x86: fixup_irqs() doesnt need an argument. 



Impact: cleanup, remove on-stack cpumask.

The "map" arg is always cpu_online_mask.  Importantly, set_affinity
always ands the argument with cpu_online_mask anyway, so we don't need
to do it in fixup_irqs(), avoiding a temporary.

Signed-off-by: default avatarRusty Russell <rusty@rustcorp.com.au>
Signed-off-by: default avatarMike Travis <travis@sgi.com>
parent b78936e1

arch/x86/include/asm/irq.h

+1 −1
Original line number Diff line number Diff line
@@ -37,7 +37,7 @@ extern int irqbalance_disable(char *str); 


#ifdef CONFIG_HOTPLUG_CPU

#include <linux/cpumask.h>

extern void fixup_irqs(cpumask_t map);

extern void fixup_irqs(void);

#endif



extern unsigned int do_IRQ(struct pt_regs *regs);

arch/x86/kernel/irq_32.c

+7 −6
Original line number Diff line number Diff line
@@ -233,27 +233,28 @@ unsigned int do_IRQ(struct pt_regs *regs) 
#ifdef CONFIG_HOTPLUG_CPU

#include <mach_apic.h>



void fixup_irqs(cpumask_t map)

/* A cpu has been removed from cpu_online_mask.  Reset irq affinities. */

void fixup_irqs(void)

{

	unsigned int irq;

	static int warned;

	struct irq_desc *desc;



	for_each_irq_desc(irq, desc) {

		cpumask_t mask;

		const struct cpumask *affinity;



		if (!desc)

			continue;

		if (irq == 2)

			continue;



		cpus_and(mask, desc->affinity, map);

		if (any_online_cpu(mask) == NR_CPUS) {

		affinity = &desc->affinity;

		if (cpumask_any_and(affinity, cpu_online_mask) >= nr_cpu_ids) {

			printk("Breaking affinity for irq %i\n", irq);

			mask = map;

			affinity = cpu_all_mask;

		}

		if (desc->chip->set_affinity)

			desc->chip->set_affinity(irq, &mask);

			desc->chip->set_affinity(irq, affinity);

		else if (desc->action && !(warned++))

			printk("Cannot set affinity for irq %i\n", irq);

	}

arch/x86/kernel/irq_64.c

+8 −7
Original line number Diff line number Diff line
@@ -83,16 +83,17 @@ asmlinkage unsigned int do_IRQ(struct pt_regs *regs) 
}



#ifdef CONFIG_HOTPLUG_CPU

void fixup_irqs(cpumask_t map)

/* A cpu has been removed from cpu_online_mask.  Reset irq affinities. */

void fixup_irqs(void)

{

	unsigned int irq;

	static int warned;

	struct irq_desc *desc;



	for_each_irq_desc(irq, desc) {

		cpumask_t mask;

		int break_affinity = 0;

		int set_affinity = 1;

		const struct cpumask *affinity;



		if (!desc)

			continue;
@@ -102,23 +103,23 @@ void fixup_irqs(cpumask_t map) 
		/* interrupt's are disabled at this point */

		spin_lock(&desc->lock);



		affinity = &desc->affinity;

		if (!irq_has_action(irq) ||

		    cpus_equal(desc->affinity, map)) {

		    cpumask_equal(affinity, cpu_online_mask)) {

			spin_unlock(&desc->lock);

			continue;

		}



		cpus_and(mask, desc->affinity, map);

		if (cpus_empty(mask)) {

		if (cpumask_any_and(affinity, cpu_online_mask) >= nr_cpu_ids) {

			break_affinity = 1;

			mask = map;

			affinity = cpu_all_mask;

		}



		if (desc->chip->mask)

			desc->chip->mask(irq);



		if (desc->chip->set_affinity)

			desc->chip->set_affinity(irq, &mask);

			desc->chip->set_affinity(irq, affinity);

		else if (!(warned++))

			set_affinity = 0;

arch/x86/kernel/smpboot.c

+1 −1
Original line number Diff line number Diff line
@@ -1346,7 +1346,7 @@ void cpu_disable_common(void) 
	lock_vector_lock();

	remove_cpu_from_maps(cpu);

	unlock_vector_lock();

	fixup_irqs(cpu_online_map);

	fixup_irqs();

}



int native_cpu_disable(void)

CRA Git | Maintained and supported by SUSTech CRA and CCSE