linux-kernel - Re: [PATCH 1/1] scsi: ufs: remove redundant HPB unmap

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]

Date:   Wed, 13 Apr 2022 14:43:38 +0200
From:   Bean Huo <huobean@...il.com>
To:     Peter Wang <peter.wang@...iatek.com>,
        Po-Wen Kao <powen.kao@...iatek.com>,
        Alim Akhtar <alim.akhtar@...sung.com>,
        Avri Altman <avri.altman@....com>,
        "James E.J. Bottomley" <jejb@...ux.ibm.com>,
        "Martin K. Petersen" <martin.petersen@...cle.com>,
        Matthias Brugger <matthias.bgg@...il.com>
Cc:     wsd_upstream@...iatek.com, stanley.chu@...iatek.com,
        alice.chao@...iatek.com, chun-hung.wu@...iatek.com,
        cc.chou@...iatek.com, chaotian.jing@...iatek.com,
        jiajie.hao@...iatek.com, yohan.joung@...com, jason.li@...com,
        linux-scsi@...r.kernel.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org,
        linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH 1/1] scsi: ufs: remove redundant HPB unmap

Hi Peter,

On Wed, 2022-04-13 at 11:53 +0800, Peter Wang wrote:
> Hi Bean,
> 
> How about if reset flow not clear hpb mapping?

UFS will go through normal read, because regions are not active in the
device. The responses mabye different among different vendors. for
example, UFS device will raise a response to host to update the region
(in case of host mode) or inactivate it in case of device mode.

The best way is that inactivate the active regions in the host side in
the case the regions are not active in the device.

> 
> After device is reset by RST_N pin, if the hpb mapping table still
> exist?

no, the regions in the device will be inactivated.

> 
> BTW, Host seem not clear region mapping table in reset flow.
> 

Yes, you are right. There will be a performance impact. I will look at
it, then back to you.

By the way, there is one issue in my patch, ignore that patch.

Kind regards,
Bean