Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1270435 > unrolled thread

[PATCH 1/2] Remove EFI memmap quirk for UV

Back to article view | Back to linux.kernel

Contents

  [PATCH 1/2] Remove EFI memmap quirk for UV Alex Thorlton <athorlton@sgi.com> - 2015-11-16 19:10 +0100
    Re: [PATCH 1/2] Remove EFI memmap quirk for UV Thomas Gleixner <tglx@linutronix.de> - 2015-11-16 19:10 +0100
      Re: [PATCH 1/2] Remove EFI memmap quirk for UV Matt Fleming <matt@codeblueprint.co.uk> - 2015-11-17 11:00 +0100
        Re: [PATCH 1/2] Remove EFI memmap quirk for UV Alex Thorlton <athorlton@sgi.com> - 2015-11-17 19:30 +0100
    Re: [PATCH 1/2] Remove EFI memmap quirk for UV Borislav Petkov <bp@alien8.de> - 2015-11-17 20:40 +0100
      Re: [PATCH 1/2] Remove EFI memmap quirk for UV Alex Thorlton <athorlton@sgi.com> - 2015-11-18 03:00 +0100
        Re: [PATCH 1/2] Remove EFI memmap quirk for UV Ingo Molnar <mingo@kernel.org> - 2015-11-18 09:10 +0100
          Re: [PATCH 1/2] Remove EFI memmap quirk for UV Borislav Petkov <bp@alien8.de> - 2015-11-18 10:30 +0100
            Re: [PATCH 1/2] Remove EFI memmap quirk for UV Alex Thorlton <athorlton@sgi.com> - 2015-11-18 18:10 +0100

#1270435 — [PATCH 1/2] Remove EFI memmap quirk for UV

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.  This commit removes the quirk
from the function in question.

Signed-off-by: Alex Thorlton <athorlton@sgi.com>
Acked-by: Mike Travis <travis@sgi.com>
Acked-by: Russ Anderson <rja@sgi.com>
Cc: Matt Fleming <matt.fleming@intel.com>
Cc: Thomas Gleixner <tglx@linutronix.de>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: "H. Peter Anvin" <hpa@zytor.com>
Cc: Hedi Berriche <hedi@sgi.com>
Cc: Dimitri Sivanich <sivanich@sgi.com>
Cc: x86@kernel.org
Cc: linux-efi@vger.kernel.org

---
 arch/x86/platform/efi/quirks.c | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/arch/x86/platform/efi/quirks.c b/arch/x86/platform/efi/quirks.c
index 1c7380d..96b417c 100644
--- a/arch/x86/platform/efi/quirks.c
+++ b/arch/x86/platform/efi/quirks.c
@@ -259,12 +259,6 @@ void __init efi_apply_memmap_quirks(void)
 		pr_info("efi: Setup done, disabling due to 32/64-bit mismatch\n");
 		efi_unmap_memmap();
 	}
