My preference would to use static const over preprocessor definitions
but I also don't mind conformance if that is why you are suggesting
this change.
Signed-off-by: Kamala Narasimhan <kamala.narasimhan@xxxxxxxxxx>
Kamala
diff -r fe8a177ae9cb tools/libxl/libxl.c
--- a/tools/libxl/libxl.c Wed Jan 19 15:29:04 2011 +0000
+++ b/tools/libxl/libxl.c Thu Jan 20 08:58:06 2011 -0500
@@ -40,11 +40,20 @@
int libxl_ctx_init(libxl_ctx *ctx, int version, xentoollog_logger *lg)
{
+ struct stat stat_buf;
+
if (version != LIBXL_VERSION)
return ERROR_VERSION;
memset(ctx, 0, sizeof(libxl_ctx));
ctx->lg = lg;
memset(&ctx->version_info, 0, sizeof(libxl_version_info));
+
+ if ( stat(XENSTORE_PID_FILE, &stat_buf) != 0 ) {
+ LIBXL__LOG(ctx, LIBXL__LOG_ERROR,
+ "Is xenstore daemon running?\nStat on file %s returned -
\"%s\".\n",
+ XENSTORE_PID_FILE, strerror(errno));
+ return ERROR_FAIL;
+ }
ctx->xch = xc_interface_open(lg,lg,0);
if (!ctx->xch) {
diff -r fe8a177ae9cb tools/libxl/libxl_internal.h
--- a/tools/libxl/libxl_internal.h Wed Jan 19 15:29:04 2011 +0000
+++ b/tools/libxl/libxl_internal.h Thu Jan 20 08:58:06 2011 -0500
@@ -104,6 +104,7 @@ typedef struct {
#define AUTO_PHP_SLOT 0x100
#define SYSFS_PCI_DEV "/sys/bus/pci/devices"
#define SYSFS_PCIBACK_DRIVER "/sys/bus/pci/drivers/pciback"
+#define XENSTORE_PID_FILE "/var/run/xenstore.pid"
#define PROC_PCI_NUM_RESOURCES 7
#define PCI_BAR_IO 0x01
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|