Re: [PATCH] x86: Fix PAGE_OFFSET for kernels since 4.20

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 09, 2019 at 10:33:11AM +0530, Bhupesh Sharma wrote:
> Sorry Guys,
> 
> On Tue, Sep 3, 2019 at 11:36 PM Donald Buczek <buczek@xxxxxxxxxxxxx> wrote:
> >
> > Dear Simon,
> >
> > On 03.09.19 16:41, Simon Horman wrote:
> > > On Fri, Aug 30, 2019 at 11:12:58AM +0200, Donald Buczek wrote:
> > >> Linux kernel commit d52888aa2753 ("x86/mm: Move LDT remap out of KASLR
> > >> region on 5-level paging") changed the base of the direct mapping
> > >> from 0xffff880000000000 to 0xffff888000000000. This was merged
> > >> into v4.20-rc2.
> > >>
> > >> Update to new address accordingly.
> > >
> > > Thanks, applied for inclusion in v2.0.20.
> >
> > I must admit, that I overlooked that d52888aa2753 ("x86/mm: Move LDT remap out of KASLR region on 5-level paging") was backported to 4.19.5 as 4074ca7d8a18 ("x86/mm: Move LDT remap out of KASLR region on 5-level paging").
> >
> > Bhupesh Sharma wanted to look into a a dynamic solution, which, of course, would be better. But if you still want to include this (e.g. as the fallback for the dynamic solution), you might want to replace the patch with the following one, which selects the new PAGE_OFFSET starting from kernel version 4.19.5 instead of 4.20.0.
> 
> I was in a conference last week and was not able to connect to my test
> machine(s) to develop and test a more generic fix, as I suggested in
> my earlier message.
> 
> I will try to work on and send a generic fix in a couple of days.

Thanks,

I think it would be best if this patch,
23b67f048dde ("x86: Fix PAGE_OFFSET for kernels since 4.20"),
was reverted.

Donald, Bhupesh, what do you think?

_______________________________________________
kexec mailing list
kexec@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/kexec



[Index of Archives]     [LM Sensors]     [Linux Sound]     [ALSA Users]     [ALSA Devel]     [Linux Audio Users]     [Linux Media]     [Kernel]     [Gimp]     [Yosemite News]     [Linux Media]

  Powered by Linux