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

From: Ingo Molnar
Date: Tue Mar 11 2014 - 07:12:11 EST



* Prarit Bhargava <prarit@xxxxxxxxxx> 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 a relocation failure.
>
> [v2]: modified WARN to pr_crit and changed message

Please fix the changelog to conform to the standard changelog style:

- first describe the symptoms of the bug - how does a user notice?

- then describe how the code behaves today and how that is causing the bug

- and then only describe how it's fixed.

The first item is the most important one - yet it's missing from this
changelog.

Thanks,

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