[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 08/19] dev-storage: usring error_report instead
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 08/19] dev-storage: usring error_report instead of fprintf/printf |
Date: |
Thu, 18 Sep 2014 12:13:28 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.0 |
Il 18/09/2014 11:32, address@hidden ha scritto:
> From: Gonglei <address@hidden>
>
> Signed-off-by: Gonglei <address@hidden>
> ---
> hw/usb/dev-storage.c | 12 ++++++------
> 1 file changed, 6 insertions(+), 6 deletions(-)
>
> diff --git a/hw/usb/dev-storage.c b/hw/usb/dev-storage.c
> index 6dc5f47..681959d 100644
> --- a/hw/usb/dev-storage.c
> +++ b/hw/usb/dev-storage.c
> @@ -409,19 +409,19 @@ static void usb_msd_handle_data(USBDevice *dev,
> USBPacket *p)
> switch (s->mode) {
> case USB_MSDM_CBW:
> if (p->iov.size != 31) {
> - fprintf(stderr, "usb-msd: Bad CBW size");
> + error_report("usb-msd: Bad CBW size");
> goto fail;
> }
> usb_packet_copy(p, &cbw, 31);
> if (le32_to_cpu(cbw.sig) != 0x43425355) {
> - fprintf(stderr, "usb-msd: Bad signature %08x\n",
> - le32_to_cpu(cbw.sig));
> + error_report("usb-msd: Bad signature %08x",
> + le32_to_cpu(cbw.sig));
> goto fail;
> }
> DPRINTF("Command on LUN %d\n", cbw.lun);
> scsi_dev = scsi_device_find(&s->bus, 0, 0, cbw.lun);
> if (scsi_dev == NULL) {
> - fprintf(stderr, "usb-msd: Bad LUN %d\n", cbw.lun);
> + error_report("usb-msd: Bad LUN %d", cbw.lun);
> goto fail;
> }
> tag = le32_to_cpu(cbw.tag);
> @@ -680,13 +680,13 @@ static USBDevice *usb_msd_init(USBBus *bus, const char
> *filename)
> pstrcpy(fmt, len, p2);
> qemu_opt_set(opts, "format", fmt);
> } else if (*filename != ':') {
> - printf("unrecognized USB mass-storage option %s\n", filename);
> + error_report("unrecognized USB mass-storage option %s",
> filename);
> return NULL;
> }
> filename = p1;
> }
> if (!*filename) {
> - printf("block device specification needed\n");
> + error_report("block device specification needed");
> return NULL;
> }
> qemu_opt_set(opts, "file", filename);
>
Reviewed-by: Paolo Bonzini <address@hidden>
- Re: [Qemu-devel] [PATCH 02/19] usb-bus: convert USBDeviceClass init to realize, (continued)
- [Qemu-devel] [PATCH 06/19] usb-hub: convert init to realize, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 01/19] usb-storage: fix possible memory leak and missing error message, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 04/19] libusb: convert init to realize, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 08/19] dev-storage: usring error_report instead of fprintf/printf, arei.gonglei, 2014/09/18
- Re: [Qemu-devel] [PATCH 08/19] dev-storage: usring error_report instead of fprintf/printf,
Paolo Bonzini <=
- [Qemu-devel] [PATCH 12/19] dev-serial: convert init to realize, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 10/19] dev-uas: using error_report instead of fprintf, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 18/19] usb-mtp: convert init to realize, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 19/19] usb-bus: remove "init" from USBDeviceClass struct, arei.gonglei, 2014/09/18
- [Qemu-devel] [PATCH 13/19] usb-ccid: convert init to realize, arei.gonglei, 2014/09/18