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 <ming.lei@xxxxxxxxxx> > --- > drivers/scsi/advansys.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/scsi/advansys.c b/drivers/scsi/advansys.c > index 926311c792d5..a242a62caaa1 100644 > --- a/drivers/scsi/advansys.c > +++ b/drivers/scsi/advansys.c > @@ -7710,7 +7710,7 @@ adv_get_sglist(struct asc_board *boardp, adv_req_t *reqp, > sg_block->sg_ptr = 0L; /* Last ADV_SG_BLOCK in list. */ > return ADV_SUCCESS; > } > - slp++; > + slp = sg_next(slp); > } > sg_block->sg_cnt = NO_OF_SG_PER_BLOCK; > prev_sg_block = sg_block; Reviewed-by: Ewan D. Milne <emilne@xxxxxxxxxx>