On Sun, Jun 21, 2020 at 12:35:13PM +0000, Avri Altman wrote: > +Alim & Asutosh > > Hi Satya, > > > > > Avri, > > > > > This patch series adds support for inline encryption to UFS using > > > the inline encryption support in the block layer. It follows the JEDEC > > > UFSHCI v2.1 specification, which defines inline encryption for UFS. > > > > I'd appreciate it if you could review this series. > > > > Thanks! > > > > -- > > Martin K. Petersen Oracle Linux Engineering > A quick question and a comment: > > Does the IE infrastructure that you've added to the block layer invented for ufs? It wasn't invented "just" for UFS, but it was guided by how UFS inline crypto works. It is meant to be generic, and should be how we support any arbitrary IE hardware in the kernel. > Do you see other devices using it in the future? > Certainly - at the very least, I think it's likely we'll add support for eMMC inline crypto (although inline encryption support for eMMC was only added in an unreleased/yet to be released spec). It's certainly my hope that we add IE support for other classes of devices too. > Today, chipset vendors are using a different scheme for their IE. > Need their ack before reviewing your patches. > Sure :). The Inline Encryption patches (the block layer, fscrypt, f2fs, ext4 and these UFS patches) have been part of the Android common kernel https://android.googlesource.com/kernel/common/ for quite a while now, and chipset vendors have been working with us on ensuring that their UFS cards are supported by these patches (by adding code similar to Eric's RFC: Inline crypto support on DragonBoard 845c at https://www.spinics.net/lists/linux-scsi/msg141472.html or Mediatek's "scsi: ufs-mediatek: add inline encryption support" at https://lkml.kernel.org/linux-scsi/20200304022101.14165-1-stanley.chu@xxxxxxxxxxxx/ for their individual chipsets). > Thanks, > Avri
