linux-kernel - RE: [PATCH v1] scsi: ufs: modify Tactive time setting conditions

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] [day] [month] [year] [list]

Message-ID: <02b301d803a0$39a31770$ace94650$@samsung.com>
Date:   Fri, 7 Jan 2022 13:56:07 +0530
From:   "Alim Akhtar" <alim.akhtar@...sung.com>
To:     "'SEO HOYOUNG'" <hy50.seo@...sung.com>,
        <linux-scsi@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        <kwmad.kim@...sung.com>, <avri.altman@....com>,
        <jejb@...ux.ibm.com>, <martin.petersen@...cle.com>,
        <beanhuo@...ron.com>, <asutoshd@...eaurora.org>,
        <cang@...eaurora.org>, <bvanassche@....org>,
        <grant.jung@...sung.com>, <junwoo80.lee@...sung.com>,
        <bhoon95.kim@...sung.com>
Subject: RE: [PATCH v1] scsi: ufs: modify Tactive time setting conditions



>-----Original Message-----
>From: SEO HOYOUNG [mailto:hy50.seo@...sung.com]
>Sent: Friday, January 7, 2022 1:18 AM
>To: linux-scsi@...r.kernel.org; linux-kernel@...r.kernel.org;
>kwmad.kim@...sung.com; alim.akhtar@...sung.com;
>avri.altman@....com; jejb@...ux.ibm.com; martin.petersen@...cle.com;
>beanhuo@...ron.com; asutoshd@...eaurora.org; cang@...eaurora.org;
>bvanassche@....org; grant.jung@...sung.com;
>junwoo80.lee@...sung.com; bhoon95.kim@...sung.com
>Cc: SEO HOYOUNG <hy50.seo@...sung.com>
>Subject: [PATCH v1] scsi: ufs: modify Tactive time setting conditions
>
>The Tactive time determine the waiting time before burst at hibern8 exit and
>is determined by H/W at linkup state However, in the case of samsung
>devices, guided host's Tactive time +100us for stability.
>If the HCI's Tactive time is equal or greater than the device, +100us should be
>set.
>
>Change-Id: Ie9b502c69264d3e5931ee8b515388094441fa065

Please remove this and advisable to run checkpatch before posting.

>Signed-off-by: SEO HOYOUNG <hy50.seo@...sung.com>
>---
> drivers/scsi/ufs/ufshcd.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
>diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c index
>1049e41abd5b..460d2b440d2e 100644
>--- a/drivers/scsi/ufs/ufshcd.c
>+++ b/drivers/scsi/ufs/ufshcd.c
>@@ -7815,7 +7815,7 @@ static int
>ufshcd_quirk_tune_host_pa_tactivate(struct ufs_hba *hba)
> 	peer_pa_tactivate_us = peer_pa_tactivate *
> 			     gran_to_us_table[peer_granularity - 1];
>
>-	if (pa_tactivate_us > peer_pa_tactivate_us) {
>+	if (pa_tactivate_us >= peer_pa_tactivate_us) {
> 		u32 new_peer_pa_tactivate;
>
> 		new_peer_pa_tactivate = pa_tactivate_us /
>--
>2.26.0