public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Daejun Park <daejun7.park@samsung.com>
To: Christoph Hellwig <hch@infradead.org>,
	Daejun Park <daejun7.park@samsung.com>
Cc: "avri.altman@wdc.com" <avri.altman@wdc.com>,
	"jejb@linux.ibm.com" <jejb@linux.ibm.com>,
	"martin.petersen@oracle.com" <martin.petersen@oracle.com>,
	"asutoshd@codeaurora.org" <asutoshd@codeaurora.org>,
	"beanhuo@micron.com" <beanhuo@micron.com>,
	"stanley.chu@mediatek.com" <stanley.chu@mediatek.com>,
	"cang@codeaurora.org" <cang@codeaurora.org>,
	"bvanassche@acm.org" <bvanassche@acm.org>,
	"tomas.winkler@intel.com" <tomas.winkler@intel.com>,
	ALIM AKHTAR <alim.akhtar@samsung.com>,
	"linux-scsi@vger.kernel.org" <linux-scsi@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Sang-yoon Oh <sangyoon.oh@samsung.com>,
	Sung-Jun Park <sungjun07.park@samsung.com>,
	yongmyung lee <ymhungry.lee@samsung.com>,
	Jinyoung CHOI <j-young.choi@samsung.com>,
	Adel Choi <adel.choi@samsung.com>,
	BoRam Shin <boram.shin@samsung.com>,
	SEUNGUK SHIN <seunguk.shin@samsung.com>
Subject: Re: [PATCH v11 0/4] scsi: ufs: Add Host Performance Booster Support
Date: Thu, 17 Sep 2020 09:25:50 +0900	[thread overview]
Message-ID: <231786897.01600303381578.JavaMail.epsvc@epcpadp2> (raw)
In-Reply-To: <20200916052208.GB12923@infradead.org>

> > Hi All,
> > 
> > I want to know how to improve this patch.
> 
> Drop it and fix the actual UFS feature to not be so horrible?
> 

Hello Christoph,

Thanks for the comment.

The HPB is essential feature for mobile devices.

NAND-based storage needs logical to physical mapping, which is cached in
the storage to minimize translation overhead. UFS is a NAND-based storage
device with limited resources, mainly used for mobile devices. Typically,
SSD has enough cache space for mapping information in SSD, but UFS has not
enough.
So, UFS stores L2P mapping in NAND and performs IO using demand loading 
for translation. Due to overhead of demand loading, it degrades random read
performance.

The HPB is a feature which uses host memory to relieve this problem. By 
using the HPB feature, UFS can provide improved random read performance
without mapping data thrashing problem. Therefore, the HPB is currently
already included in the Linux kernel code in android devices, however it 
is maintained as out-of-tree.

While upstreaming HPB feature to mainline kernel, we received various
comments from several reviewers (thanks!) and the HPB feature can be
improved. I think it would be good to make mainline the feature that are
mainly used.

Thanks,
Daejun


  reply	other threads:[~2020-09-17  0:49 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CGME20200902031713epcms2p664cebf386ba19d3d05895fec89aaf4fe@epcms2p6>
2020-09-02  3:17 ` [PATCH v11 0/4] scsi: ufs: Add Host Performance Booster Support Daejun Park
2020-09-02  3:20   ` [PATCH v11 1/4] scsi: ufs: Add HPB feature related parameters Daejun Park
2020-09-02  3:21   ` [PATCH v11 2/4] scsi: ufs: Introduce HPB feature Daejun Park
2020-09-02  3:21   ` [PATCH v11 3/4] scsi: ufs: L2P map management for HPB read Daejun Park
2020-09-02  3:22   ` [PATCH v11 4/4] scsi: ufs: Prepare HPB read for cached sub-region Daejun Park
2020-09-15 23:05   ` [PATCH v11 0/4] scsi: ufs: Add Host Performance Booster Support Daejun Park
2020-09-16  5:22     ` Christoph Hellwig
2020-09-17  0:25       ` Daejun Park [this message]
2020-09-18 19:51       ` Bart Van Assche
2020-09-21  8:03         ` Daejun Park

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=231786897.01600303381578.JavaMail.epsvc@epcpadp2 \
    --to=daejun7.park@samsung.com \
    --cc=adel.choi@samsung.com \
    --cc=alim.akhtar@samsung.com \
    --cc=asutoshd@codeaurora.org \
    --cc=avri.altman@wdc.com \
    --cc=beanhuo@micron.com \
    --cc=boram.shin@samsung.com \
    --cc=bvanassche@acm.org \
    --cc=cang@codeaurora.org \
    --cc=hch@infradead.org \
    --cc=j-young.choi@samsung.com \
    --cc=jejb@linux.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-scsi@vger.kernel.org \
    --cc=martin.petersen@oracle.com \
    --cc=sangyoon.oh@samsung.com \
    --cc=seunguk.shin@samsung.com \
    --cc=stanley.chu@mediatek.com \
    --cc=sungjun07.park@samsung.com \
    --cc=tomas.winkler@intel.com \
    --cc=ymhungry.lee@samsung.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox