diff options
author | Keir Fraser <keir.fraser@citrix.com> | 2009-11-23 06:59:06 +0000 |
---|---|---|
committer | Keir Fraser <keir.fraser@citrix.com> | 2009-11-23 06:59:06 +0000 |
commit | c14ce8a1257fbbc43c4196b186a7677209c6b12e (patch) | |
tree | beb3772633724bc4443868c3abf2f2722358a5f4 /tools/libxl/libxl_exec.c | |
parent | 3cfaf63e09773f7046e35b9287a94323840fcacf (diff) | |
download | xen-c14ce8a1257fbbc43c4196b186a7677209c6b12e.tar.gz xen-c14ce8a1257fbbc43c4196b186a7677209c6b12e.tar.bz2 xen-c14ce8a1257fbbc43c4196b186a7677209c6b12e.zip |
libxenlight: Clean up logging arrangements
* Introduce new variants of the logging functions which include
errno values (converted using strerror) in the messages passed to
the
application's logging callback.
* Use the new errno-including logging functions everywhere where
appropriate. In general, xc_... functions return errno values or 0;
xs_... functions return 0 or -1 (or some such) setting errno.
* When libxl_xs_get_dompath fails, do not treat it as an allocation
error. It isn't: it usually means xenstored failed.
* Remove many spurious \n's from log messages. (The applications log
callback is expected to add a \n if it wants to do that, so libxl's
logging functions should be passed strings without \n.)
Signed-off-by: Ian Jackson <Ian.Jackson@eu.citrix.com>
Diffstat (limited to 'tools/libxl/libxl_exec.c')
-rw-r--r-- | tools/libxl/libxl_exec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tools/libxl/libxl_exec.c b/tools/libxl/libxl_exec.c index 8d7928b4e8..8a589b676c 100644 --- a/tools/libxl/libxl_exec.c +++ b/tools/libxl/libxl_exec.c @@ -28,7 +28,7 @@ int libxl_exec(struct libxl_ctx *ctx, int stdinfd, int stdoutfd, int stderrfd, pid = fork(); if (pid == -1) { - XL_LOG(ctx, XL_LOG_ERROR, "fork failed"); + XL_LOG_ERRNO(ctx, XL_LOG_ERROR, "fork failed"); return -1; } if (pid == 0) { |