On 15/06/12 12:53, Juan Quintela wrote:> Anthony PERARD<anthony.perard@citrix.com> wrote:
>> In the next release of Xen (4.2), xs.h became deprecated.
>>
>> Signed-off-by: Anthony PERARD<anthony.perard@citrix.com>
>> ---
>> 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