RE: [PATCH v8 0/4] scsi: ufs: Add Host Performance Booster Support

[Alim Akhtar]

> -----Original Message-----
> From: Avri Altman <Avri.Altman@xxxxxxx>
> Sent: 06 August 2020 19:27
> To: Bean Huo <huobean@xxxxxxxxx>; daejun7.park@xxxxxxxxxxx;
> jejb@xxxxxxxxxxxxx; martin.petersen@xxxxxxxxxx; asutoshd@xxxxxxxxxxxxxx;
> beanhuo@xxxxxxxxxx; stanley.chu@xxxxxxxxxxxx; cang@xxxxxxxxxxxxxx;
> bvanassche@xxxxxxx; tomas.winkler@xxxxxxxxx; ALIM AKHTAR
> <alim.akhtar@xxxxxxxxxxx>
> Cc: linux-scsi@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Sang-yoon Oh
> <sangyoon.oh@xxxxxxxxxxx>; Sung-Jun Park
> <sungjun07.park@xxxxxxxxxxx>; yongmyung lee
> <ymhungry.lee@xxxxxxxxxxx>; Jinyoung CHOI <j-young.choi@xxxxxxxxxxx>;
> Adel Choi <adel.choi@xxxxxxxxxxx>; BoRam Shin
> <boram.shin@xxxxxxxxxxx>
> Subject: RE: [PATCH v8 0/4] scsi: ufs: Add Host Performance Booster Support
> 
> 
> >
> > On Thu, 2020-08-06 at 10:12 +0000, Avri Altman wrote:
> > > > >
> > > >
> > > > we didn't see you Acked-by in the pathwork, would you like to add
> > > > them?
> > > > Just for reminding us that you have agreed to mainline this series
> > > > patchset.
> > >
> > > I acked it -
> > > https://protect2.fireeye.com/url?k=039c5a1c-5e48e674-039dd153-0cc47a
> > > 3356b2-
> 66867eb5b9700b6a&q=1&u=https%3A%2F%2Fwww.spinics.net%2Flists%
> > > 2Flinux-scsi%2Fmsg144660.html
> > > And asked Martin to move forward -
> > > https://protect2.fireeye.com/url?k=94dceb38-c9085750-94dd6077-0cc47a
> > > 3356b2-
> 19ab1f41f48ff179&q=1&u=https%3A%2F%2Fwww.spinics.net%2Flists%
> > > 2Flinux-scsi%2Fmsg144738.html Which he did, and got some sparse
> > > errors:
> > > https://protect2.fireeye.com/url?k=a40e2dd1-f9da91b9-a40fa69e-0cc47a
> > > 3356b2-
> 81fae05297aebb0e&q=1&u=https%3A%2F%2Fwww.spinics.net%2Flists%
> > > 2Flinux-scsi%2Fmsg144977.html
> > > Which I asked Daejun to fix -
> > > https://protect2.fireeye.com/url?k=6badf100-36794d68-6bac7a4f-0cc47a
> > > 3356b2-
> f84580e236611583&q=1&u=https%3A%2F%2Fwww.spinics.net%2Flists%
> > > 2Flinux-scsi%2Fmsg144987.html
> > >
> > > For the next chain of events I guess you can follow by yourself.
> > >
> > > Thanks,
> > > Avri
> >
> > Avri
> > Sorry for making you confusing. yes, I knew that, and following.
> > I mean Acked-by tag in the patchset, then we see your acked in the
> > patchwork, and let others know that you acked it, rather than going
> > backtrack history email.
> >
> > Hi Daejun
> > I think you can add Avri's Acked-by tag in your patchset, just for
> > quickly moving forward and reminding.
> Ahhh - One moment please -
> While rebasing the v8 on my platform, I noticed some substantial changes since
> v6.
> e.g. the hpb lun ref counting isn't there anymore, as well as some more stuff.
> While those changes might be only for the best,  I think any tested-by tag should
> be re-assign.
> 
> Anyway, as for myself, I am not planning to put any more time in this, until there
> is a clear decision where this series is going to.
> 
> Martin - Are you considering to merge the HPB feature eventually to mainline
> kernel?
> 
V8 has removed the "UFS feature layer" which was  the main topic of discussion. What else we thing is blocking this to be in mainline?
Bart / Martin, any thought?


> Thanks,
> Avri
> >
> > thanks,
> > Bean