{"id":2222468,"url":"http://patchwork.ozlabs.org/api/patches/2222468/?format=json","web_url":"http://patchwork.ozlabs.org/project/qemu-devel/patch/99be0e0e0d43c77389dd076d079b195844817076.1775959096.git.chao.liu.zevorn@gmail.com/","project":{"id":14,"url":"http://patchwork.ozlabs.org/api/projects/14/?format=json","name":"QEMU Development","link_name":"qemu-devel","list_id":"qemu-devel.nongnu.org","list_email":"qemu-devel@nongnu.org","web_url":"","scm_url":"","webscm_url":"","list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<99be0e0e0d43c77389dd076d079b195844817076.1775959096.git.chao.liu.zevorn@gmail.com>","list_archive_url":null,"date":"2026-04-12T02:20:20","name":"[v6,3/7] target/riscv: add sdext Debug Mode helpers","commit_ref":null,"pull_url":null,"state":"new","archived":false,"hash":"3b962898eb5ace68e340bdec2cecf2523326d48a","submitter":{"id":92265,"url":"http://patchwork.ozlabs.org/api/people/92265/?format=json","name":"Chao Liu","email":"chao.liu.zevorn@gmail.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/qemu-devel/patch/99be0e0e0d43c77389dd076d079b195844817076.1775959096.git.chao.liu.zevorn@gmail.com/mbox/","series":[{"id":499584,"url":"http://patchwork.ozlabs.org/api/series/499584/?format=json","web_url":"http://patchwork.ozlabs.org/project/qemu-devel/list/?series=499584","date":"2026-04-12T02:20:20","name":"riscv: add initial sdext support","version":6,"mbox":"http://patchwork.ozlabs.org/series/499584/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/2222468/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/2222468/checks/","tags":{},"related":[],"headers":{"Return-Path":"<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@legolas.ozlabs.org","Authentication-Results":["legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256\n header.s=20251104 header.b=dRfa+4jy;\n\tdkim-atps=neutral","legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=nongnu.org\n (client-ip=209.51.188.17; helo=lists1p.gnu.org;\n envelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n receiver=patchwork.ozlabs.org)"],"Received":["from lists1p.gnu.org (lists1p.gnu.org [209.51.188.17])\n\t(using TLSv1.2 with cipher ECDHE-ECDSA-AES256-GCM-SHA384 (256/256 bits))\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4ftZ7105Whz1xtJ\n\tfor <incoming@patchwork.ozlabs.org>; Sun, 12 Apr 2026 12:22:03 +1000 (AEST)","from localhost ([::1] helo=lists1p.gnu.org)\n\tby lists1p.gnu.org with esmtp (Exim 4.90_1)\n\t(envelope-from <qemu-devel-bounces@nongnu.org>)\n\tid 1wBkS0-0004Dk-Fc; Sat, 11 Apr 2026 22:21:28 -0400","from eggs.gnu.org ([2001:470:142:3::10])\n by lists1p.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)\n (Exim 4.90_1) (envelope-from <chao.liu.zevorn@gmail.com>)\n id 1wBkRw-0004DL-VE\n for qemu-devel@nongnu.org; Sat, 11 Apr 2026 22:21:26 -0400","from mail-qv1-xf41.google.com ([2607:f8b0:4864:20::f41])\n by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128)\n (Exim 4.90_1) (envelope-from <chao.liu.zevorn@gmail.com>)\n id 1wBkRt-0008GT-L3\n for qemu-devel@nongnu.org; Sat, 11 Apr 2026 22:21:24 -0400","by mail-qv1-xf41.google.com with SMTP id\n 6a1803df08f44-8a08fa355a1so49657916d6.0\n for <qemu-devel@nongnu.org>; Sat, 11 Apr 2026 19:21:21 -0700 (PDT)","from ZEVORN-PC.bbrouter ([162.244.208.119])\n by smtp.gmail.com with ESMTPSA id\n 6a1803df08f44-8aca05592ecsm15455576d6.29.2026.04.11.19.21.16\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Sat, 11 Apr 2026 19:21:20 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=gmail.com; s=20251104; t=1775960481; x=1776565281; darn=nongnu.org;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=7vVCl9oFeB3SQLBVtH9GYT4ioHX7OL/vziaYxhp85Ro=;\n b=dRfa+4jyZqQLHv/o1zI2rEiPh3WseNaM8vouc/eAN3tbPOKfOYTDmm6npw9ENNTIgd\n /RYsQ5pNz3ohs+DKqX1uf7fVnCkSDQORHdnB9H8HEuzsEIpgQS6IaggK24TvCt3ScGsn\n Bb8g3m8Dh6gnxpj70wAhsXm8sP/aIOLfyS8J5NjfAh7PuPPT+9//Mbq+cez52Hnv+xDf\n iRX8LP1krrrx84pJWpQn6lAc/CMgVv4ydiYvMxIuoYRJHeCucYlWOSvAlDBxS/YVqDXf\n t63dCWy3WpgYhfv5tyIHox1x3zUebtahRgB1ccv+C85UG0Uw/mBJ04ArfEdRPGZZ0xLe\n VW3g==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1775960481; x=1776565281;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from\n :to:cc:subject:date:message-id:reply-to;\n bh=7vVCl9oFeB3SQLBVtH9GYT4ioHX7OL/vziaYxhp85Ro=;\n b=ZBfV13zfNhjxWSoot1KbWynQYArktwjxG5cZGPPBDjxnDo5Yh+rbZVf8gMCrQTELos\n D2ZDbE3RzawHpHoN7xRQUSJtQ2yHpoooQrLMr2p4veeBWy3Fjr03nHK+1/Y8WtdOL+mg\n 0g75YUMzTwqMKZAMsxKx301Q/ZKPNbTOvJ0v8iZZys+4gZnUQhL0Y9lPuQLbwknI+P40\n IRPNt7cd8D2ZW1KpiNIyD4xrWy3dVNJE0xZ+KCZu41q7ixK1WFjnBKvH05WZ8BdbDgu3\n wvlPZmmVb3miDuLNsxuKTCuDSqzFcrko5o5D7pCqa5xQd4DLun+YqMAdAsjXTEnpxlwf\n 0KfQ==","X-Gm-Message-State":"AOJu0YwoYdSAFeEVmBFYJPz31hJ8RjalmdEVaqhjg94fQfxYUTKA4bHa\n 2QIiQCj6dfGTDLFkKwfprQpr6diQ3R83KOJqe+vMUc+jc+04GxqmZC0j","X-Gm-Gg":"AeBDieu8LKjtJ4+ZYWtRz0NFv7/Ie1yTeqqkzGpORMweRkQV+TtFpoK87qIIdRmmoO3\n Lm7ngMd2L9KxiBPbubmJYCgPQdBUAwNcevKN5dUzF9UoIL0tpgI/haFTnG0x5PKJsmES1x3yFbP\n vkvpSOGsehuKVoGWOruemaoE3f2gCtNZ6STcvGIrNj77eXj3SskRg7P31V2gmS/kI6Xcxbzce4B\n g4nGRvM5FaVZud8Zw5ncYlWA/ZcVX7rxVJEF0QTlzmyj1iIas/m8Ej0+FzuG55nzOnpy55V9LfH\n q4pV92l7IN0nZFEcIFsVo0FqgtxmXHMsgoe8TuTQ8haPTyMDuzLDDCBBMCJ1tPBQxvN14JscUBC\n 150t965q44/iJr61Vqb+XC82KxY1Crxtm1ksK9HHR82IUykU5B3DxrYIRzE8vSjUKyYFE5bArNz\n D8t6XygpmbjEclyvvB+kA5xbd1UC7Tx1oPDQuY9p0tIyEgA/Fn4CYz0A1/fVRPQfUmYceI","X-Received":"by 2002:a05:6214:19e2:b0:89c:6ba4:e306 with SMTP id\n 6a1803df08f44-8ac862c8eaamr150741326d6.43.1775960480548;\n Sat, 11 Apr 2026 19:21:20 -0700 (PDT)","From":"Chao Liu <chao.liu.zevorn@gmail.com>","To":"Pierrick Bouvier <pierrick.bouvier@linaro.org>,\n Palmer Dabbelt <palmer@dabbelt.com>,\n Alistair Francis <alistair.francis@wdc.com>,\n Weiwei Li <liwei1518@gmail.com>,\n Daniel Henrique Barboza <daniel.barboza@oss.qualcomm.com>,\n Liu Zhiwei <zhiwei_liu@linux.alibaba.com>,\n Chao Liu <chao.liu.zevorn@gmail.com>","Cc":"qemu-devel@nongnu.org, tangtao1634@phytium.com.cn,\n devel@lists.libvirt.org,\n qemu-riscv@nongnu.org","Subject":"[PATCH v6 3/7] target/riscv: add sdext Debug Mode helpers","Date":"Sun, 12 Apr 2026 10:20:20 +0800","Message-ID":"\n <99be0e0e0d43c77389dd076d079b195844817076.1775959096.git.chao.liu.zevorn@gmail.com>","X-Mailer":"git-send-email 2.53.0","In-Reply-To":"<cover.1775959096.git.chao.liu.zevorn@gmail.com>","References":"<cover.1775959096.git.chao.liu.zevorn@gmail.com>","MIME-Version":"1.0","Content-Transfer-Encoding":"8bit","Received-SPF":"pass client-ip=2607:f8b0:4864:20::f41;\n envelope-from=chao.liu.zevorn@gmail.com; helo=mail-qv1-xf41.google.com","X-Spam_score_int":"-20","X-Spam_score":"-2.1","X-Spam_bar":"--","X-Spam_report":"(-2.1 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,\n DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001,\n RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001,\n SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no","X-Spam_action":"no action","X-BeenThere":"qemu-devel@nongnu.org","X-Mailman-Version":"2.1.29","Precedence":"list","List-Id":"qemu development <qemu-devel.nongnu.org>","List-Unsubscribe":"<https://lists.nongnu.org/mailman/options/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>","List-Archive":"<https://lists.nongnu.org/archive/html/qemu-devel>","List-Post":"<mailto:qemu-devel@nongnu.org>","List-Help":"<mailto:qemu-devel-request@nongnu.org?subject=help>","List-Subscribe":"<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n <mailto:qemu-devel-request@nongnu.org?subject=subscribe>","Errors-To":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org","Sender":"qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org"},"content":"RISC-V Debug Specification:\nhttps://github.com/riscv/riscv-debug-spec/releases/tag/1.0\n\nAdd helpers to enter/leave Debug Mode and to update dpc/dcsr.\nModel resume without a Debug Module by leaving Debug Mode at\ncpu_exec_enter and continuing from dpc.\n\nSigned-off-by: Chao Liu <chao.liu.zevorn@gmail.com>\nReviewed-by: Daniel Henrique Barboza <daniel.barboza@oss.qualcomm.com>\nTested-by: Tao Tang <tangtao1634@phytium.com.cn>\n---\n target/riscv/cpu.h         |  3 ++\n target/riscv/cpu_helper.c  | 84 ++++++++++++++++++++++++++++++++++++++\n target/riscv/debug.c       |  5 +++\n target/riscv/tcg/tcg-cpu.c | 14 +++++++\n 4 files changed, 106 insertions(+)","diff":"diff --git a/target/riscv/cpu.h b/target/riscv/cpu.h\nindex 2a265faae5..62732957a4 100644\n--- a/target/riscv/cpu.h\n+++ b/target/riscv/cpu.h\n@@ -624,6 +624,9 @@ bool riscv_cpu_tlb_fill(CPUState *cs, vaddr address, int size,\n char *riscv_isa_string(RISCVCPU *cpu);\n int riscv_cpu_max_xlen(RISCVCPUClass *mcc);\n bool riscv_cpu_option_set(const char *optname);\n+void riscv_cpu_enter_debug_mode(CPURISCVState *env, target_ulong pc,\n+                                uint32_t cause);\n+void riscv_cpu_leave_debug_mode(CPURISCVState *env);\n \n #ifndef CONFIG_USER_ONLY\n void riscv_cpu_do_interrupt(CPUState *cpu);\ndiff --git a/target/riscv/cpu_helper.c b/target/riscv/cpu_helper.c\nindex 659150c646..0ff11a45de 100644\n--- a/target/riscv/cpu_helper.c\n+++ b/target/riscv/cpu_helper.c\n@@ -136,6 +136,90 @@ bool riscv_env_smode_dbltrp_enabled(CPURISCVState *env, bool virt)\n #endif\n }\n \n+#ifndef CONFIG_USER_ONLY\n+static bool riscv_sdext_enabled(CPURISCVState *env)\n+{\n+    return riscv_cpu_cfg(env)->ext_sdext;\n+}\n+#endif\n+\n+void riscv_cpu_enter_debug_mode(CPURISCVState *env, target_ulong pc,\n+                                uint32_t cause)\n+{\n+#ifndef CONFIG_USER_ONLY\n+    if (!riscv_sdext_enabled(env)) {\n+        return;\n+    }\n+\n+    env->debug_mode = true;\n+    env->dpc = pc & get_xepc_mask(env);\n+    env->dcsr &= ~(DCSR_CAUSE_MASK | DCSR_PRV_MASK | DCSR_V);\n+    env->dcsr |= ((target_ulong)(cause & 0x7)) << DCSR_CAUSE_SHIFT;\n+    env->dcsr |= env->priv & DCSR_PRV_MASK;\n+    if (env->virt_enabled && riscv_has_ext(env, RVH)) {\n+        env->dcsr |= DCSR_V;\n+    }\n+\n+    if (env_archcpu(env)->cfg.ext_zicfilp) {\n+        if (env->elp) {\n+            env->dcsr |= DCSR_PELP;\n+        } else {\n+            env->dcsr &= ~DCSR_PELP;\n+        }\n+        env->elp = false;\n+    }\n+#endif\n+}\n+\n+void riscv_cpu_leave_debug_mode(CPURISCVState *env)\n+{\n+#ifndef CONFIG_USER_ONLY\n+    if (!riscv_sdext_enabled(env)) {\n+        return;\n+    }\n+\n+    target_ulong new_priv = env->dcsr & DCSR_PRV_MASK;\n+    bool new_virt = riscv_has_ext(env, RVH) && (env->dcsr & DCSR_V);\n+\n+    if (new_priv > PRV_M) {\n+        new_priv = PRV_M;\n+    }\n+    if (new_priv == PRV_M) {\n+        new_virt = false;\n+    }\n+\n+    if (new_priv == PRV_S && !riscv_has_ext(env, RVS)) {\n+        new_priv = PRV_M;\n+        new_virt = false;\n+    } else if (new_priv == PRV_U && !riscv_has_ext(env, RVU)) {\n+        new_priv = riscv_has_ext(env, RVS) ? PRV_S : PRV_M;\n+        new_virt = false;\n+    }\n+\n+    env->debug_mode = false;\n+    riscv_cpu_set_mode(env, new_priv, new_virt);\n+\n+    if (env_archcpu(env)->cfg.ext_zicfilp) {\n+        env->elp = cpu_get_fcfien(env) && (env->dcsr & DCSR_PELP);\n+        env->dcsr &= ~DCSR_PELP;\n+    }\n+\n+    if (new_priv != PRV_M) {\n+        env->mstatus = set_field(env->mstatus, MSTATUS_MPRV, 0);\n+    }\n+\n+    if (env_archcpu(env)->cfg.ext_smdbltrp && new_priv != PRV_M) {\n+        env->mstatus = set_field(env->mstatus, MSTATUS_MDT, 0);\n+    }\n+    if (env_archcpu(env)->cfg.ext_ssdbltrp && (new_priv == PRV_U || new_virt)) {\n+        env->mstatus = set_field(env->mstatus, MSTATUS_SDT, 0);\n+        if (new_virt && new_priv == PRV_U) {\n+            env->vsstatus = set_field(env->vsstatus, MSTATUS_SDT, 0);\n+        }\n+    }\n+#endif\n+}\n+\n RISCVPmPmm riscv_pm_get_pmm(CPURISCVState *env)\n {\n #ifndef CONFIG_USER_ONLY\ndiff --git a/target/riscv/debug.c b/target/riscv/debug.c\nindex 5664466749..5877a60c50 100644\n--- a/target/riscv/debug.c\n+++ b/target/riscv/debug.c\n@@ -927,6 +927,11 @@ void riscv_cpu_debug_excp_handler(CPUState *cs)\n     RISCVCPU *cpu = RISCV_CPU(cs);\n     CPURISCVState *env = &cpu->env;\n \n+    /* Triggers must not match or fire while in Debug Mode. */\n+    if (env->debug_mode) {\n+        return;\n+    }\n+\n     if (cs->watchpoint_hit) {\n         if (cs->watchpoint_hit->flags & BP_CPU) {\n             do_trigger_action(env, DBG_ACTION_BP);\ndiff --git a/target/riscv/tcg/tcg-cpu.c b/target/riscv/tcg/tcg-cpu.c\nindex 0613450691..4b17fbb611 100644\n--- a/target/riscv/tcg/tcg-cpu.c\n+++ b/target/riscv/tcg/tcg-cpu.c\n@@ -266,6 +266,19 @@ static vaddr riscv_pointer_wrap(CPUState *cs, int mmu_idx,\n     }\n     return extract64(result, 0, 64 - pm_len);\n }\n+\n+static void riscv_cpu_exec_enter(CPUState *cs)\n+{\n+    RISCVCPU *cpu = RISCV_CPU(cs);\n+    CPURISCVState *env = &cpu->env;\n+\n+    if (!cpu->cfg.ext_sdext || !env->debug_mode) {\n+        return;\n+    }\n+    target_ulong pc = env->dpc;\n+    riscv_cpu_leave_debug_mode(env);\n+    env->pc = pc;\n+}\n #endif\n \n const TCGCPUOps riscv_tcg_ops = {\n@@ -282,6 +295,7 @@ const TCGCPUOps riscv_tcg_ops = {\n #ifndef CONFIG_USER_ONLY\n     .tlb_fill = riscv_cpu_tlb_fill,\n     .pointer_wrap = riscv_pointer_wrap,\n+    .cpu_exec_enter = riscv_cpu_exec_enter,\n     .cpu_exec_interrupt = riscv_cpu_exec_interrupt,\n     .cpu_exec_halt = riscv_cpu_has_work,\n     .cpu_exec_reset = cpu_reset,\n","prefixes":["v6","3/7"]}