Re: [Xen-devel] [PATCH v11 11/13] x86: make Xen early boot code relocatable

[Daniel Kiper <daniel.kiper@xxxxxxxxxx>]

On Mon, Jan 09, 2017 at 08:05:01PM -0600, Doug Goldstein wrote:
> On 12/5/16 4:25 PM, Daniel Kiper wrote:

[...]

> >          /* Save trampoline address for later use. */
> >          shl     $4, %ecx
> > -        mov     %ecx,sym_phys(trampoline_phys)
> > +        mov     %ecx,sym_fs(trampoline_phys)
> > +
> > +        /* Save Xen image load base address for later use. */
> > +        mov     %esi,sym_fs(xen_phys_start)
> > +        mov     %esi,sym_fs(trampoline_xen_phys_start)
> > +
> > +        /* Setup stack. %ss was initialized earlier. */
> > +        lea     1024+sym_esi(cpu0_stack),%esp
>
> Not sure if this is the bad code or what but after this patch in the
> series the stack is at 0xAD000 which is in the middle of VGA memory so
> the stack is getting screwy (hence my email last week). And the stack is
> bad in trampoline_setup, before trampoline_setup the stack seems fine.
>
> This is from my hacked up version of iPXE...
>
> Type / Addr / Number of Pages / Flags
>
> Conventional: 0x000059000 :      69 : f
> Reserved:     0x00009e000 :       2 : f
> Conventional: 0x000100000 :   65280 : f
>
> As you can see from 0xA0000 to 0x100000 isn't mentioned but everything I
> can find on EFI says if the range isn't mentioned you shouldn't use it.

Could you provide full EFI memory map from this machine?

Daniel

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel