From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 37C33CD128A for ; Mon, 8 Apr 2024 02:19:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:References:MIME-Version:Message-ID:Date :Subject:In-Reply-To:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=X5vhpFA3boDporG+BmfH5+5UguRAwZk2Jy7w2p2wVGA=; b=dhoPlN4/vkXrKF g92K9P/5T9H2toIZZqZ+owz8vCyMKRZUpRlQYX4h8/xXyVJnsoEy6Rx7mPr+P+a10iIumnzisFLaf Tv5gtXxwcRgT2lLysi5/b3Id7kvw7Y4YcficuXd6JK+09L9vobEZhoPh6DYj+c2o5ScW05EReo7cn 3/3GRCUP06GrmqPmfOMRlFOtphsYudCHH8baj4ovrgMljlLmplVYI3riChm3W9qAvkV6X3yGraN2a AZjv3UWXTDI7iQ+pXPunKg8gUtvCv3gYxRxNiz+iGhGdonfwUvtKUGIIf77nO8TRkJ9fRNJcCsqLp UrvM0vMDMz1wgzM4V8vQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rtebp-0000000E1hK-1TZl; Mon, 08 Apr 2024 02:19:45 +0000 Received: from mailout4.samsung.com ([203.254.224.34]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rtebl-0000000E1fk-0hBG for linux-arm-kernel@lists.infradead.org; Mon, 08 Apr 2024 02:19:44 +0000 Received: from epcas1p2.samsung.com (unknown [182.195.41.46]) by mailout4.samsung.com (KnoxPortal) with ESMTP id 20240408021926epoutp0412217fc9322238f884bcd0b06c466cb5~ELJKtRyiN0117001170epoutp04B for ; Mon, 8 Apr 2024 02:19:26 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 mailout4.samsung.com 20240408021926epoutp0412217fc9322238f884bcd0b06c466cb5~ELJKtRyiN0117001170epoutp04B DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=samsung.com; s=mail20170921; t=1712542766; bh=6z21GiTFcgH933n4iaXSqNPN9+/JmUWP0CRuM5wSe9c=; h=From:To:Cc:In-Reply-To:Subject:Date:References:From; b=rbbl71AqGrywr5TbEiaBrvE6FOVSLoPAVgi0TGJe1ELM9zvE5slhOx5u8Dktx2qiV SFUOWNaIz0o7PC5sPm6PMfthns3xdRQZ31QiB/GWGKqxTb6luCoEDBOq048hGeh1O9 DNiljbtMNio06h+QR8wrISM+CADcnIWR7sCPproc= Received: from epsnrtp3.localdomain (unknown [182.195.42.164]) by epcas1p4.samsung.com (KnoxPortal) with ESMTP id 20240408021926epcas1p4cb7652f8e2c646c500a059f98a143758~ELJKTEewJ2103021030epcas1p4r; Mon, 8 Apr 2024 02:19:26 +0000 (GMT) Received: from epsmges1p4.samsung.com (unknown [182.195.36.227]) by epsnrtp3.localdomain (Postfix) with ESMTP id 4VCXpk1pCCz4x9QG; Mon, 8 Apr 2024 02:19:26 +0000 (GMT) Received: from epcas1p4.samsung.com ( [182.195.41.48]) by epsmges1p4.samsung.com (Symantec Messaging Gateway) with SMTP id A7.9A.10158.E2453166; Mon, 8 Apr 2024 11:19:26 +0900 (KST) Received: from epsmtrp2.samsung.com (unknown [182.195.40.14]) by epcas1p3.samsung.com (KnoxPortal) with ESMTPA id 20240408021925epcas1p3b607cf70b5fee481edec970e682dba1b~ELJJ48FL72956129561epcas1p3L; Mon, 8 Apr 2024 02:19:25 +0000 (GMT) Received: from epsmgmc1p1new.samsung.com (unknown [182.195.42.40]) by epsmtrp2.samsung.com (KnoxPortal) with ESMTP id 20240408021925epsmtrp27b9fa9239313ef81b229d5cbb5d7218e~ELJJ4A8XB2042120421epsmtrp2I; Mon, 8 Apr 2024 02:19:25 +0000 (GMT) X-AuditID: b6c32a38-b41fa700000027ae-96-6613542e2356 Received: from epsmtip1.samsung.com ( [182.195.34.30]) by epsmgmc1p1new.samsung.com (Symantec Messaging Gateway) with SMTP id 2A.A6.07541.D2453166; Mon, 8 Apr 2024 11:19:25 +0900 (KST) Received: from sgsupark03 (unknown [10.252.69.53]) by epsmtip1.samsung.com (KnoxPortal) with ESMTPA id 20240408021925epsmtip1a75db1e7770f66e247a3afd9b0e36ce2~ELJJs2DM-2544725447epsmtip1b; Mon, 8 Apr 2024 02:19:25 +0000 (GMT) From: "Seongsu Park" To: "'Mark Rutland'" Cc: , , , , , "'Leem ChaeHoon'" , "'Gyeonggeon Choi'" , "'Soomin Cho'" , "'DaeRo Lee'" , "'kmasta'" In-Reply-To: Subject: RE: [PATCH v2] arm64: Fix double TCR_T0SZ_OFFSET shift Date: Mon, 8 Apr 2024 11:19:25 +0900 Message-ID: <03fc01da895b$2d352da0$879f88e0$@samsung.com> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQEDioMuMfHdTugzmv/rxcIOwvfVJwJDhwiMArk0WuCy47pJUA== Content-Language: ko X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrNJsWRmVeSWpSXmKPExsWy7bCmga5eiHCawdq3XBY/v7xntHi/rIfR YlPbfUaLPxt3M1l8+buU2WLT42usFpd3zWGzWHr9IpPFpB9bGC3aP79gtWi5Y+rA7bFm3hpG j52z7rJ7bFrVyeaxeUm9x6GzC1g9Pm+SC2CLyrbJSE1MSS1SSM1Lzk/JzEu3VfIOjneONzUz MNQ1tLQwV1LIS8xNtVVy8QnQdcvMAbpOSaEsMacUKBSQWFyspG9nU5RfWpKqkJFfXGKrlFqQ klNgVqBXnJhbXJqXrpeXWmJlaGBgZApUmJCdMW1uecEUwYof91+xNTA+4e1i5OSQEDCRWPXk ESuILSSwg1Gi8T9PFyMXkP2JUaLh8CRWCOcbo8SHw1cYYTo+nrzCBJHYyygxYdtEFgjnOZDz /xZYFZuAjsS3b1NYQGwRAW2JP1N7wOLMAveYJBadzgKxOYFqHh7tYgexhQXsJdYcm8oEYrMI qEg8XziFGcTmFbCUmHb5M5QtKHFy5hMWiDnyEtvfzmGGuEhB4ufTZawQu5wkPp+8xgRRIyIx u7ONGeQ4CYE9HBIfb/yEesFFYumsqVDNwhKvjm9hh7ClJD6/28sGYRdL7PuyBqqmRuLBvD1Q tr1Ex7M2oCM4gBZoSqzfpQ+xi0/i3dceVpCwhACvREebEISpLHFuqyOEKSnxZ4EexAwPiRW3 tjBOYFScheSvWUj+moXk/lkIqxYwsqxiFEstKM5NTy02LDCBx3Ryfu4mRnCq1bLYwTj37Qe9 Q4xMHIyHGCU4mJVEeINNBdOEeFMSK6tSi/Lji0pzUosPMZoCQ3ois5Rocj4w2eeVxBuaWBqY mBmZWBhbGpspifOeuVKWKiSQnliSmp2aWpBaBNPHxMEp1cA0nbupQFj6glQT06arWV9VJ/TY 32z65b0z0+bI5jTVs7KvZfTT/b5smrH1jvy3Lj3Tf2uF0/WNE1J+Tyt8va/krYWahv3xzkcv m6f5XJ73+mrw6Zt1Exbe0PS0SlLzEAl/ZiI467nr1QOaK/bJM59SMz/pYCbIYLM1K5p5Ykbr 6rau4E0zvjJ0VzxOEw/j1C74MWvGkasTL1XVnSo7arPm/KKHRz+w1HPaFe0QP6HKGCossyTQ Unei/YlpC3kEZ64Tyn1z23XBtu+hSmGT1nuWB1S9SSr4FLf+veRW2TfvQrpeNKcWXrgSOy/T WX6hIE/m4cs37ju/KG8QPMuXdtDn7Uujtk+rD4h+W6B/fXmmEktxRqKhFnNRcSIAZecHKj4E AAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFprBIsWRmVeSWpSXmKPExsWy7bCSnK5uiHCawYwpRhY/v7xntHi/rIfR YlPbfUaLPxt3M1l8+buU2WLT42usFpd3zWGzWHr9IpPFpB9bGC3aP79gtWi5Y+rA7bFm3hpG j52z7rJ7bFrVyeaxeUm9x6GzC1g9Pm+SC2CL4rJJSc3JLEst0rdL4MqYNre8YIpgxY/7r9ga GJ/wdjFyckgImEh8PHmFqYuRi0NIYDejxKoPHYwQCUmJ9neXWboYOYBsYYnDh4shap4ySmz+ fYsdpIZNQEfi27cpLCC2iIC2xJ+pPYwgRcwCL5gkOpouQU1dyyixY+kiJpAqTqCOh0e7wLqF Bewl1hybChZnEVCReL5wCjOIzStgKTHt8mcoW1Di5MwnYFcwC+hJtG0EO45ZQF5i+9s5zBCH Kkj8fLqMFeIIJ4nPJ68xQdSISMzubGOewCg8C8mkWQiTZiGZNAtJxwJGllWMkqkFxbnpucmG BYZ5qeV6xYm5xaV56XrJ+bmbGMGRp6Wxg/He/H96hxiZOBgPMUpwMCuJ8AabCqYJ8aYkVlal FuXHF5XmpBYfYpTmYFES5zWcMTtFSCA9sSQ1OzW1ILUIJsvEwSnVwOTDdnerOl9qj9M1VrGE 2yubWIRsV8yw8FTxXS23Jqt462/N4Kwkll8pa8WmJO5l2hK9o4M7S2/zkrLPpTOW2uypzJzF Krno7Pz/cQfSrz2b/XfDe7nZHWVJ13cVzbAz+5D0wV/50b1FizKuT5w3wcrs2qnfL6ruFXKv 0pdZdKbpQsfuj6FLHZ6fNtoSatmRzrAusq6Bqb7kaTzj56oV94XmhEWyPq9oqtSRymusO3u5 cIr33KX5j5V0799NPJEkP/GftP2kkk2ccxki5j7b42uYHiH385/e09Ozbh+L0GNLaHj2/UB2 ZfU9ffl7HVOm1yUtbHM3Ci0VDhDimpCfJpcULfUoetPkf1+C9nOsXq7EUpyRaKjFXFScCAD0 ewUUKwMAAA== X-CMS-MailID: 20240408021925epcas1p3b607cf70b5fee481edec970e682dba1b X-Msg-Generator: CA CMS-TYPE: 101P DLP-Filter: Pass X-CFilter-Loop: Reflected X-CMS-RootMailID: 20240403024240epcas1p155cd55ca8c1dae6114b970cf3f57084e References: <20240403024236.193428-1-sgsu.park@samsung.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240407_191941_947786_1A82A622 X-CRM114-Status: GOOD ( 29.99 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org > On Wed, Apr 03, 2024 at 11:42:36AM +0900, Seongsu Park wrote: > > We have already shifted the value of t0sz in TCR_T0SZ by TCR_T0SZ_OFFSET. > > So, the TCR_T0SZ_OFFSET shift here should be removed. > > Can we please write a better commit message? > > This doesn't explain: > > * Where we have already shifted the value of t0sz, nor why it makes sense > to do > that there. > > * That the value of TCR_T0SZ_OFFSET is 0, and hence shifting this > repeatedly is > beningn, and this patch is a cleanup rather than a fix. > > Mark. Thank you for feedback. I'll send v3 patch. In v3, We will upgrade the commit message, and add the cpu_set_tcr_t0sz macro. Please check v3! > > > Co-developed-by: Leem ChaeHoon > > Signed-off-by: Leem ChaeHoon > > Co-developed-by: Gyeonggeon Choi > > Signed-off-by: Gyeonggeon Choi > > Co-developed-by: Soomin Cho > > Signed-off-by: Soomin Cho > > Co-developed-by: DaeRo Lee > > Signed-off-by: DaeRo Lee > > Co-developed-by: kmasta > > Signed-off-by: kmasta > > Signed-off-by: Seongsu Park > > --- > > > > Changes in v2: > > - Condition is updated > > > > --- > > arch/arm64/include/asm/mmu_context.h | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/arch/arm64/include/asm/mmu_context.h > > b/arch/arm64/include/asm/mmu_context.h > > index c768d16b81a4..bd19f4c758b7 100644 > > --- a/arch/arm64/include/asm/mmu_context.h > > +++ b/arch/arm64/include/asm/mmu_context.h > > @@ -72,11 +72,11 @@ static inline void __cpu_set_tcr_t0sz(unsigned > > long t0sz) { > > unsigned long tcr = read_sysreg(tcr_el1); > > > > - if ((tcr & TCR_T0SZ_MASK) >> TCR_T0SZ_OFFSET == t0sz) > > + if ((tcr & TCR_T0SZ_MASK) == t0sz) > > return; > > > > tcr &= ~TCR_T0SZ_MASK; > > - tcr |= t0sz << TCR_T0SZ_OFFSET; > > + tcr |= t0sz; > > write_sysreg(tcr, tcr_el1); > > isb(); > > } > > -- > > 2.34.1 > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel