From nobody Sun Nov 24 23:05:51 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass(p=none dis=none) header.from=nongnu.org ARC-Seal: i=1; a=rsa-sha256; t=1720065815; cv=none; d=zohomail.com; s=zohoarc; b=lUpixSZ6JLpjfYzK9RiaGXvgAO9OBjI3YZNs1kOpxZrQ8N7dhYF37PPgSqDNH+XOUBD21rRBN/wh9tNntbrXL+QlLMGQ11AQdkQXO0/qXEguZWC0liwmqTRONs7l6dNApwp6pftr8Oy1b0VwQvv9WI7cV0Zhw1RkwevbbN1rFkg= ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1720065815; h=Content-Type:Content-Transfer-Encoding:Cc:Cc:Date:Date:From:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Archive:List-Help:List-Unsubscribe:MIME-Version:Message-ID:Reply-To:Reply-To:References:Sender:Subject:Subject:To:To:Message-Id; bh=8D84u/vRIBxS+wbJBMPOIVJUhOvD6+DTqzypLThcUPk=; b=OhlWrZ4TnfmfAaTUOsB0hub4MjsxjXKiZhkqcZeroLZ3d6q01wPyDz1ozh06mF2JKLmtGYadbql0/BFstkzmE5/qm8mDCYaDswdpJjd5vK449MYoNwGnQ2y0hAoS1Zjrj37C3bCocmpzWsx/GjipRcz/7N05ISZQyq0+sVaUGCA= ARC-Authentication-Results: i=1; mx.zohomail.com; spf=pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom=qemu-devel-bounces+importer=patchew.org@nongnu.org; dmarc=pass header.from= (p=none dis=none) Return-Path: Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) by mx.zohomail.com with SMTPS id 172006581578398.19634318777912; Wed, 3 Jul 2024 21:03:35 -0700 (PDT) Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1sPDg7-0001fL-92; Thu, 04 Jul 2024 00:02:39 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1sPDg5-0001d1-Ka for qemu-devel@nongnu.org; Thu, 04 Jul 2024 00:02:37 -0400 Received: from 60-248-80-70.hinet-ip.hinet.net ([60.248.80.70] helo=Atcsqr.andestech.com) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1sPDg3-0003UB-Iv for qemu-devel@nongnu.org; Thu, 04 Jul 2024 00:02:37 -0400 Received: from mail.andestech.com (ATCPCS34.andestech.com [10.0.1.134]) by Atcsqr.andestech.com with ESMTPS id 46442KK0005709 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=OK); Thu, 4 Jul 2024 12:02:20 +0800 (+08) (envelope-from alvinga@andestech.com) Received: from atctrx.andestech.com (10.0.15.190) by ATCPCS34.andestech.com (10.0.1.134) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Thu, 4 Jul 2024 12:02:20 +0800 To: , CC: , , , , , Alvin Chang Subject: [PATCH 1/2] target/riscv: Preliminary textra trigger CSR writting support Date: Thu, 4 Jul 2024 12:01:59 +0800 Message-ID: <20240704040200.243892-2-alvinga@andestech.com> X-Mailer: git-send-email 2.34.1 In-Reply-To: <20240704040200.243892-1-alvinga@andestech.com> References: <20240704040200.243892-1-alvinga@andestech.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable X-Originating-IP: [10.0.15.190] X-ClientProxiedBy: ATCPCS33.andestech.com (10.0.1.100) To ATCPCS34.andestech.com (10.0.1.134) X-DNSRBL: X-SPAM-SOURCE-CHECK: pass X-MAIL: Atcsqr.andestech.com 46442KK0005709 Received-SPF: pass (zohomail.com: domain of gnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; envelope-from=qemu-devel-bounces+importer=patchew.org@nongnu.org; helo=lists.gnu.org; Received-SPF: pass client-ip=60.248.80.70; envelope-from=alvinga@andestech.com; helo=Atcsqr.andestech.com X-Spam_score_int: -8 X-Spam_score: -0.9 X-Spam_bar: / X-Spam_report: (-0.9 / 5.0 requ) BAYES_00=-1.9, RDNS_DYNAMIC=0.982, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, TVD_RCVD_IP=0.001 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: , Reply-to: Alvin Chang From: Alvin Chang via Errors-To: qemu-devel-bounces+importer=patchew.org@nongnu.org Sender: qemu-devel-bounces+importer=patchew.org@nongnu.org X-ZM-MESSAGEID: 1720065819231100009 Content-Type: text/plain; charset="utf-8" This commit allows program to write textra trigger CSR for type 2, 3, 6 triggers. In this preliminary patch, the textra.MHVALUE and the textra.MHSELECT fields are allowed to be configured. Other fields, such as textra.SBYTEMASK, textra.SVALUE, and textra.SSELECT, are hardwired to zero for now. For textra.MHSELECT field, the only legal values are 0 (ignore) and 4 (mcontext). Writing 1~3 into textra.MHSELECT will be changed to 0, and writing 5~7 into textra.MHSELECT will be changed to 4. This behavior is aligned to RISC-V SPIKE simulator. Signed-off-by: Alvin Chang --- target/riscv/cpu_bits.h | 10 +++++ target/riscv/debug.c | 81 ++++++++++++++++++++++++++++++++++++++--- 2 files changed, 85 insertions(+), 6 deletions(-) diff --git a/target/riscv/cpu_bits.h b/target/riscv/cpu_bits.h index c257c5ed7d..0530b4f9f4 100644 --- a/target/riscv/cpu_bits.h +++ b/target/riscv/cpu_bits.h @@ -906,6 +906,16 @@ typedef enum RISCVException { #define JVT_BASE (~0x3F) =20 /* Debug Sdtrig CSR masks */ +#define TEXTRA32_MHVALUE 0xFC000000 +#define TEXTRA32_MHSELECT 0x03800000 +#define TEXTRA32_SBYTEMASK 0x000C0000 +#define TEXTRA32_SVALUE 0x0003FFFC +#define TEXTRA32_SSELECT 0x00000003 +#define TEXTRA64_MHVALUE 0xFFF8000000000000ULL +#define TEXTRA64_MHSELECT 0x0007000000000000ULL +#define TEXTRA64_SBYTEMASK 0x000000F000000000ULL +#define TEXTRA64_SVALUE 0x00000003FFFFFFFCULL +#define TEXTRA64_SSELECT 0x0000000000000003ULL #define MCONTEXT32 0x0000003F #define MCONTEXT64 0x0000000000001FFFULL #define MCONTEXT32_HCONTEXT 0x0000007F diff --git a/target/riscv/debug.c b/target/riscv/debug.c index 0b5099ff9a..f7d8f5e320 100644 --- a/target/riscv/debug.c +++ b/target/riscv/debug.c @@ -217,6 +217,69 @@ static inline void warn_always_zero_bit(target_ulong v= al, target_ulong mask, } } =20 +static target_ulong textra_validate(CPURISCVState *env, target_ulong tdata= 3) +{ + target_ulong mhvalue, mhselect; + target_ulong mhselect_new; + target_ulong textra; + const uint32_t mhselect_no_rvh[8] =3D { 0, 0, 0, 0, 4, 4, 4, 4 }; + + switch (riscv_cpu_mxl(env)) { + case MXL_RV32: + mhvalue =3D get_field(tdata3, TEXTRA32_MHVALUE); + mhselect =3D get_field(tdata3, TEXTRA32_MHSELECT); + /* Validate unimplemented (always zero) bits */ + warn_always_zero_bit(tdata3, (target_ulong)TEXTRA32_SBYTEMASK, + "sbytemask"); + warn_always_zero_bit(tdata3, (target_ulong)TEXTRA32_SVALUE, + "svalue"); + warn_always_zero_bit(tdata3, (target_ulong)TEXTRA32_SSELECT, + "sselect"); + break; + case MXL_RV64: + case MXL_RV128: + mhvalue =3D get_field(tdata3, TEXTRA64_MHVALUE); + mhselect =3D get_field(tdata3, TEXTRA64_MHSELECT); + /* Validate unimplemented (always zero) bits */ + warn_always_zero_bit(tdata3, (target_ulong)TEXTRA64_SBYTEMASK, + "sbytemask"); + warn_always_zero_bit(tdata3, (target_ulong)TEXTRA64_SVALUE, + "svalue"); + warn_always_zero_bit(tdata3, (target_ulong)TEXTRA64_SSELECT, + "sselect"); + break; + default: + g_assert_not_reached(); + } + + /* Validate mhselect. */ + mhselect_new =3D mhselect_no_rvh[mhselect]; + + /* Write legal values into textra */ + textra =3D 0; + switch (riscv_cpu_mxl(env)) { + case MXL_RV32: + textra =3D set_field(textra, TEXTRA32_MHVALUE, mhvalue); + textra =3D set_field(textra, TEXTRA32_MHSELECT, mhselect_new); + break; + case MXL_RV64: + case MXL_RV128: + textra =3D set_field(textra, TEXTRA64_MHVALUE, mhvalue); + textra =3D set_field(textra, TEXTRA64_MHSELECT, mhselect_new); + break; + default: + g_assert_not_reached(); + } + + if (textra !=3D tdata3) { + qemu_log_mask(LOG_GUEST_ERROR, + "different value 0x" TARGET_FMT_lx " write to tdata3= \n", + textra); + } + + return textra; +} + static void do_trigger_action(CPURISCVState *env, target_ulong trigger_ind= ex) { trigger_action_t action =3D get_trigger_action(env, trigger_index); @@ -441,8 +504,10 @@ static void type2_reg_write(CPURISCVState *env, target= _ulong index, } break; case TDATA3: - qemu_log_mask(LOG_UNIMP, - "tdata3 is not supported for type 2 trigger\n"); + new_val =3D textra_validate(env, val); + if (new_val !=3D env->tdata3[index]) { + env->tdata3[index] =3D new_val; + } break; default: g_assert_not_reached(); @@ -558,8 +623,10 @@ static void type6_reg_write(CPURISCVState *env, target= _ulong index, } break; case TDATA3: - qemu_log_mask(LOG_UNIMP, - "tdata3 is not supported for type 6 trigger\n"); + new_val =3D textra_validate(env, val); + if (new_val !=3D env->tdata3[index]) { + env->tdata3[index] =3D new_val; + } break; default: g_assert_not_reached(); @@ -741,8 +808,10 @@ static void itrigger_reg_write(CPURISCVState *env, tar= get_ulong index, "tdata2 is not supported for icount trigger\n"); break; case TDATA3: - qemu_log_mask(LOG_UNIMP, - "tdata3 is not supported for icount trigger\n"); + new_val =3D textra_validate(env, val); + if (new_val !=3D env->tdata3[index]) { + env->tdata3[index] =3D new_val; + } break; default: g_assert_not_reached(); --=20 2.34.1