Re: [Xen-devel] [xen-unstable bisection] complete test-amd64-amd64-xl-win

[Ian Campbell <Ian.Campbell@xxxxxxxxxx>]

On Mon, 2012-07-02 at 10:19 +0100, Roger Pau Monnà wrote:
> On Mon, Jul 2, 2012 at 6:45 AM, xen.org <ian.jackson@xxxxxxxxxxxxx> wrote:
> > branch xen-unstable
> > xen branch xen-unstable
> > job test-amd64-amd64-xl-win
> > test windows-install
> >
> > Tree: linux git://xenbits.xen.org/linux-pvops.git
> > Tree: qemu git://xenbits.xen.org/staging/qemu-xen-unstable.git
> > Tree: qemuu git://xenbits.xen.org/staging/qemu-upstream-unstable.git
> > Tree: xen http://xenbits.xen.org/staging/xen-unstable.hg
> >
> > *** Found and reproduced problem changeset ***
> >
> >   Bug is in tree:  xen http://xenbits.xen.org/staging/xen-unstable.hg
> >   Bug introduced:  dbf54d93ac40
> >   Bug not present: d70030d6a87c
> >
> >
> >   changeset:   25563:dbf54d93ac40
> >   user:        Ian Campbell <ian.campbell@xxxxxxxxxx>
> >   date:        Fri Jun 29 08:58:22 2012 +0100
> >
> >       xl: initialise domid to an explicitly invalid value
> >
> >       also ensure it is invalid whenever we destroy the domain.
> >
> >       Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>
> >       Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
> >       Committed-by: Ian Jackson <ian.jackson.citrix.com>
> 
> I haven't tested it, but I think we are missing a "domid =
> INVALID_DOMID;" in the "LIBXL_ACTION_ON_SHUTDOWN_RESTART_RENAME" case.

That sounds plausible, thanks.

I'll test and submit a patch.

Ian.



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