diff options
author | Konrad Rzeszutek Wilk <konrad.wilk@oracle.com> | 2013-05-13 15:29:12 -0400 |
---|---|---|
committer | Ian Campbell <ian.campbell@citrix.com> | 2013-05-14 10:02:00 +0100 |
commit | 93b241e9d2c23b75f400a9ccda40838b9d13e27d (patch) | |
tree | 2dbefc5b94d337822d69830eb7da3c8c95a31a47 /tools/libxl/xl_cmdimpl.c | |
parent | bec8f17e48439ee5b8370f4e431ccd9a9514bee7 (diff) | |
download | xen-93b241e9d2c23b75f400a9ccda40838b9d13e27d.tar.gz xen-93b241e9d2c23b75f400a9ccda40838b9d13e27d.tar.bz2 xen-93b241e9d2c23b75f400a9ccda40838b9d13e27d.zip |
libxl: Change claim_mode from bool to int.
During the review it was noticed that it would be better if internally
the claim_mode was held as an 'int' instead of a 'bool'. The reason
is that during the startup of xl, one has call the libxl_defbool_setdefault.
otherwise any usage of claim_mode would result in assert break.
The assert is due to the fact that using defbool without any set
values (either true of false) will cause it hit an assertion.
If we use an 'int' we don't have to worry about it and by default
the value of zero will suffice for checks whether the claim is
enabled or disabled.
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Acked-by: Ian Campbell <Ian.Campbell@citrix.com>
Diffstat (limited to 'tools/libxl/xl_cmdimpl.c')
-rw-r--r-- | tools/libxl/xl_cmdimpl.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c index edf0325fdd..609ce49a7d 100644 --- a/tools/libxl/xl_cmdimpl.c +++ b/tools/libxl/xl_cmdimpl.c @@ -734,7 +734,7 @@ static void parse_config_data(const char *config_source, if (!xlu_cfg_get_long (config, "maxmem", &l, 0)) b_info->max_memkb = l * 1024; - b_info->claim_mode = claim_mode; + libxl_defbool_set(&b_info->claim_mode, claim_mode); if (xlu_cfg_get_string (config, "on_poweroff", &buf, 0)) buf = "destroy"; @@ -4607,7 +4607,7 @@ static void output_physinfo(void) /* * Only if enabled (claim_mode=1) or there are outstanding claims. */ - if (libxl_defbool_val(claim_mode) || info.outstanding_pages) + if (claim_mode || info.outstanding_pages) printf("outstanding_claims : %"PRIu64"\n", info.outstanding_pages / i); if (!libxl_get_freecpus(ctx, &cpumap)) { @@ -5911,7 +5911,7 @@ int main_claims(int argc, char **argv) /* No options */ } - if (!libxl_defbool_val(claim_mode)) + if (!claim_mode) fprintf(stderr, "claim_mode not enabled (see man xl.conf).\n"); info = libxl_list_domain(ctx, &nb_domain); |