[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 33/46] hw/scsi: Replace fprintf(stderr, "*\n"
From: |
David Gibson |
Subject: |
Re: [Qemu-devel] [PATCH v3 33/46] hw/scsi: Replace fprintf(stderr, "*\n" with error_report() |
Date: |
Fri, 20 Oct 2017 10:50:55 +1100 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Thu, Oct 19, 2017 at 09:17:40AM -0700, Alistair Francis wrote:
> Replace a large number of the fprintf(stderr, "*\n" calls with
> error_report(). The functions were renamed with these commands and then
> compiler issues where manually fixed.
>
> find ./* -type f -exec sed -i \
> 'N;N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr,
> "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N;N;N;N;N;N;N;N;N;N; {s|fprintf(stderr,
> "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N;N;N;N;N;N;N;N; {s|fprintf(stderr,
> "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N;N;N;N;N;N;N; {s|fprintf(stderr,
> "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N;N;N;N;N;N; {s|fprintf(stderr,
> "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N;N;N;N;N; {s|fprintf(stderr,
> "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N;N;N;N; {s|fprintf(stderr,
> "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N;N;N; {s|fprintf(stderr,
> "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N;N; {s|fprintf(stderr, "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}'
> \
> {} +
> find ./* -type f -exec sed -i \
> 'N;N; {s|fprintf(stderr, "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
> find ./* -type f -exec sed -i \
> 'N; {s|fprintf(stderr, "\(.*\)\\n"\(.*\));|error_report("\1"\2);|Ig}' \
> {} +
>
> Some lines where then manually tweaked to pass checkpatch.
>
> Signed-off-by: Alistair Francis <address@hidden>
> Cc: Paolo Bonzini <address@hidden>
> Cc: David Gibson <address@hidden>
> Cc: Alexander Graf <address@hidden>
> ---
> V2:
> - Split hw patch into individual directories
>
> hw/scsi/lsi53c895a.c | 4 ++--
> hw/scsi/spapr_vscsi.c | 49
> +++++++++++++++++++++--------------------
> hw/scsi/virtio-scsi-dataplane.c | 6 ++---
> 3 files changed, 30 insertions(+), 29 deletions(-)
virtio_scsi part
Acked-by: David Gibson <address@hidden>
>
> diff --git a/hw/scsi/lsi53c895a.c b/hw/scsi/lsi53c895a.c
> index 191505df5b..7c9a930e2f 100644
> --- a/hw/scsi/lsi53c895a.c
> +++ b/hw/scsi/lsi53c895a.c
> @@ -14,7 +14,7 @@
> */
>
> #include "qemu/osdep.h"
> -
> +#include "qemu/error-report.h"
> #include "hw/hw.h"
> #include "hw/pci/pci.h"
> #include "hw/scsi/scsi.h"
> @@ -1501,7 +1501,7 @@ again:
> This is apparently sufficient to beat the drivers into submission.
> */
> if (!(s->sien0 & LSI_SIST0_UDC))
> - fprintf(stderr, "inf. loop with UDC masked\n");
> + error_report("inf. loop with UDC masked");
> lsi_script_scsi_interrupt(s, LSI_SIST0_UDC, 0);
> lsi_disconnect(s);
> } else if (s->istat1 & LSI_ISTAT1_SRUN && !s->waiting) {
> diff --git a/hw/scsi/spapr_vscsi.c b/hw/scsi/spapr_vscsi.c
> index 360db53ac8..8c4eabcf3d 100644
> --- a/hw/scsi/spapr_vscsi.c
> +++ b/hw/scsi/spapr_vscsi.c
> @@ -32,6 +32,7 @@
> * - Maybe do autosense (PAPR seems to mandate it, linux doesn't care)
> */
> #include "qemu/osdep.h"
> +#include "qemu/error-report.h"
> #include "qemu-common.h"
> #include "cpu.h"
> #include "hw/hw.h"
> @@ -179,7 +180,7 @@ static int vscsi_send_iu(VSCSIState *s, vscsi_req *req,
> rc = spapr_vio_dma_write(&s->vdev, req->crq.s.IU_data_ptr,
> &req->iu, length);
> if (rc) {
> - fprintf(stderr, "vscsi_send_iu: DMA write failure !\n");
> + error_report("vscsi_send_iu: DMA write failure !");
> }
>
> req->crq.s.valid = 0x80;
> @@ -197,7 +198,7 @@ static int vscsi_send_iu(VSCSIState *s, vscsi_req *req,
>
> rc1 = spapr_vio_send_crq(&s->vdev, req->crq.raw);
> if (rc1) {
> - fprintf(stderr, "vscsi_send_iu: Error sending response\n");
> + error_report("vscsi_send_iu: Error sending response");
> return rc1;
> }
>
> @@ -330,7 +331,7 @@ static int vscsi_fetch_desc(VSCSIState *s, struct
> vscsi_req *req,
> break;
> }
> default:
> - fprintf(stderr, "VSCSI: Unknown format %x\n", req->dma_fmt);
> + error_report("VSCSI: Unknown format %x", req->dma_fmt);
> return -1;
> }
>
> @@ -518,7 +519,7 @@ static void vscsi_transfer_data(SCSIRequest *sreq,
> uint32_t len)
>
> trace_spapr_vscsi_transfer_data(sreq->tag, len, req);
> if (req == NULL) {
> - fprintf(stderr, "VSCSI: Can't find request for tag 0x%x\n",
> sreq->tag);
> + error_report("VSCSI: Can't find request for tag 0x%x", sreq->tag);
> return;
> }
>
> @@ -527,7 +528,7 @@ static void vscsi_transfer_data(SCSIRequest *sreq,
> uint32_t len)
> rc = vscsi_srp_transfer_data(s, req, req->writing, buf, len);
> }
> if (rc < 0) {
> - fprintf(stderr, "VSCSI: RDMA error rc=%d!\n", rc);
> + error_report("VSCSI: RDMA error rc=%d!", rc);
> req->dma_error = true;
> scsi_req_cancel(req->sreq);
> return;
> @@ -547,7 +548,7 @@ static void vscsi_command_complete(SCSIRequest *sreq,
> uint32_t status, size_t re
>
> trace_spapr_vscsi_command_complete(sreq->tag, status, req);
> if (req == NULL) {
> - fprintf(stderr, "VSCSI: Can't find request for tag 0x%x\n",
> sreq->tag);
> + error_report("VSCSI: Can't find request for tag 0x%x", sreq->tag);
> return;
> }
>
> @@ -639,7 +640,7 @@ static void *vscsi_load_request(QEMUFile *f, SCSIRequest
> *sreq)
> memset(req, 0, sizeof(*req));
> rc = vmstate_load_state(f, &vmstate_spapr_vscsi_req, req, 1);
> if (rc) {
> - fprintf(stderr, "VSCSI: failed loading request tag#%u\n", sreq->tag);
> + error_report("VSCSI: failed loading request tag#%u", sreq->tag);
> return NULL;
> }
> assert(req->active);
> @@ -827,7 +828,7 @@ static int vscsi_process_tsk_mgmt(VSCSIState *s,
> vscsi_req *req)
> uint64_t tag = iu->srp.rsp.tag;
> uint8_t sol_not = iu->srp.cmd.sol_not;
>
> - fprintf(stderr, "vscsi_process_tsk_mgmt %02x\n",
> + error_report("vscsi_process_tsk_mgmt %02x",
> iu->srp.tsk_mgmt.tsk_mgmt_func);
>
> d = vscsi_device_find(&s->bus, be64_to_cpu(req->iu.srp.tsk_mgmt.lun),
> &lun);
> @@ -932,10 +933,10 @@ static int vscsi_handle_srp_req(VSCSIState *s,
> vscsi_req *req)
> case SRP_CRED_RSP:
> case SRP_AER_REQ:
> case SRP_AER_RSP:
> - fprintf(stderr, "VSCSI: Unsupported opcode %02x\n", opcode);
> + error_report("VSCSI: Unsupported opcode %02x", opcode);
> break;
> default:
> - fprintf(stderr, "VSCSI: Unknown type %02x\n", opcode);
> + error_report("VSCSI: Unknown type %02x", opcode);
> }
>
> return done;
> @@ -953,7 +954,7 @@ static int vscsi_send_adapter_info(VSCSIState *s,
> vscsi_req *req)
> rc = spapr_vio_dma_read(&s->vdev, be64_to_cpu(sinfo->buffer),
> &info, be16_to_cpu(sinfo->common.length));
> if (rc) {
> - fprintf(stderr, "vscsi_send_adapter_info: DMA read failure !\n");
> + error_report("vscsi_send_adapter_info: DMA read failure !");
> }
> #endif
> memset(&info, 0, sizeof(info));
> @@ -967,7 +968,7 @@ static int vscsi_send_adapter_info(VSCSIState *s,
> vscsi_req *req)
> rc = spapr_vio_dma_write(&s->vdev, be64_to_cpu(sinfo->buffer),
> &info, be16_to_cpu(sinfo->common.length));
> if (rc) {
> - fprintf(stderr, "vscsi_send_adapter_info: DMA write failure !\n");
> + error_report("vscsi_send_adapter_info: DMA write failure !");
> }
>
> sinfo->common.status = rc ? cpu_to_be32(1) : 0;
> @@ -987,7 +988,7 @@ static int vscsi_send_capabilities(VSCSIState *s,
> vscsi_req *req)
> req_len = len = be16_to_cpu(vcap->common.length);
> buffer = be64_to_cpu(vcap->buffer);
> if (len > sizeof(cap)) {
> - fprintf(stderr, "vscsi_send_capabilities: capabilities size mismatch
> !\n");
> + error_report("vscsi_send_capabilities: capabilities size mismatch
> !");
>
> /*
> * Just read and populate the structure that is known.
> @@ -997,7 +998,7 @@ static int vscsi_send_capabilities(VSCSIState *s,
> vscsi_req *req)
> }
> rc = spapr_vio_dma_read(&s->vdev, buffer, &cap, len);
> if (rc) {
> - fprintf(stderr, "vscsi_send_capabilities: DMA read failure !\n");
> + error_report("vscsi_send_capabilities: DMA read failure !");
> }
>
> /*
> @@ -1013,7 +1014,7 @@ static int vscsi_send_capabilities(VSCSIState *s,
> vscsi_req *req)
>
> rc = spapr_vio_dma_write(&s->vdev, buffer, &cap, len);
> if (rc) {
> - fprintf(stderr, "vscsi_send_capabilities: DMA write failure !\n");
> + error_report("vscsi_send_capabilities: DMA write failure !");
> }
> if (req_len > len) {
> /*
> @@ -1034,11 +1035,11 @@ static int vscsi_handle_mad_req(VSCSIState *s,
> vscsi_req *req)
>
> switch (be32_to_cpu(mad->empty_iu.common.type)) {
> case VIOSRP_EMPTY_IU_TYPE:
> - fprintf(stderr, "Unsupported EMPTY MAD IU\n");
> + error_report("Unsupported EMPTY MAD IU");
> retlen = sizeof(mad->empty_iu);
> break;
> case VIOSRP_ERROR_LOG_TYPE:
> - fprintf(stderr, "Unsupported ERROR LOG MAD IU\n");
> + error_report("Unsupported ERROR LOG MAD IU");
> retlen = sizeof(mad->error_log);
> break;
> case VIOSRP_ADAPTER_INFO_TYPE:
> @@ -1053,7 +1054,7 @@ static int vscsi_handle_mad_req(VSCSIState *s,
> vscsi_req *req)
> request_handled = true;
> break;
> default:
> - fprintf(stderr, "VSCSI: Unknown MAD type %02x\n",
> + error_report("VSCSI: Unknown MAD type %02x",
> be32_to_cpu(mad->empty_iu.common.type));
> /*
> * PAPR+ says that "The length field is set to the length
> @@ -1078,7 +1079,7 @@ static void vscsi_got_payload(VSCSIState *s, vscsi_crq
> *crq)
>
> req = vscsi_get_req(s);
> if (req == NULL) {
> - fprintf(stderr, "VSCSI: Failed to get a request !\n");
> + error_report("VSCSI: Failed to get a request !");
> return;
> }
>
> @@ -1088,7 +1089,7 @@ static void vscsi_got_payload(VSCSIState *s, vscsi_crq
> *crq)
> * of the structure.
> */
> if (crq->s.IU_length > sizeof(union viosrp_iu)) {
> - fprintf(stderr, "VSCSI: SRP IU too long (%d bytes) !\n",
> + error_report("VSCSI: SRP IU too long (%d bytes) !",
> crq->s.IU_length);
> vscsi_put_req(req);
> return;
> @@ -1097,7 +1098,7 @@ static void vscsi_got_payload(VSCSIState *s, vscsi_crq
> *crq)
> /* XXX Handle failure differently ? */
> if (spapr_vio_dma_read(&s->vdev, crq->s.IU_data_ptr, &req->iu,
> crq->s.IU_length)) {
> - fprintf(stderr, "vscsi_got_payload: DMA read failure !\n");
> + error_report("vscsi_got_payload: DMA read failure !");
> vscsi_put_req(req);
> return;
> }
> @@ -1158,16 +1159,16 @@ static int vscsi_do_crq(struct VIOsPAPRDevice *dev,
> uint8_t *crq_data)
> case VIOSRP_AIX_FORMAT:
> case VIOSRP_LINUX_FORMAT:
> case VIOSRP_INLINE_FORMAT:
> - fprintf(stderr, "vscsi_do_srq: Unsupported payload format
> %02x\n",
> + error_report("vscsi_do_srq: Unsupported payload format %02x",
> crq.s.format);
> break;
> default:
> - fprintf(stderr, "vscsi_do_srq: Unknown payload format %02x\n",
> + error_report("vscsi_do_srq: Unknown payload format %02x",
> crq.s.format);
> }
> break;
> default:
> - fprintf(stderr, "vscsi_do_crq: unknown CRQ %02x %02x ...\n",
> + error_report("vscsi_do_crq: unknown CRQ %02x %02x ...",
> crq.raw[0], crq.raw[1]);
> };
>
> diff --git a/hw/scsi/virtio-scsi-dataplane.c b/hw/scsi/virtio-scsi-dataplane.c
> index add4b3f4a4..e047361310 100644
> --- a/hw/scsi/virtio-scsi-dataplane.c
> +++ b/hw/scsi/virtio-scsi-dataplane.c
> @@ -97,7 +97,7 @@ static int virtio_scsi_vring_init(VirtIOSCSI *s, VirtQueue
> *vq, int n,
> /* Set up virtqueue notify */
> rc = virtio_bus_set_host_notifier(VIRTIO_BUS(qbus), n, true);
> if (rc != 0) {
> - fprintf(stderr, "virtio-scsi: Failed to set host notifier (%d)\n",
> + error_report("virtio-scsi: Failed to set host notifier (%d)",
> rc);
> s->dataplane_fenced = true;
> return rc;
> @@ -141,8 +141,8 @@ int virtio_scsi_dataplane_start(VirtIODevice *vdev)
> /* Set up guest notifier (irq) */
> rc = k->set_guest_notifiers(qbus->parent, vs->conf.num_queues + 2, true);
> if (rc != 0) {
> - fprintf(stderr, "virtio-scsi: Failed to set guest notifiers (%d), "
> - "ensure -enable-kvm is set\n", rc);
> + error_report("virtio-scsi: Failed to set guest notifiers (%d), "
> + "ensure -enable-kvm is set", rc);
> goto fail_guest_notifiers;
> }
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
- [Qemu-devel] [PATCH v3 25/46] hw/moxie: Replace fprintf(stderr, "*\n" with error_report(), (continued)
- [Qemu-devel] [PATCH v3 25/46] hw/moxie: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 26/46] hw/net: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 28/46] hw/nvram: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 27/46] hw/nios2: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 32/46] hw/s390x: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 30/46] hw/pci*: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 29/46] hw/openrisc: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 31/46] hw/ppc: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 33/46] hw/scsi: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- Re: [Qemu-devel] [PATCH v3 33/46] hw/scsi: Replace fprintf(stderr, "*\n" with error_report(),
David Gibson <=
- [Qemu-devel] [PATCH v3 34/46] hw/sd: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 35/46] hw/sh4: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 37/46] hw/ssi: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 36/46] hw/sparc*: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 38/46] hw/timer: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 40/46] hw/watchdog: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19
- [Qemu-devel] [PATCH v3 41/46] hw/xen*: Replace fprintf(stderr, "*\n" with error_report(), Alistair Francis, 2017/10/19