[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-block] [PATCH V2] fix:readcapacity 10 failure was shown even 1
From: |
Eric Blake |
Subject: |
Re: [Qemu-block] [PATCH V2] fix:readcapacity 10 failure was shown even 16 sent |
Date: |
Mon, 14 Dec 2015 07:58:28 -0700 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 |
[adding qemu-devel - ALL patches must cc qemu-devel, even if they are
also sent to a sublist like qemu-block]
On 12/13/2015 07:52 PM, Zhu Lingshan wrote:
> Follow Peter's comment, it is the patch V2.
This sentence should appear...
> This patch can fix that error: readcapacity 10 failure will shown even it was
> readcapacity16 sent.
Long line; try to keep commit messages wrapped around 72 or fewer
characters, since 'git log' will display it with indentation and you
still want to read it all in an 80-column window.
Subject line is a bit awkward; I suggest:
iscsi: fix readcapacity error message
The error message for readcapacity 16 incorrectly mentioned a
readcapacity 10 failure.
>
> Signed-off-by: Zhu Lingshan <address@hidden>
> ---
...here, since it explains why you sent v2, but will make no sense a
year from now when browsing just qemu.git.
> block/iscsi.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/block/iscsi.c b/block/iscsi.c
> index bd1f1bf..bd3473e 100644
> --- a/block/iscsi.c
> +++ b/block/iscsi.c
> @@ -1270,7 +1270,7 @@ static void iscsi_readcapacity_sync(IscsiLun *iscsilun,
> Error **errp)
> && retries-- > 0);
>
> if (task == NULL || task->status != SCSI_STATUS_GOOD) {
> - error_setg(errp, "iSCSI: failed to send readcapacity10 command.");
> + error_setg(errp, "iSCSI: failed to send readcapacity10/16 command.");
While you're touching this, please also drop the trailing '.'
> } else if (!iscsilun->block_size ||
> iscsilun->block_size % BDRV_SECTOR_SIZE) {
> error_setg(errp, "iSCSI: the target returned an invalid "
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature