linux-kernel - Re: [PATCH v11 0/4] scsi: ufs: Add Host Performance Booster Support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <231786897.01600303381578.JavaMail.epsvc@epcpadp2>
Date:   Thu, 17 Sep 2020 09:25:50 +0900
From:   Daejun Park <daejun7.park@...sung.com>
To:     Christoph Hellwig <hch@...radead.org>,
        Daejun Park <daejun7.park@...sung.com>
CC:     "avri.altman@....com" <avri.altman@....com>,
        "jejb@...ux.ibm.com" <jejb@...ux.ibm.com>,
        "martin.petersen@...cle.com" <martin.petersen@...cle.com>,
        "asutoshd@...eaurora.org" <asutoshd@...eaurora.org>,
        "beanhuo@...ron.com" <beanhuo@...ron.com>,
        "stanley.chu@...iatek.com" <stanley.chu@...iatek.com>,
        "cang@...eaurora.org" <cang@...eaurora.org>,
        "bvanassche@....org" <bvanassche@....org>,
        "tomas.winkler@...el.com" <tomas.winkler@...el.com>,
        ALIM AKHTAR <alim.akhtar@...sung.com>,
        "linux-scsi@...r.kernel.org" <linux-scsi@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Sang-yoon Oh <sangyoon.oh@...sung.com>,
        Sung-Jun Park <sungjun07.park@...sung.com>,
        yongmyung lee <ymhungry.lee@...sung.com>,
        Jinyoung CHOI <j-young.choi@...sung.com>,
        Adel Choi <adel.choi@...sung.com>,
        BoRam Shin <boram.shin@...sung.com>,
        SEUNGUK SHIN <seunguk.shin@...sung.com>
Subject: Re: [PATCH v11 0/4] scsi: ufs: Add Host Performance Booster Support

> > 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