]> xenbits.xensource.com Git - people/tklengyel/xen.git/commitdiff
libs/vchan: Fix -Wsingle-bit-bitfield-constant-conversion
authorAndrew Cooper <andrew.cooper3@citrix.com>
Tue, 8 Aug 2023 13:53:42 +0000 (14:53 +0100)
committerAndrew Cooper <andrew.cooper3@citrix.com>
Tue, 8 Aug 2023 13:57:49 +0000 (14:57 +0100)
Gitlab reports:

  init.c:348:18: error: implicit truncation from 'int' to a one-bit wide bit-field changes value from 1 to -1 [-Werror,-Wsingle-bit-bitfield-constant-conversion]
          ctrl->is_server = 1;
                          ^ ~
  1 error generated.
  make[6]: *** [/builds/xen-project/people/andyhhp/xen/tools/libs/vchan/../../../tools/Rules.mk:188: init.o] Error 1

In Xen 4.18, this was fixed with c/s 99ab02f63ea8 ("tools: convert bitfields
to unsigned type") but this is an ABI change which can't be backported.

Swich 1 for -1 to provide a minimally invasive way to fix the build.

No functional change.

Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com>
tools/libs/vchan/init.c

index 9195bd3b98759897d324bf0a712f7788be1e5865..b3de8f96fe3a6283dc4809303af9ce257ee669b9 100644 (file)
@@ -345,7 +345,7 @@ struct libxenvchan *libxenvchan_server_init(struct xentoollog_logger *logger,
 
        ctrl->ring = NULL;
        ctrl->event = NULL;
-       ctrl->is_server = 1;
+       ctrl->is_server = -1;
        ctrl->server_persist = 0;
 
        ctrl->read.order = min_order(left_min);