On Sun, 2022-04-03 at 14:54 -0700, Bart Van Assche wrote: > On 4/3/22 14:36, Bean Huo wrote: > > Yes, I reviewed the entire series of patches and there are no > > significant structural changes. Still want to squeeze ufs driver > > under > > driver/scsi/ufs/. No major conflict with pending submissions. Go > > ahead. > > Hi Bean, > > It has been suggested to move the UFS driver code into the following > two > directories to keep paths short: > * drivers/ufs/core > * drivers/ufs/host > Bart, Very interested in this design. I'm assuming you're still going to continue parsing SCSI commands. Can we also shorten the UFS command path? Meaning we convert block requests directly to UFS UPIU commands? instead of like the current one: block request -> CDB -> UPIU. > That approach is similar to the approach followed for the MMC and > NVMe > drivers. > > Additionally, some other SCSI drivers already occur outside the > drivers/scsi directory, e.g. the drivers/infiniband/ulp/srp/ib_srp.c > driver. I'll take a look at its design, it seems to give us hope that we can make the UFS stack more vertical. Kind regards, Bean > > Thanks, > > Bart.
