lkml.org 
[lkml]   [2013]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [regression, bisected] x86: efi: Pass boot services variable info to runtime code
On Sat, Jun 01, 2013 at 08:01:59PM +0900, Linus Torvalds wrote:
> On Fri, 31 May 2013, Matthew Garrett wrote:
> >
> > 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
>
> So "differently break" doesn't matter, if it's old breakage, and people
> thus don't really expect it to work. We need to fix bugs without *new*
> breakage, and quite frankly, I have been distressed by hearing the EFI
> "specifications" mentioned so many times in this thread.

The old breakage has only ever been present in -rcs, not in released
kernels. We worked around the worst of it with the patchset that's
causing the problems. We can avoid that old breakage by reverting the
patches that introduced it, but that gets us back to bricking some
machines.

> Anyway, I'm traveling with absolutely horrendous internet access, so can
> somebody please send a description of the revert with the relevant
> information, because I literally have a hard time extracting it all from
> this thread because my email access is so slow and flaky... Make it easy
> for me to do the revert with a good explanation message, please,

I've finally got some information from Samsung that *should* let us fix
this without anything else breaking in the process, but since we're
dealing with firmware there's obviously a risk that that won't actually
be true. I'll send that later today.

--
Matthew Garrett | mjg59@srcf.ucam.org


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