Re: [PATCH 4/4] ACPI, APEI, Add APEI _OSC support

From: Huang Ying
Date: Tue Jun 21 2011 - 22:21:30 EST

On 06/21/2011 09:22 PM, Matthew Garrett wrote:
> On Tue, Jun 21, 2011 at 03:16:27PM +0800, Huang Ying wrote:
>> + rc = apei_osc_setup();
>> + if (rc)
>> + pr_info(GHES_PFX "Evaluate APEI _OSC failed!\n");
> Hm. This is maybe a little strong. It'd be valid for a machine to return
> an error here but still have the GHES functionality enabled via the
> generic call, but this message would still show up and potentially
> confuse the user. Can we keep a flag to check whether the generic method
> gave us control, and only give the error if both fail to enable it?

At least on some of my testing machine, generic _OSC call will not
return any error even it does not support APEI bit. So I think
sometimes it may be helpful to printk something here. To avoid
confusion, can we change the message as follow.

- generic _OSC succeeded, APEI _OSC failed: APEI firmware first mode is
enabled by APEI bit.
- generic _OSC failed, APEI _OSC succeeded: APEI firmware first mode is
enabled by APEI _OSC.
- both succeeded: APEI firmware first mode is enabled by APEI bit and
- both failed: Failed to enable APEI firmware first mode!

Best Regards,
Huang Ying
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at