lkml.org 
[lkml]   [2011]   [Jun]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 4/4] ACPI, APEI, Add APEI _OSC support
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
APEI _OSC.
- both failed: Failed to enable APEI firmware first mode!

Best Regards,
Huang Ying


\
 
 \ /
  Last update: 2011-06-22 04:23    [W:0.057 / U:0.256 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site