[SeaBIOS] [Xen-devel] Regression booting winxp under Xen.

Ian Campbell Ian.Campbell at citrix.com
Mon Nov 3 16:42:57 CET 2014


On Mon, 2014-11-03 at 09:59 -0500, Kevin O'Connor wrote:
> On Mon, Nov 03, 2014 at 10:05:05AM +0000, Ian Campbell wrote:
> > On Mon, 2014-11-03 at 09:59 +0000, Ian Campbell wrote:
> > 
> > > I've not investigated more thoroughly yes, just posting in case
> > > something obvious leaps out at someone. The automated test is currently
> > > bisecting the issue, once it is done I'll let you know the result.
> > 
> > If I'd read a bit further through my Monday morning INBOX I'd have
> > found:
> >         http://lists.xen.org/archives/html/xen-devel/2014-11/msg00001.html
> >         
> > which indicates that the bisector has fingered:
> > 
> >   commit 99cb8f3e9af516954b2f2fba97ce1856e3d7b93f
> 
> Sorry about that - I missed one of the stack offset conversions in the
> PNP part of that change.  The fix is below and I just pushed it to
> the main repo.
> 
> Thanks for catching it.

Thanks for the quick fix!

> -Kevin
> 
> 
> --- a/src/romlayout.S
> +++ b/src/romlayout.S
> @@ -286,7 +286,7 @@ entry_pnp_real:
>          movw %cx, %ds
>          leal BREGS_size-6+12(%esp), %eax  // %eax points to start of u16 args
>          calll handle_pnp
> -        movw %ax, 12(%esp)      // Modify %eax to return %ax
> +        movw %ax, BREGS_eax(%esp)   // Modify %eax to return %ax
>          POPBREGS
>          popfl
>          popl %esp





More information about the SeaBIOS mailing list