Re: [PATCH 1/2] add helper for highmem checks

H. Peter Anvin
2013-02-11T18:11:31Z

On 02/11/2013 09:32 AM, Dave Hansen wrote:
> On 02/09/2013 02:47 AM, Borislav Petkov wrote:
>> On Sat, Feb 09, 2013 at 10:41:21AM +0100, Borislav Petkov wrote:
>> With this change, they definitely fix something because I even get X on
>> the box started. Previously, it would spit out the warning and wouldn't
>> start X with the login window. And my suspicion is that wdm (WINGs
>> display manager) I'm using, does /dev/mem accesses when it starts and it
>> obviously failed. Now not so much :-)
>
> That's crazy. Didn't expect that at all.
>
> I guess X is happier getting an error than getting random pages back.
> I'm working on a set of patches now that should get it _working_ instead
> of just returning an error.
>

Awesome :)

-hpa

--
H. Peter Anvin, Intel Open Source Technology Center
I work for Intel. I don't speak on their behalf.