Re: [Xen-devel] [PATCH 4 of 4] libxl: add statup checks to libxl__wait_for_device_model

[Ian Campbell <Ian.Campbell@xxxxxxxxxxxxx>]

On Tue, 2011-05-24 at 16:57 +0100, Ian Jackson wrote:
> Ian Campbell writes ("Re: [Xen-devel] [PATCH 4 of 4] libxl: add statup checks 
> to libxl__wait_for_device_model"):
> > I accidentally removed the close of the read side of the pipes from the
> > intermediate process while refactoring. So this incremental fix is
> > needed:
> 
> This hunk was in your O_CLOEXEC patch so is now in the tree, although
> not part of the changeset for libxl startup checks.

Either way is fine with me.

Thanks,
Ian.

> 
> Ian.
> 
> > --- a/tools/libxl/libxl_exec.c      Wed May 04 17:03:14 2011 +0100
> > +++ b/tools/libxl/libxl_exec.c      Wed May 04 17:22:05 2011 +0100
> > @@ -144,6 +144,7 @@ int libxl__spawn_spawn(libxl__gc *gc,
> >      }
> >  
> >      /* we are now the intermediate process */
> > +    if (for_spawn) close(pipes[0]);
> >  
> >      child = fork();
> >      if (child == -1)



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel