lkml.org 
[lkml]   [2017]   [Dec]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCHv4 3/4] x86/boot/compressed/64: Introduce place_trampoline()

* Kirill A. Shutemov <kirill@shutemov.name> wrote:

> On Thu, Dec 07, 2017 at 07:30:48AM +0100, Ingo Molnar wrote:
> > > We also need a small stack in the trampoline to re-enable long mode via
> > > long return. But stack and code can share the page just fine.
> >
> > BTW., I'm not sure this is necessarily a good idea: it means writable+executable
> > memory, which we generally try to avoid. How complicated would it be to have them
> > separate?
>
> It's trivial: you only need to bump TRAMPOLINE_32BIT_SIZE.
>
> But it doesn't make much sense. We're running from indentity mapping: all
> memory is r/w without NX bit set (and IA32_EFER.NXE is 0).

Ok, fair enough!

Thanks,

Ingo

\
 
 \ /
  Last update: 2017-12-08 12:29    [W:2.144 / U:0.548 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site