-
-	/*
-	 * UV doesn't support the new EFI pagetable mapping yet.
-	 */
-	if (is_uv_system())
-		set_bit(EFI_OLD_MEMMAP, &efi.flags);
 }
 
 /*
-- 
1.8.5.6

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [next] | [standalone]

#1270437

On Mon, 16 Nov 2015, Alex Thorlton wrote:

CC'ing Matt under his correct e-mail address.

> 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.  This commit removes the quirk
> from the function in question.
> 
> Signed-off-by: Alex Thorlton <athorlton@sgi.com>
> Acked-by: Mike Travis <travis@sgi.com>
> Acked-by: Russ Anderson <rja@sgi.com>
> Cc: Matt Fleming <matt.fleming@intel.com>
> Cc: Thomas Gleixner <tglx@linutronix.de>
> Cc: Ingo Molnar <mingo@redhat.com>
> Cc: "H. Peter Anvin" <hpa@zytor.com>
> Cc: Hedi Berriche <hedi@sgi.com>
> Cc: Dimitri Sivanich <sivanich@sgi.com>
> Cc: x86@kernel.org
> Cc: linux-efi@vger.kernel.org
> 
> ---
>  arch/x86/platform/efi/quirks.c | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/arch/x86/platform/efi/quirks.c b/arch/x86/platform/efi/quirks.c
> index 1c7380d..96b417c 100644
> --- a/arch/x86/platform/efi/quirks.c
> +++ b/arch/x86/platform/efi/quirks.c
> @@ -259,12 +259,6 @@ void __init efi_apply_memmap_quirks(void)
>  		pr_info("efi: Setup done, disabling due to 32/64-bit mismatch\n");
>  		efi_unmap_memmap();
>  	}
> -
> -	/*
> -	 * UV doesn't support the new EFI pagetable mapping yet.
> -	 */
> -	if (is_uv_system())
> -		set_bit(EFI_OLD_MEMMAP, &efi.flags);
>  }
>  
>  /*
> -- 
> 1.8.5.6
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1271030

On Mon, 16 Nov, at 07:02:48PM, Thomas Gleixner wrote:
> On Mon, 16 Nov 2015, Alex Thorlton wrote:
> 
> CC'ing Matt under his correct e-mail address.
> 
> > 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.  This commit removes the quirk
> > from the function in question.
> > 
> > Signed-off-by: Alex Thorlton <athorlton@sgi.com>
> > Acked-by: Mike Travis <travis@sgi.com>
> > Acked-by: Russ Anderson <rja@sgi.com>
> > Cc: Matt Fleming <matt.fleming@intel.com>
> > Cc: Thomas Gleixner <tglx@linutronix.de>
> > Cc: Ingo Molnar <mingo@redhat.com>
> > Cc: "H. Peter Anvin" <hpa@zytor.com>
> > Cc: Hedi Berriche <hedi@sgi.com>
> > Cc: Dimitri Sivanich <sivanich@sgi.com>
> > Cc: x86@kernel.org
> > Cc: linux-efi@vger.kernel.org
> > 
> > ---
> >  arch/x86/platform/efi/quirks.c | 6 ------
> >  1 file changed, 6 deletions(-)
> > 
> > diff --git a/arch/x86/platform/efi/quirks.c b/arch/x86/platform/efi/quirks.c
> > index 1c7380d..96b417c 100644
> > --- a/arch/x86/platform/efi/quirks.c
> > +++ b/arch/x86/platform/efi/quirks.c
> > @@ -259,12 +259,6 @@ void __init efi_apply_memmap_quirks(void)
> >  		pr_info("efi: Setup done, disabling due to 32/64-bit mismatch\n");
> >  		efi_unmap_memmap();
> >  	}
> > -
> > -	/*
> > -	 * UV doesn't support the new EFI pagetable mapping yet.
> > -	 */
> > -	if (is_uv_system())
> > -		set_bit(EFI_OLD_MEMMAP, &efi.flags);

Awesome! Thanks Alex.

Can I also close https://bugzilla.kernel.org/show_bug.cgi?id=75021 ?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1271521

On Tue, Nov 17, 2015 at 09:52:08AM +0000, Matt Fleming wrote:
> Awesome! Thanks Alex.
> 
> Can I also close https://bugzilla.kernel.org/show_bug.cgi?id=75021 ?

Yep!  Thanks, Matt!

- Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1271607

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?

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1271811

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.

We won't be checking this down into the stable branches.  Mainly just
planning ahead here for future UV and distro releases.

- Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1271906

* 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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1272033

On Wed, Nov 18, 2015 at 09:00:47AM +0100, Ingo Molnar wrote:
> 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 ...

Or since we're touching BIOS anyway, maybe stick a bit somewhere which
says "EFI is fixed/quirk not needed anymore" when set and software can
check that bit instead of checking BIOS versions...

-- 
Regards/Gruss,
    Boris.

ECO tip #101: Trim your mails when you reply.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [next] | [standalone]

#1272393

On Wed, Nov 18, 2015 at 10:23:16AM +0100, Borislav Petkov wrote:
> On Wed, Nov 18, 2015 at 09:00:47AM +0100, Ingo Molnar wrote:
> > 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 ...
> 
> Or since we're touching BIOS anyway, maybe stick a bit somewhere which
> says "EFI is fixed/quirk not needed anymore" when set and software can
> check that bit instead of checking BIOS versions...

After talking through this with Russ, we agree that we should fix this
up to play nice on UV1, without users needing to add the memmap quirk to
the command line.  I'll figure out which check we need to make and spin
another version.

Thanks for looking over this, guys!

- Alex
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web