RE: [PATCH v4 00/16] introduce exynosauto v9 ufs driver

["Chanho Park" <chanho61.park@samsung.com>]

Hi, Thanks for your review.

> -----Original Message-----
> From: Avri Altman <Avri.Altman@wdc.com>
> Sent: Thursday, October 14, 2021 7:05 PM
> To: Chanho Park <chanho61.park@samsung.com>; Alim Akhtar
> <alim.akhtar@samsung.com>; James E . J . Bottomley <jejb@linux.ibm.com>;
> Martin K . Petersen <martin.petersen@oracle.com>; Krzysztof Kozlowski
> <krzysztof.kozlowski@canonical.com>
> Cc: Bean Huo <beanhuo@micron.com>; Bart Van Assche <bvanassche@acm.org>;
> Adrian Hunter <adrian.hunter@intel.com>; hch@infradead.org; Can Guo
> <cang@codeaurora.org>; Jaegeuk Kim <jaegeuk@kernel.org>; Jaehoon Chung
> <jh80.chung@samsung.com>; Gyunghoon Kwon <goodjob.kwon@samsung.com>; Sowon
> Na <sowon.na@samsung.com>; linux-samsung-soc@vger.kernel.org; linux-
> scsi@vger.kernel.org
> Subject: RE: [PATCH v4 00/16] introduce exynosauto v9 ufs driver
> 
> Hi,
> >
> > In ExynosAuto(variant of the Exynos for automotive), the UFS Storage
> > needs to be accessed from multi-OS. To increase IO performance and
> > reduce SW complexity, we implemented UFS-IOV to support storage IO
> > virtualization feature on UFS.
> >
> > IO virtualization increases IO performance and reduce SW complexity
> > with small area cost. And IO virtualization supports virtual machine
> > isolation for Security and Safety which are requested by Multi-OS
> > system such as automotive application.
> >
> > Below figure is the conception of UFS-IOV architeture.
> Conception --> a conceptual design
> 
> >
> >     +------+          +------+
> >     | OS#1 |          | OS#2 |
> >     +------+          +------+
> >        |                 |
> >  +------------+     +------------+
> >  |  Physical  |     |   Virtual  |
> >  |    Host    |     |    Host    |
> >  +------------+     +------------+
> >    |      |              | <-- UTP_CMD_SAP, UTP_TM_SAP
> >    |   +-------------------------+
> >    |   |    Function Arbitor     |
> >    |   +-------------------------+
> >  +-------------------------------+
> >  |           UTP Layer           |
> >  +-------------------------------+
> >  +-------------------------------+
> >  |           UIC Layer           |
> >  +-------------------------------+
> >
> > There are two types of host controllers on the UFS host controller
> > that we designed.
> > The UFS device has a Function Arbitor that arranges commands of each
> host.
> > When each host transmits a command to the Arbitor, the Arbitor
> > transmits it to the UTP layer.
> - Arbitor --> arbiter
>  - ufs devise --> host controller
> And maybe rephrase the above description (and sketch) so it is clear that
> the PH, VH, and function arbiter are all hw modules in the host controller.

I'll rephrase the descriptions next patchset. Thanks.

Best Regards,
Chanho Park