From: Ingo Molnar <mingo@kernel.org>
To: Alex Thorlton <athorlton@sgi.com>
Cc: Borislav Petkov <bp@alien8.de>,
linux-kernel@vger.kernel.org,
Matt Fleming <matt.fleming@intel.com>,
Thomas Gleixner <tglx@linutronix.de>,
Ingo Molnar <mingo@redhat.com>, "H. Peter Anvin" <hpa@zytor.com>,
Hedi Berriche <hedi@sgi.com>, Dimitri Sivanich <sivanich@sgi.com>,
x86@kernel.org, linux-efi@vger.kernel.org
Subject: Re: [PATCH 1/2] Remove EFI memmap quirk for UV
Date: Wed, 18 Nov 2015 09:00:47 +0100 [thread overview]
Message-ID: <20151118080047.GA22813@gmail.com> (raw)
In-Reply-To: <20151118015630.GC161386@asylum.americas.sgi.com>
* Alex Thorlton <athorlton@sgi.com> wrote:
> On Tue, Nov 17, 2015 at 08:32:59PM +0100, Borislav Petkov wrote:
> > On Mon, Nov 16, 2015 at 11:59:40AM -0600, Alex Thorlton wrote:
> > > Commit a5d90c923bcf ("x86/efi: Quirk out SGI UV") added a quirk to
> > > efi_apply_memmap_quirks to force SGI UV systems to fall back to the old
> > > EFI memmap mechanism. We have a BIOS fix for this issue now, so we no
> > > longer need this quirk in the kernel.
> >
> > So does that mean that all UV systems out there will get the BIOS update
> > or at least will have a BIOS update available so that we won't need the
> > quirk at all?
>
> Essentially, yes. The BIOS fix isn't available on UV1, but that's older
> hardware at this point, and the distro kernels that run on it will
> still have the quirk. This change is mainly so that we can get the
> un-quirked code pulled in for the next major distro release.
So how about older UV systems that don't want to upgrade the BIOS for whatever
reason but wants to upgrade to a new distro will have a non-booting kernel?
Also, what about UV systems that compile their own kernels for whatever reason?
> We won't be checking this down into the stable branches. Mainly just planning
> ahead here for future UV and distro releases.
But eventually the new kernel will become the stable kernel, and we'll be breaking
old systems with old BIOSen, which is not good.
We should at least check the BIOS version via a DMI quirk and panic in some nicely
informative 'upgrade your BIOS!' way to ease the transition ...
Thanks,
Ingo
next prev parent reply other threads:[~2015-11-18 8:00 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-11-16 17:59 [PATCH 1/2] Remove EFI memmap quirk for UV Alex Thorlton
[not found] ` <1447696781-159611-1-git-send-email-athorlton-sJ/iWh9BUns@public.gmane.org>
2015-11-16 17:59 ` [PATCH 2/2] Remove extra mapping code for UV MMRs Alex Thorlton
2015-11-16 18:02 ` [PATCH 1/2] Remove EFI memmap quirk for UV Thomas Gleixner
2015-11-17 9:52 ` Matt Fleming
2015-11-17 18:24 ` Alex Thorlton
2015-11-17 19:32 ` Borislav Petkov
[not found] ` <20151117193258.GB9374-fF5Pk5pvG8Y@public.gmane.org>
2015-11-18 1:56 ` Alex Thorlton
2015-11-18 8:00 ` Ingo Molnar [this message]
[not found] ` <20151118080047.GA22813-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-11-18 9:23 ` Borislav Petkov
[not found] ` <20151118092315.GB4138-fF5Pk5pvG8Y@public.gmane.org>
2015-11-18 17:00 ` Alex Thorlton
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20151118080047.GA22813@gmail.com \
--to=mingo@kernel.org \
--cc=athorlton@sgi.com \
--cc=bp@alien8.de \
--cc=hedi@sgi.com \
--cc=hpa@zytor.com \
--cc=linux-efi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=matt.fleming@intel.com \
--cc=mingo@redhat.com \
--cc=sivanich@sgi.com \
--cc=tglx@linutronix.de \
--cc=x86@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).