Re: [Xen-devel] [PATCH] Fix bootloader handling when empty string is being output

[Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>]

Michal Novotny writes ("[Xen-devel] [PATCH] Fix bootloader handling when empty 
string is being output"):
> this is the patch to fix empty string as the output value
> of the bootloader string. If there is no output then
> xend is being hung indefinitely unless you press Ctrl + C keys
> to trigger SIGINT. This patch fixes the issue by implementing
> timeout to the select() function for reading from pipe and also
> checking for state of the process - i.e. whether the process is
> running or whether it's dead or zombie (because fork() call does
> leave zombies when the process is done but the parent process is
> still running).

This is a rather strange way to go about collecting the exit status
from a process.  What is pygrub's parent in this situation ?  Isn't it
xend ?  In which case xend has no need to grobbling around in /proc,
it can just call waitpid.

Ian.

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