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 lists.gnu.org (lists.gnu.org [209.51.188.17]) (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 73228ECAAA1 for ; Wed, 31 Aug 2022 01:32:52 +0000 (UTC) Received: from localhost ([::1]:42656 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1oTCb4-0005hK-Sa for qemu-devel@archiver.kernel.org; Tue, 30 Aug 2022 21:32:50 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:55772) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1oTCYn-0004tX-MG for qemu-devel@nongnu.org; Tue, 30 Aug 2022 21:30:30 -0400 Received: from hsmtpd-def.xspmail.jp ([202.238.198.244]:19580) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1oTCYl-0001lS-00 for qemu-devel@nongnu.org; Tue, 30 Aug 2022 21:30:29 -0400 X-Country-Code: JP Received: from sakura.ysato.name (ik1-413-38519.vs.sakura.ne.jp [153.127.30.23]) by hsmtpd-out-1.asahinet.cluster.xspmail.jp (Halon) with ESMTPA id d1b1a525-5392-46b0-9c45-3d27c02e3c6b; Wed, 31 Aug 2022 10:30:18 +0900 (JST) Received: from SIOS1075.ysato.ml (ae222174.dynamic.ppp.asahi-net.or.jp [14.3.222.174]) by sakura.ysato.name (Postfix) with ESMTPSA id A67B81C0075; Wed, 31 Aug 2022 10:30:16 +0900 (JST) Date: Wed, 31 Aug 2022 10:30:15 +0900 Message-ID: <87y1v5xkrc.wl-ysato@users.sourceforge.jp> From: Yoshinori Sato To: Richard Henderson Cc: BALATON Zoltan , qemu-devel@nongnu.org, alex.bennee@linaro.org, qemu-stable@nongnu.org Subject: Re: [PATCH] target/sh4: Fix TB_FLAG_UNALIGN In-Reply-To: References: <20220829021325.154978-1-richard.henderson@linaro.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM-LB/1.14.9 (=?ISO-8859-4?Q?Goj=F2?=) APEL-LB/10.8 EasyPG/1.0.0 Emacs/27.1 (x86_64-pc-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Received-SPF: softfail client-ip=202.238.198.244; envelope-from=ysato@users.sourceforge.jp; helo=hsmtpd-def.xspmail.jp X-Spam_score_int: -11 X-Spam_score: -1.2 X-Spam_bar: - X-Spam_report: (-1.2 / 5.0 requ) BAYES_00=-1.9, SPF_HELO_PASS=-0.001, SPF_SOFTFAIL=0.665, T_SCC_BODY_TEXT_LINE=-0.01 autolearn=no autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On Tue, 30 Aug 2022 01:10:29 +0900, Richard Henderson wrote: >=20 > On 8/29/22 02:05, BALATON Zoltan wrote: > > On Sun, 28 Aug 2022, Richard Henderson wrote: > >> The value previously chosen overlaps GUSA_MASK. > >>=20 > >> Cc: qemu-stable@nongnu.org > >> Fixes: 4da06fb3062 ("target/sh4: Implement prctl_unalign_sigbus") > >> Resolves: https://gitlab.com/qemu-project/qemu/-/issues/856 > >> Signed-off-by: Richard Henderson > >> --- > >> target/sh4/cpu.h | 2 +- > >> 1 file changed, 1 insertion(+), 1 deletion(-) > >>=20 > >> diff --git a/target/sh4/cpu.h b/target/sh4/cpu.h > >> index 9f15ef913c..e79cbc59e2 100644 > >> --- a/target/sh4/cpu.h > >> +++ b/target/sh4/cpu.h > >> @@ -84,7 +84,7 @@ > >> #define DELAY_SLOT_RTE=A0=A0=A0=A0=A0=A0=A0=A0 (1 << 2) > >>=20 > >> #define TB_FLAG_PENDING_MOVCA=A0 (1 << 3) > >> -#define TB_FLAG_UNALIGN=A0=A0=A0=A0=A0=A0=A0 (1 << 4) > >> +#define TB_FLAG_UNALIGN=A0=A0=A0=A0=A0=A0=A0 (1 << 13) > >=20 > > Is it worth a comment to note why that value to avoid the same > > problem if another flag is added in the future? >=20 > Hmm, or perhaps move it down below, so that we see bit 3 used, then bits = 4-12, then bit 13. >=20 >=20 > r~ It looks like the gUSA and unalign access flags are mixed. I think the flags should also be separated as the two features are not rela= ted. --=20 Yosinori Sato