From: Roger Pau Monne Date: Tue, 5 Mar 2013 17:06:29 +0000 (+0000) Subject: libxl: don't launch more than one tapdisk process for each disk X-Git-Tag: archive/raspbian/4.8.0-1+rpi1~1^2~7117 X-Git-Url: https://dgit.raspbian.org/?a=commitdiff_plain;h=ec398660e89ca18bb8d061d5047d682bd383778a;p=xen.git libxl: don't launch more than one tapdisk process for each disk When adding a disk don't launch multiple tapdisk instances for the same disk, if transaction fails in device_disk_add reuse the same tapdisk for further tries instead of creating a new instance each time a transaction fails. Reported-by: Darren Shepherd Signed-off-by: Roger Pau Monne Tested-by: Darren Shepherd --- diff --git a/tools/libxl/libxl.c b/tools/libxl/libxl.c index 73e0dc312d..345d88b83f 100644 --- a/tools/libxl/libxl.c +++ b/tools/libxl/libxl.c @@ -2032,7 +2032,7 @@ static void device_disk_add(libxl__egc *egc, uint32_t domid, STATE_AO_GC(aodev->ao); flexarray_t *front = NULL; flexarray_t *back = NULL; - char *dev, *script; + char *dev = NULL, *script; libxl__device *device; int rc; libxl_ctx *ctx = gc->owner; @@ -2095,12 +2095,15 @@ static void device_disk_add(libxl__egc *egc, uint32_t domid, break; case LIBXL_DISK_BACKEND_TAP: - dev = libxl__blktap_devpath(gc, disk->pdev_path, disk->format); - if (!dev) { - LOG(ERROR, "failed to get blktap devpath for %p\n", - disk->pdev_path); - rc = ERROR_FAIL; - goto out; + if (dev == NULL) { + dev = libxl__blktap_devpath(gc, disk->pdev_path, + disk->format); + if (!dev) { + LOG(ERROR, "failed to get blktap devpath for %p\n", + disk->pdev_path); + rc = ERROR_FAIL; + goto out; + } } flexarray_append(back, "tapdisk-params"); flexarray_append(back, libxl__sprintf(gc, "%s:%s",