RE: [EXT] RE: [PATCH 0/1] Revert "scsi: ufs: Let the SCSI core allocate per-command UFS data"

From: Bean Huo (beanhuo)
Date: Wed Mar 11 2020 - 08:53:11 EST


Hi, Avri
Do you want to revert all or two (scsi: ufs: Let the SCSI core allocate per-command UFS data, scsi: core: Introduce {init,exit}_cmd_priv() )?
Because the patches "scsi: ufs: Simplify two tests" and "scsi: ufs: Introduce ufshcd_init_lrb()" are ok to me.
No problem keeping them. Just this one "scsi: core: Introduce {init,exit}_cmd_priv()" is not necessary, since no drivers it now.

//Bean


>
> Hi,
>
> >
> > From: Bean Huo <beanhuo@xxxxxxxxxx>
> >
> > Hi, Martin
> >
> > Based on Bart's feedack, the less risky way is to revert commit:
> >
> > 34656dda81ac "scsi: ufs: Let the SCSI core allocate per-command UFS data"
> >
> > Bean Huo (1):
> > Revert "scsi: ufs: Let the SCSI core allocate per-command UFS data"
> Maybe it's safer to revert the entire series?
>
> Thanks,
> Avri
>
>
> >
> > drivers/scsi/ufs/ufshcd.c | 198 ++++++++++++++------------------------
> > drivers/scsi/ufs/ufshcd.h | 5 +
> > 2 files changed, 75 insertions(+), 128 deletions(-)
> >
> > --
> > 2.17.1