[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] scsi: Improve error messages more
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] scsi: Improve error messages more |
Date: |
Mon, 28 Apr 2014 13:03:04 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.2 (gnu/linux) |
Paolo Bonzini <address@hidden> writes:
> Make the message the same for both scsi-block and scsi-generic,
> including the strerror() output in both and making an explicit
> reference to SG_IO. Also s/can not/cannot/.
>
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> hw/scsi/scsi-disk.c | 5 +++--
> hw/scsi/scsi-generic.c | 8 ++++++--
> 2 files changed, 9 insertions(+), 4 deletions(-)
>
> diff --git a/hw/scsi/scsi-disk.c b/hw/scsi/scsi-disk.c
> index d2e532e..342a78b 100644
> --- a/hw/scsi/scsi-disk.c
> +++ b/hw/scsi/scsi-disk.c
> @@ -2463,9 +2463,10 @@ static int scsi_block_initfn(SCSIDevice *dev)
> }
>
> /* check we are using a driver managing SG_IO (version 3 and after) */
> - rc = bdrv_ioctl(s->qdev.conf.bs, SG_GET_VERSION_NUM, &sg_version);
> + rc = bdrv_ioctl(s->qdev.conf.bs, SG_GET_VERSION_NUM, &sg_version);
> if (rc < 0) {
> - error_report("scsi-block: can not get version number: %s",
> + error_report("scsi-block: cannot get SG_IO version number: %s. "
> + "Is this a SCSI device?",
> strerror(-rc));
> return -1;
> }
While you're improving: is the scsi-block: prefix still useful? The
error message comes out like this for me:
qemu: -device scsi-block,drive=foo: scsi-block: cannot get SG_IO version
number: Operation not supported. Is this a SCSI device?
[...]