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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Thu, 24 Dec 2020 10:21:09 +0800 From: Can Guo <cang@...eaurora.org> To: Stanley Chu <stanley.chu@...iatek.com> Cc: linux-scsi@...r.kernel.org, martin.petersen@...cle.com, avri.altman@....com, alim.akhtar@...sung.com, jejb@...ux.ibm.com, beanhuo@...ron.com, asutoshd@...eaurora.org, matthias.bgg@...il.com, bvanassche@....org, linux-mediatek@...ts.infradead.org, linux-arm-kernel@...ts.infradead.org, linux-kernel@...r.kernel.org, kuohong.wang@...iatek.com, peter.wang@...iatek.com, chun-hung.wu@...iatek.com, andy.teng@...iatek.com, chaotian.jing@...iatek.com, cc.chou@...iatek.com, jiajie.hao@...iatek.com, alice.chao@...iatek.com Subject: Re: [PATCH v1] scsi: ufs-mediatek: Enable UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL On 2020-12-23 12:41, Can Guo wrote: > On 2020-12-23 12:19, Stanley Chu wrote: >> Hi Can, >> >> On Tue, 2020-12-22 at 19:34 +0800, Can Guo wrote: >>> On 2020-12-22 15:29, Stanley Chu wrote: >>> > Flush during hibern8 is sufficient on MediaTek platforms, thus >>> > enable UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL to skip enabling >>> > fWriteBoosterBufferFlush during WriteBooster initialization. >>> > >>> > Signed-off-by: Stanley Chu <stanley.chu@...iatek.com> >>> > --- >>> > drivers/scsi/ufs/ufs-mediatek.c | 1 + >>> > 1 file changed, 1 insertion(+) >>> > >>> > diff --git a/drivers/scsi/ufs/ufs-mediatek.c >>> > b/drivers/scsi/ufs/ufs-mediatek.c >>> > index 80618af7c872..c55202b92a43 100644 >>> > --- a/drivers/scsi/ufs/ufs-mediatek.c >>> > +++ b/drivers/scsi/ufs/ufs-mediatek.c >>> > @@ -661,6 +661,7 @@ static int ufs_mtk_init(struct ufs_hba *hba) >>> > >>> > /* Enable WriteBooster */ >>> > hba->caps |= UFSHCD_CAP_WB_EN; >>> > + hba->quirks |= UFSHCI_QUIRK_SKIP_MANUAL_WB_FLUSH_CTRL; >>> > hba->vps->wb_flush_threshold = UFS_WB_BUF_REMAIN_PERCENT(80); >>> > >>> > if (host->caps & UFS_MTK_CAP_DISABLE_AH8) >>> >>> I guess we need it too... >> >> AHHA, if you decide to add this in your platform too later, maybe we >> could change the way it does: Keep manual flush disabled by default >> and >> remove this quirk. >> > > Yeah... I will get back with an answer later. Hi Stanley, Do you see any substantial benefit of having fWriteBoosterBufferFlushEn disabled? Thanks, Can Guo. > > Thanks, > > Can Guo. > >> Thanks, >> Stanley Chu >>> >>> Change LGTM. >>> >>> Regards, >>> >>> Can Guo.
Powered by blists - more mailing lists