[PATCH] x86, fix x86 fixup_irqs() error handling [v3]

2014-04-02 Thread Prarit Bhargava
Resending ... didn't see this in tree yet and want to make sure it doesn't get dropped. Last submit here: http://marc.info/?l=linux-kernel=139453958911067=2 P. ---8<--- Several patches to fix cpu hotplug and the down'd cpu's irq relocations have been submitted in the past month or so. The

[PATCH] x86, fix x86 fixup_irqs() error handling [v3]

2014-04-02 Thread Prarit Bhargava
Resending ... didn't see this in tree yet and want to make sure it doesn't get dropped. Last submit here: http://marc.info/?l=linux-kernelm=139453958911067w=2 P. ---8--- Several patches to fix cpu hotplug and the down'd cpu's irq relocations have been submitted in the past month or so. The

[PATCH] x86, fix x86 fixup_irqs() error handling [v3]

2014-03-11 Thread Prarit Bhargava
Several patches to fix cpu hotplug and the down'd cpu's irq relocations have been submitted in the past month or so. The patches should resolve the problems with cpu hotplug and irq relocation, however, there is always a possibility that a bug still exists. The big problem with debugging these

Re: [PATCH] x86, fix x86 fixup_irqs() error handling [v2]

2014-03-11 Thread Ingo Molnar
* Prarit Bhargava wrote: > fixup_irqs() calls chip->set_irq_affinity which eventually calls > __assign_irq_vector(). Errors are not propogated back from this function call > and this results in silent irq relocation failures. This patch fixes this > issue and prints out a warning if there is

Re: [PATCH] x86, fix x86 fixup_irqs() error handling [v2]

2014-03-11 Thread Ingo Molnar
* Prarit Bhargava pra...@redhat.com wrote: fixup_irqs() calls chip-set_irq_affinity which eventually calls __assign_irq_vector(). Errors are not propogated back from this function call and this results in silent irq relocation failures. This patch fixes this issue and prints out a warning

[PATCH] x86, fix x86 fixup_irqs() error handling [v3]

2014-03-11 Thread Prarit Bhargava
Several patches to fix cpu hotplug and the down'd cpu's irq relocations have been submitted in the past month or so. The patches should resolve the problems with cpu hotplug and irq relocation, however, there is always a possibility that a bug still exists. The big problem with debugging these

[PATCH] x86, fix x86 fixup_irqs() error handling [v2]

2014-03-06 Thread Prarit Bhargava
fixup_irqs() calls chip->set_irq_affinity which eventually calls __assign_irq_vector(). Errors are not propogated back from this function call and this results in silent irq relocation failures. This patch fixes this issue and prints out a warning if there is a relocation failure. [v2]:

Re: [PATCH] x86, fix x86 fixup_irqs() error handling

2014-03-06 Thread Thomas Gleixner
On Wed, 5 Mar 2014, David Rientjes wrote: > On Wed, 5 Mar 2014, Prarit Bhargava wrote: > > > diff --git a/arch/x86/kernel/irq.c b/arch/x86/kernel/irq.c > > index d99f31d..55fab61 100644 > > --- a/arch/x86/kernel/irq.c > > +++ b/arch/x86/kernel/irq.c > > @@ -351,6 +351,7 @@ void fixup_irqs(void)

Re: [PATCH] x86, fix x86 fixup_irqs() error handling

2014-03-06 Thread Thomas Gleixner
On Wed, 5 Mar 2014, David Rientjes wrote: On Wed, 5 Mar 2014, Prarit Bhargava wrote: diff --git a/arch/x86/kernel/irq.c b/arch/x86/kernel/irq.c index d99f31d..55fab61 100644 --- a/arch/x86/kernel/irq.c +++ b/arch/x86/kernel/irq.c @@ -351,6 +351,7 @@ void fixup_irqs(void)

[PATCH] x86, fix x86 fixup_irqs() error handling [v2]

2014-03-06 Thread Prarit Bhargava
fixup_irqs() calls chip-set_irq_affinity which eventually calls __assign_irq_vector(). Errors are not propogated back from this function call and this results in silent irq relocation failures. This patch fixes this issue and prints out a warning if there is a relocation failure. [v2]: modified

Re: [PATCH] x86, fix x86 fixup_irqs() error handling

2014-03-05 Thread Prarit Bhargava
On 03/05/2014 04:09 PM, David Rientjes wrote: > On Wed, 5 Mar 2014, Prarit Bhargava wrote: > >> diff --git a/arch/x86/kernel/irq.c b/arch/x86/kernel/irq.c >> index d99f31d..55fab61 100644 >> --- a/arch/x86/kernel/irq.c >> +++ b/arch/x86/kernel/irq.c >> @@ -351,6 +351,7 @@ void fixup_irqs(void)

Re: [PATCH] x86, fix x86 fixup_irqs() error handling

2014-03-05 Thread David Rientjes
On Wed, 5 Mar 2014, Prarit Bhargava wrote: > diff --git a/arch/x86/kernel/irq.c b/arch/x86/kernel/irq.c > index d99f31d..55fab61 100644 > --- a/arch/x86/kernel/irq.c > +++ b/arch/x86/kernel/irq.c > @@ -351,6 +351,7 @@ void fixup_irqs(void) > struct irq_desc *desc; > struct irq_data

[PATCH] x86, fix x86 fixup_irqs() error handling

2014-03-05 Thread Prarit Bhargava
fixup_irqs() calls chip->set_irq_affinity which eventually calls __assign_irq_vector(). Errors are not propogated back from this function call and this results in silent irq relocation failures. This patch fixes this issue and prints out a warning if there is a relocation failure. Cc: Thomas

[PATCH] x86, fix x86 fixup_irqs() error handling

2014-03-05 Thread Prarit Bhargava
fixup_irqs() calls chip-set_irq_affinity which eventually calls __assign_irq_vector(). Errors are not propogated back from this function call and this results in silent irq relocation failures. This patch fixes this issue and prints out a warning if there is a relocation failure. Cc: Thomas

Re: [PATCH] x86, fix x86 fixup_irqs() error handling

2014-03-05 Thread David Rientjes
On Wed, 5 Mar 2014, Prarit Bhargava wrote: diff --git a/arch/x86/kernel/irq.c b/arch/x86/kernel/irq.c index d99f31d..55fab61 100644 --- a/arch/x86/kernel/irq.c +++ b/arch/x86/kernel/irq.c @@ -351,6 +351,7 @@ void fixup_irqs(void) struct irq_desc *desc; struct irq_data *data;

Re: [PATCH] x86, fix x86 fixup_irqs() error handling

2014-03-05 Thread Prarit Bhargava
On 03/05/2014 04:09 PM, David Rientjes wrote: On Wed, 5 Mar 2014, Prarit Bhargava wrote: diff --git a/arch/x86/kernel/irq.c b/arch/x86/kernel/irq.c index d99f31d..55fab61 100644 --- a/arch/x86/kernel/irq.c +++ b/arch/x86/kernel/irq.c @@ -351,6 +351,7 @@ void fixup_irqs(void) struct