[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20121105103753.GA25850@kroah.com>
Date: Mon, 5 Nov 2012 11:37:53 +0100
From: Greg KH <gregkh@...uxfoundation.org>
To: Matt Fleming <matt@...sole-pimps.org>
Cc: Corentin Chary <corentincj@...aif.net>,
Matthew Garrett <mjg@...hat.com>, linux-kernel@...r.kernel.org,
linux-efi@...r.kernel.org, platform-driver-x86@...r.kernel.org,
Alessandro Crismani <alessandro.crismani@...il.com>,
Mikhail Bakhterev <mike.bakhterev@...il.com>,
Patrick H <kernel@...storm.net>,
Matt Fleming <matt.fleming@...el.com>,
"H. Peter Anvin" <hpa@...or.com>, stable@...r.kernel.org
Subject: Re: [PATCH] samsung-laptop: Disable if CONFIG_EFI=y
On Sun, Nov 04, 2012 at 05:35:06PM +0000, Matt Fleming wrote:
> From: Matt Fleming <matt.fleming@...el.com>
>
> We've started getting reports of users seeing Machine Check Exceptions
> when booting their Samsung laptops in UEFI mode,
>
> https://bugzilla.kernel.org/show_bug.cgi?id=47121
>
> This module seems to be the culprit as it's grovelling around in the
> 0xf0000 region which has no mapping in either the e820 or EFI memory
> maps on the affected machines.
So, does this mean that if we try to ioremap a memory location in the
kernel, like this driver is, it will not fail, but, when accessing the
memory, bad things happen?
That's not good, shouldn't the call to ioremap_nocache() have failed
originally? That sounds like a core EFI/platform bug here, and one that
you might run into other places.
Shouldn't fixing that be the real fix?
thanks,
greg k-h
--
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