[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130531021749.GA24287@sgi.com>
Date: Thu, 30 May 2013 21:17:49 -0500
From: Russ Anderson <rja@....com>
To: Jiri Kosina <jkosina@...e.cz>
Cc: Matthew Garrett <matthew.garrett@...ula.com>,
joeyli <jlee@...e.com>, Matt Fleming <matt@...sole-pimps.org>,
"matt.fleming@...el.com" <matt.fleming@...el.com>,
"linux-efi@...r.kernel.org" <linux-efi@...r.kernel.org>,
"x86@...nel.org" <x86@...nel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Ingo Molnar <mingo@...nel.org>,
Thomas Gleixner <tglx@...utronix.de>,
"H. Peter Anvin" <hpa@...ux.intel.com>,
Borislav Petkov <bp@...en8.de>
Subject: Re: [regression, bisected] x86: efi: Pass boot services variable
info to runtime code
On Fri, May 31, 2013 at 12:30:43AM +0200, Jiri Kosina wrote:
> On Thu, 30 May 2013, Russ Anderson wrote:
>
> > > > That's a great idea. This patch moves the QueryVariableInfo()
> > > > call from bootime to runtime, in efi_late_init(). The attached
> > > > patch is consistent with the UEFI spec and avoids the problem.
> > >
> > > No, that defeats the entire point of the original patch.
> >
> > How so? It is still calling QueryVariableInfo()
> > before the data is used.
>
> You lose information provided by QueryVariableInfo() about boot-only
> variables once the transition boot -> runtime has happened.
Is that information really more important than the ability
to boot?
Correct me if I'm wrong, but linux was able to boot without
the boottime QueryVariableInfo() call up until 3.9-rc7,
and it still does on systems that do not use EFI stubs (ie
grub and elilo). It is only when linux uses EFI stubs (ie
grub2) that linux makes the boottime QueryVariableInfo()
call. So why is that call, or whatever is dependent on it,
more important than booting?
Thanks,
--
Russ Anderson, OS RAS/Partitioning Project Lead
SGI - Silicon Graphics Inc rja@....com
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists