> -----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