[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 14/14] usb: Proper error propagation for usb_dev
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH 14/14] usb: Proper error propagation for usb_device_attach errors |
Date: |
Tue, 31 May 2011 11:56:53 +0200 |
User-agent: |
Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc15 Thunderbird/3.1.10 |
Am 31.05.2011 11:51, schrieb Hans de Goede:
> Hi,
>
> On 05/31/2011 11:42 AM, Michael Tokarev wrote:
>> 31.05.2011 13:35, Hans de Goede wrote:
>>> ---
>>> hw/usb-bus.c | 23 ++++++++++++-----------
>>> hw/usb-msd.c | 5 +++--
>>> usb-linux.c | 6 +++++-
>>> 3 files changed, 20 insertions(+), 14 deletions(-)
>>>
>>> diff --git a/hw/usb-bus.c b/hw/usb-bus.c
>>> index 0a49921..2ae2678 100644
>>> --- a/hw/usb-bus.c
>>> +++ b/hw/usb-bus.c
>>
>>> if (dev->attached) {
>>> - fprintf(stderr, "Warning: tried to attach usb device %s twice\n",
>>> + fprintf(stderr, "Error: tried to attach usb device %s twice\n",
>>> dev->product_desc);
>>
>> qemu_error() maybe, while we're at it?
>> Here and in a few other places.
>
> That does not seem to exist, do you perhaps mean error_printf() ?
error_report() is what you should use, so that messages go to the
monitor if the function is called from a monitor command. error_printf()
is used by it internally, but usually isn't used directly.
Kevin
- [Qemu-devel] [PATCH 03/14] usb-linux: Teach about super speed, (continued)
- [Qemu-devel] [PATCH 03/14] usb-linux: Teach about super speed, Hans de Goede, 2011/05/31
- [Qemu-devel] [PATCH 07/14] usb-linux: If opening a device fails remove it from our filter list, Hans de Goede, 2011/05/31
- [Qemu-devel] [PATCH 08/14] usb-linux: Don't try to open the same device twice, Hans de Goede, 2011/05/31
- [Qemu-devel] [PATCH 10/14] usb-linux: Enlarge buffer for descriptors to 8192 bytes, Hans de Goede, 2011/05/31
- [Qemu-devel] [PATCH 09/14] usb-linux: Don't declare a usbdevice_name, Hans de Goede, 2011/05/31
- [Qemu-devel] [PATCH 11/14] usb-bus: Add knowledge of USB_SPEED_SUPER to usb_speed helper, Hans de Goede, 2011/05/31
- [Qemu-devel] [PATCH 12/14] usb-bus: Don't allow attaching a device to a bus with no free ports, Hans de Goede, 2011/05/31
- [Qemu-devel] [PATCH 14/14] usb: Proper error propagation for usb_device_attach errors, Hans de Goede, 2011/05/31
[Qemu-devel] [PATCH 13/14] usb-bus: Don't detach non attached devices on device exit, Hans de Goede, 2011/05/31