Messages in this thread | | | From | "Bean Huo (beanhuo)" <> | Subject | RE: [EXT] [PATCH v6 6/8] scsi: ufs: add LU Dedicated buffer mode support for WriteBooster | Date | Mon, 4 May 2020 19:11:04 +0000 |
| |
> From: Stanley Chu <stanley.chu@mediatek.com> > Sent: Monday, May 4, 2020 4:56 PM > To: linux-scsi@vger.kernel.org; martin.petersen@oracle.com; > avri.altman@wdc.com; alim.akhtar@samsung.com; jejb@linux.ibm.com; > asutoshd@codeaurora.org > Cc: Bean Huo (beanhuo) <beanhuo@micron.com>; cang@codeaurora.org; > matthias.bgg@gmail.com; bvanassche@acm.org; linux- > mediatek@lists.infradead.org; linux-arm-kernel@lists.infradead.org; linux- > kernel@vger.kernel.org; kuohong.wang@mediatek.com; > peter.wang@mediatek.com; chun-hung.wu@mediatek.com; > andy.teng@mediatek.com; Stanley Chu <stanley.chu@mediatek.com> > Subject: [EXT] [PATCH v6 6/8] scsi: ufs: add LU Dedicated buffer mode support > for WriteBooster > > According to UFS specification, there are two WriteBooster mode of > operations: "LU dedicated buffer" mode and "shared buffer" mode. > In the "LU dedicated buffer" mode, the WriteBooster Buffer is dedicated to a > logical unit. > > If the device supports the "LU dedicated buffer" mode, this mode is configured > by setting bWriteBoosterBufferType to 00h. The logical unit WriteBooster Buffer > size is configured by setting the dLUNumWriteBoosterBufferAllocUnits field of > the related Unit Descriptor. Only a value greater than zero enables the > WriteBooster feature in the logical unit. > > Modify ufshcd_wb_probe() as above description to support LU Dedicated buffer > mode. > > Note that according to UFS 3.1 specification, the valid value of > bDeviceMaxWriteBoosterLUs parameter in Geometry Descriptor is 1, which > means at most one LUN can have WriteBooster buffer in "LU dedicated buffer > mode". Therefore this patch supports only one LUN with WriteBooster enabled. > All WriteBooster related sysfs nodes are specifically mapped to the LUN with > WriteBooster enabled in LU Dedicated buffer mode. > > Signed-off-by: Stanley Chu <stanley.chu@mediatek.com> > Reviewed-by: Avri Altman <avri.altman@wdc.com> Reviewed-by: Bean Huo <beanhuo@micron.com>
| |