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

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



On Mon, 2014-11-03 at 15:42 +0000, Ian Campbell wrote:
> 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!

FYI the next Xen osstest run didn't suffer from this issue, thanks.

(in case you go looking at the results, it did fail for other Xen
related reasons, nothing to do with seabios, but this issue is
definitely gone.)

> 
> > -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
> 
> 
> 
> _______________________________________________
> SeaBIOS mailing list
> SeaBIOS@xxxxxxxxxxx
> http://www.seabios.org/mailman/listinfo/seabios
> 



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


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.