From 857221510ae8441ca2d90e6360ad1201ccc9099c Mon Sep 17 00:00:00 2001 From: Gerd Hoffmann Date: Thu, 21 Jul 2011 16:43:43 -0300 Subject: [RHEL6 qemu-kvm PATCH 05/65] usb: Proper error propagation for usb_device_attach errors RH-Author: Gerd Hoffmann Message-id: <1311266648-1179-6-git-send-email-kraxel@redhat.com> Patchwork-id: 30136 O-Subject: [RHEL-6.2 kvm PATCH 05/30] usb: Proper error propagation for usb_device_attach errors Bugzilla: 723858 723863 RH-Acked-by: Hans de Goede RH-Acked-by: Kevin Wolf RH-Acked-by: Jes Sorensen RH-Acked-by: Alex Williamson From: Hans de Goede Signed-off-by: Gerd Hoffmann (cherry picked from commit fa19bf831b8c87006533645ece58bc620aca00bc) --- hw/usb-bus.c | 25 +++++++++++++------------ hw/usb-msd.c | 5 +++-- usb-linux.c | 6 +++++- 3 files changed, 21 insertions(+), 15 deletions(-) Signed-off-by: Eduardo Habkost --- hw/usb-bus.c | 25 +++++++++++++------------ hw/usb-msd.c | 5 +++-- usb-linux.c | 6 +++++- 3 files changed, 21 insertions(+), 15 deletions(-) diff --git a/hw/usb-bus.c b/hw/usb-bus.c index 935c4ee..bd40668 100644 --- a/hw/usb-bus.c +++ b/hw/usb-bus.c @@ -75,7 +75,7 @@ static int usb_qdev_init(DeviceState *qdev, DeviceInfo *base) QLIST_INIT(&dev->strings); rc = dev->info->init(dev); if (rc == 0 && dev->auto_attach) - usb_device_attach(dev); + rc = usb_device_attach(dev); return rc; } @@ -121,7 +121,7 @@ USBDevice *usb_create(USBBus *bus, const char *name) bus = usb_bus_find(-1); if (!bus) return NULL; - fprintf(stderr, "%s: no bus specified, using \"%s\" for \"%s\"\n", + error_report("%s: no bus specified, using \"%s\" for \"%s\"\n", __FUNCTION__, bus->qbus.name, name); } #endif @@ -168,20 +168,20 @@ void usb_unregister_port(USBBus *bus, USBPort *port) bus->nfree--; } -static void do_attach(USBDevice *dev) +static int do_attach(USBDevice *dev) { USBBus *bus = usb_bus_from_device(dev); USBPort *port; if (dev->attached) { - fprintf(stderr, "Warning: tried to attach usb device %s twice\n", + error_report("Error: tried to attach usb device %s twice\n", dev->product_desc); - return; + return -1; } if (bus->nfree == 0) { - fprintf(stderr, "Warning: tried to attach usb device %s to a bus with no free ports\n", + error_report("Error: tried to attach usb device %s to a bus with no free ports\n", dev->product_desc); - return; + return -1; } if (dev->port_path) { QTAILQ_FOREACH(port, &bus->free, next) { @@ -190,9 +190,9 @@ static void do_attach(USBDevice *dev) } } if (port == NULL) { - fprintf(stderr, "Warning: usb port %s (bus %s) not found\n", + error_report("Error: usb port %s (bus %s) not found\n", dev->port_path, bus->qbus.name); - return; + return -1; } } else { port = QTAILQ_FIRST(&bus->free); @@ -206,6 +206,8 @@ static void do_attach(USBDevice *dev) QTAILQ_INSERT_TAIL(&bus->used, port, next); bus->nused++; + + return 0; } int usb_device_attach(USBDevice *dev) @@ -217,8 +219,7 @@ int usb_device_attach(USBDevice *dev) (unless a physical port location is specified). */ usb_create_simple(bus, "usb-hub"); } - do_attach(dev); - return 0; + return do_attach(dev); } int usb_device_detach(USBDevice *dev) @@ -227,7 +228,7 @@ int usb_device_detach(USBDevice *dev) USBPort *port; if (!dev->attached) { - fprintf(stderr, "Warning: tried to detach unattached usb device %s\n", + error_report("Error: tried to detach unattached usb device %s\n", dev->product_desc); return -1; } diff --git a/hw/usb-msd.c b/hw/usb-msd.c index 73058dc..d7fb560 100644 --- a/hw/usb-msd.c +++ b/hw/usb-msd.c @@ -480,8 +480,9 @@ static void usb_msd_password_cb(void *opaque, int err) MSDState *s = opaque; if (!err) - usb_device_attach(&s->dev); - else + err = usb_device_attach(&s->dev); + + if (err) qdev_unplug(&s->dev.qdev); } diff --git a/usb-linux.c b/usb-linux.c index 2f8353e..3b819d4 100644 --- a/usb-linux.c +++ b/usb-linux.c @@ -1168,10 +1168,14 @@ static int usb_host_open(USBHostDevice *dev, int bus_num, pstrcpy(dev->dev.product_desc, sizeof(dev->dev.product_desc), prod_name); + ret = usb_device_attach(&dev->dev); + if (ret) { + goto fail; + } + /* USB devio uses 'write' flag to check for async completions */ qemu_set_fd_handler(dev->fd, NULL, async_complete, dev); - usb_device_attach(&dev->dev); return 0; fail: -- 1.7.3.2