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