On Mon, 2019-06-10 at 23:03 +0800, Ming Lei wrote:
> The current way isn't safe for chained sgl, so use sgl helper to
> operate sgl.
> 
> Signed-off-by: Ming Lei <[email protected]>
> ---
>  drivers/scsi/vmw_pvscsi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/scsi/vmw_pvscsi.c b/drivers/scsi/vmw_pvscsi.c
> index ecee4b3ff073..d71abd416eb4 100644
> --- a/drivers/scsi/vmw_pvscsi.c
> +++ b/drivers/scsi/vmw_pvscsi.c
> @@ -335,7 +335,7 @@ static void pvscsi_create_sg(struct pvscsi_ctx *ctx,
>       BUG_ON(count > PVSCSI_MAX_NUM_SG_ENTRIES_PER_SEGMENT);
>  
>       sge = &ctx->sgl->sge[0];
> -     for (i = 0; i < count; i++, sg++) {
> +     for (i = 0; i < count; i++, sg = sg_next(sg)) {
>               sge[i].addr   = sg_dma_address(sg);
>               sge[i].length = sg_dma_len(sg);
>               sge[i].flags  = 0;

Reviewed-by: Ewan D. Milne <[email protected]>

Reply via email to