lkml.org 
[lkml]   [2013]   [May]   [31]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [regression, bisected] x86: efi: Pass boot services variable info to runtime code
From
Date
On Fri, 2013-05-31 at 15:34 +0100, Matthew Garrett wrote:
> On Fri, May 31, 2013 at 02:43:56PM +0200, Ingo Molnar wrote:
>
> > 4) The revert is easy, and the functionality the original patch provided
> > was a marginal increase in debug output to begin with...
>
> I agree that a revert is probably the right thing to do here, but the
> original patch was there to permit a more accurate calculation of the
> amount of nvram in use, not to provide additional debug information.
> Reverting it is going to differently break a different set of systems

The only ones that are broken are the Samsung ones. Samsung claims to
have fixed their UEFI firmware, so we could refer any problems to them.

The signature of the Samsung failure, which this is guarding against is
that the laptop gets bricked, so it really is a nasty choice of poisons
we have to pick...

Could we hedge the QueryVariableInfo checks with a test for Samsung in
the UEFI identity strings?

James





\
 
 \ /
  Last update: 2013-05-31 17:01    [W:0.978 / U:0.052 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site