[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/5] hw/scsi/spapr_vscsi: Simplify a bit
From: |
Greg Kurz |
Subject: |
Re: [PATCH 3/5] hw/scsi/spapr_vscsi: Simplify a bit |
Date: |
Thu, 5 Mar 2020 11:21:46 +0100 |
On Wed, 4 Mar 2020 16:33:09 +0100
Philippe Mathieu-Daudé <address@hidden> wrote:
> We already have a ui pointer, use it (to simplify the next commit).
>
Small typo, s/ui/iu
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
> hw/scsi/spapr_vscsi.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/hw/scsi/spapr_vscsi.c b/hw/scsi/spapr_vscsi.c
> index 7e397ed797..3cb5a38181 100644
> --- a/hw/scsi/spapr_vscsi.c
> +++ b/hw/scsi/spapr_vscsi.c
> @@ -261,9 +261,9 @@ static int vscsi_send_rsp(VSCSIState *s, vscsi_req *req,
> if (status) {
> iu->srp.rsp.sol_not = (sol_not & 0x04) >> 2;
> if (req->senselen) {
> - req->iu.srp.rsp.flags |= SRP_RSP_FLAG_SNSVALID;
> - req->iu.srp.rsp.sense_data_len = cpu_to_be32(req->senselen);
> - memcpy(req->iu.srp.rsp.data, req->sense, req->senselen);
> + iu->srp.rsp.flags |= SRP_RSP_FLAG_SNSVALID;
> + iu->srp.rsp.sense_data_len = cpu_to_be32(req->senselen);
> + memcpy(iu->srp.rsp.data, req->sense, req->senselen);
> total_len += req->senselen;
> }
> } else {
- [PATCH 0/5] hw/scsi/spapr_vscsi: Fix time bomb zero-length array use, Philippe Mathieu-Daudé, 2020/03/04
- [PATCH 2/5] hw/scsi/spapr_vscsi: Use SRP_MAX_IU_LEN instead of sizeof flexible array, Philippe Mathieu-Daudé, 2020/03/04
- [PATCH 1/5] hw/scsi/viosrp: Add missing 'hw/scsi/srp.h' include, Philippe Mathieu-Daudé, 2020/03/04
- [PATCH 3/5] hw/scsi/spapr_vscsi: Simplify a bit, Philippe Mathieu-Daudé, 2020/03/04
- [PATCH 4/5] hw/scsi/spapr_vscsi: Introduce req_ui() helper, Philippe Mathieu-Daudé, 2020/03/04
- [PATCH 5/5] hw/scsi/spapr_vscsi: Do not mix SRP IU size with DMA buffer size, Philippe Mathieu-Daudé, 2020/03/04