Re: [regression, bisected] x86: efi: Pass boot services variableinfo to runtime code
From: Matthew Garrett
Date: Sat Jun 01 2013 - 10:40:46 EST
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@xxxxxxxxxxxxx
--
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/