[Libguestfs] [PATCH] Remove explicit guestfs=10.0.2.4:6666 kernel command line parameter.
Richard W.M. Jones
rjones at redhat.com
Thu Sep 17 14:52:50 UTC 2009
On Thu, Sep 17, 2009 at 04:44:11PM +0200, Jim Meyering wrote:
> All looks good, though this seems like it *is* a functional
> change, since it removes the -h and -p options below.
Ah yes, you're right. Removing these is OK I think because no one
would ever have used them, and they are not used by the appliance init
script.
> Two questions:
>
> First, note the definition of GUESTFWD_ADDR above
> Why the commented-out one below (in src/guestfs.c):
>
> //#define GUESTFWD_ADDR "10.0.2.4"
This is just commented out because it isn't used, and gcc now gives a
warning when we define but don't use a macro.
> Second, I saw this in the context:
>
> > /* Make sure SIGPIPE doesn't kill us. */
> > memset (&sa, 0, sizeof sa);
> > sa.sa_handler = SIG_IGN;
>
> Have you considered alternate ways of ignoring SIGPIPE?
> Ignoringn SIGPIPE across the board can cause problems (albeit subtle)
> in unsuspecting child processes.
I didn't know there were alternate ways to ignore SIGPIPE. Can you
give some examples of how it should be done? (Possibly in a new
thread -- seems a separate issue from the patch at hand).
Rich.
--
Richard Jones, Emerging Technologies, Red Hat http://et.redhat.com/~rjones
Read my programming blog: http://rwmj.wordpress.com
Fedora now supports 75 OCaml packages (the OPEN alternative to F#)
http://cocan.org/getting_started_with_ocaml_on_red_hat_and_fedora
More information about the Libguestfs
mailing list