Mailing List Archive

[xen-unstable] libxl: remove impossible check for backend != DISK_BACKEND_QDISK
# HG changeset patch
# User Ian Campbell <ian.campbell@citrix.com>
# Date 1302277086 -3600
# Node ID bad528ba2791392f7ffc9ff4bce0021a536a02b6
# Parent 1ac49d2c7d7012c27435e78706b03272cf708efc
libxl: remove impossible check for backend != DISK_BACKEND_QDISK

In this case we are already in the DISK_BACKEND_QDISK case of a switch
statement on the same variable.

It is possible that we fell through from the DISK_BACKEND_TAP case
(although I'm about to remove that in a subsequent patch), however in
that case we are explicitly falling back from blktap2 to qdisk so
DEVICE_QDISK is still the right answer.

Signed-off-by: Ian Campbell <ian.campbell@citrix.com>
Acked-by: Ian Jackson <ian.jackson@eu.citrix.com>
Committed-by: Ian Jackson <ian.jackson@eu.citrix.com>
---


diff -r 1ac49d2c7d70 -r bad528ba2791 tools/libxl/libxl.c
--- a/tools/libxl/libxl.c Fri Apr 08 16:36:20 2011 +0100
+++ b/tools/libxl/libxl.c Fri Apr 08 16:38:06 2011 +0100
@@ -1016,12 +1016,7 @@
flexarray_append(back, "params");
flexarray_append(back, libxl__sprintf(&gc, "%s:%s",
libxl__device_disk_string_of_format(disk->format), disk->pdev_path));
-
- if (libxl__blktap_enabled(&gc) &&
- disk->backend != DISK_BACKEND_QDISK)
- device.backend_kind = DEVICE_TAP;
- else
- device.backend_kind = DEVICE_QDISK;
+ device.backend_kind = DEVICE_QDISK;
break;
default:
LIBXL__LOG(ctx, LIBXL__LOG_ERROR, "unrecognized disk backend type: %d\n", disk->backend);

_______________________________________________
Xen-changelog mailing list
Xen-changelog@lists.xensource.com
http://lists.xensource.com/xen-changelog