Re: [PATCH] x86/apic: check FADT settings after enable x2apic

From: Ingo Molnar
Date: Tue Jan 29 2013 - 03:49:25 EST



* Wang, Song-Bo (Stoney) <song-bo.wang@xxxxxx> wrote:

> [...] Due to this mode mismatch, with specific HW
> configurations, there will be intermittent lost interrupts,
> which could result in a hang or data loss.

That's the key piece of information that was missing - which
should be put into the changelog into a very prominent place.

> Logically, there is a mismatch because the kernel missed
> detection on HW configurations, and we should fix it, right?
> Do we need more detailed information?

No, the above sentence is more than enough and the fix looks
nice. Note that this sentence is more useful to users than the
rest of the changelog combined!

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/