[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 05/20] iscsi: fix indentation
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH 05/20] iscsi: fix indentation |
Date: |
Mon, 10 Feb 2014 15:48:55 +0800 |
User-agent: |
Mutt/1.5.22 (2013-10-16) |
On Sun, 02/09 10:48, Paolo Bonzini wrote:
> Signed-off-by: Paolo Bonzini <address@hidden>
> ---
> block/iscsi.c | 45 +++++++++++++++++++++++----------------------
> 1 file changed, 23 insertions(+), 22 deletions(-)
>
> diff --git a/block/iscsi.c b/block/iscsi.c
> index 6f4af72..e654a57 100644
> --- a/block/iscsi.c
> +++ b/block/iscsi.c
> @@ -1065,35 +1065,36 @@ static QemuOptsList runtime_opts = {
> },
> };
>
> -static struct scsi_task *iscsi_do_inquiry(struct iscsi_context *iscsi,
> - int lun, int evpd, int pc) {
> - int full_size;
> - struct scsi_task *task = NULL;
> - task = iscsi_inquiry_sync(iscsi, lun, evpd, pc, 64);
> +static struct scsi_task *iscsi_do_inquiry(struct iscsi_context *iscsi, int
> lun,
> + int evpd, int pc)
> +{
> + int full_size;
> + struct scsi_task *task = NULL;
> + task = iscsi_inquiry_sync(iscsi, lun, evpd, pc, 64);
> + if (task == NULL || task->status != SCSI_STATUS_GOOD) {
> + goto fail;
> + }
> + full_size = scsi_datain_getfullsize(task);
> + if (full_size > task->datain.size) {
> + scsi_free_scsi_task(task);
> +
> + /* we need more data for the full list */
> + task = iscsi_inquiry_sync(iscsi, lun, evpd, pc, full_size);
> if (task == NULL || task->status != SCSI_STATUS_GOOD) {
> goto fail;
> }
> - full_size = scsi_datain_getfullsize(task);
> - if (full_size > task->datain.size) {
> - scsi_free_scsi_task(task);
> -
> - /* we need more data for the full list */
> - task = iscsi_inquiry_sync(iscsi, lun, evpd, pc, full_size);
> - if (task == NULL || task->status != SCSI_STATUS_GOOD) {
> - goto fail;
> - }
> - }
> + }
>
> - return task;
> + return task;
>
> fail:
> - error_report("iSCSI: Inquiry command failed : %s",
> - iscsi_get_error(iscsi));
> - if (task) {
> - scsi_free_scsi_task(task);
> - return NULL;
> - }
> + error_report("iSCSI: Inquiry command failed : %s",
> + iscsi_get_error(iscsi));
> + if (task) {
> + scsi_free_scsi_task(task);
> return NULL;
> + }
> + return NULL;
> }
>
> /*
> --
> 1.8.5.3
>
Reviewed-by: Fam Zheng <address@hidden>
- Re: [Qemu-devel] [PATCH 02/20] nbd: correctly propagate errors, (continued)
[Qemu-devel] [PATCH 03/20] nbd: inline tcp_socket_incoming_spec into sole caller, Paolo Bonzini, 2014/02/09
[Qemu-devel] [PATCH 09/20] cow: correctly propagate errors, Paolo Bonzini, 2014/02/09
[Qemu-devel] [PATCH 04/20] nbd: move socket wrappers to qemu-nbd, Paolo Bonzini, 2014/02/09
[Qemu-devel] [PATCH 05/20] iscsi: fix indentation, Paolo Bonzini, 2014/02/09
- Re: [Qemu-devel] [PATCH 05/20] iscsi: fix indentation,
Fam Zheng <=
[Qemu-devel] [PATCH 06/20] iscsi: correctly propagate errors in iscsi_open, Paolo Bonzini, 2014/02/09
[Qemu-devel] [PATCH 07/20] gluster: default scheme to gluster:// and host to localhost., Paolo Bonzini, 2014/02/09
[Qemu-devel] [PATCH 08/20] gluster: correctly propagate errors, Paolo Bonzini, 2014/02/09
[Qemu-devel] [PATCH 10/20] curl: correctly propagate errors, Paolo Bonzini, 2014/02/09