[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 2/2] xenstore: Use <xenstore.h>



On 15/06/12 12:53, Juan Quintela wrote:
Anthony PERARD<anthony.perard@xxxxxxxxxx>  wrote:
In the next release of Xen (4.2), xs.h became deprecated.

Signed-off-by: Anthony PERARD<anthony.perard@xxxxxxxxxx>
---
  configure       |    2 +-
  hw/xen_common.h |    6 +++++-
  2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/configure b/configure
index c2366ee..e7f66c9 100755
--- a/configure
+++ b/configure
@@ -1382,7 +1382,7 @@ EOF
    elif (
        cat>  $TMPC<<EOF
  #include<xenctrl.h>
-#include<xs.h>
+#include<xenstore.h>
  #include<stdint.h>
  #include<xen/hvm/hvm_info_table.h>
  #if !defined(HVM_MAX_VCPUS)
diff --git a/hw/xen_common.h b/hw/xen_common.h
index fe7f227..cc99204 100644
--- a/hw/xen_common.h
+++ b/hw/xen_common.h
@@ -7,7 +7,11 @@
  #include<inttypes.h>

  #include<xenctrl.h>
-#include<xs.h>
+#if CONFIG_XEN_CTRL_INTERFACE_VERSION<  420
+#  include<xs.h>
+#else
+#  include<xenstore.h>
+#endif
  #include<xen/io/xenbus.h>

  #include "hw.h"

Shouldn't we need the ifdef also in configure?  On my system xenstore.h
still don't exist.

No, configure does not need it. In the configure, I just change the header in the test for the next version of Xen. Also the define is defined by configure.

Regards,

--
Anthony PERARD

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.