From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-sender-0.a4lg.com (mail-sender.a4lg.com [153.120.152.154]) by sourceware.org (Postfix) with ESMTPS id 8454B3858D37 for ; Mon, 21 Feb 2022 13:39:01 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 8454B3858D37 Message-ID: <268f323e-9f7a-2803-4b71-46506952fb52@irq.a4lg.com> Date: Mon, 21 Feb 2022 22:38:57 +0900 Mime-Version: 1.0 Subject: Re: [PATCH v2 0/3] RISC-V: Prepare Privileged Architecture 1.12 Content-Language: en-US To: Jan Beulich References: <77d4e5d6-5059-3f44-1ee9-8e2a6bfd3af3@suse.com> From: Tsukasa OI Cc: Binutils , Vineet Gupta In-Reply-To: <77d4e5d6-5059-3f44-1ee9-8e2a6bfd3af3@suse.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-6.7 required=5.0 tests=BAYES_00, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, KAM_NUMSUBJECT, SPF_HELO_NONE, SPF_PASS, TXREP, T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: binutils@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Binutils mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 21 Feb 2022 13:39:03 -0000 On 2022/02/21 22:21, Jan Beulich wrote: > On 24.12.2021 04:22, Tsukasa OI via Binutils wrote: >> Tsukasa OI (3): >> RISC-V: Make testcase indentation consistent >> RISC-V: Unify "access all CSRs" testcases >> RISC-V: Add Privileged Architecture version 1.12 >> >> bfd/cpu-riscv.c | 1 + >> bfd/cpu-riscv.h | 1 + >> gas/config/tc-riscv.c | 2 +- >> gas/testsuite/gas/riscv/csr-dw-regnums.d | 2 +- >> gas/testsuite/gas/riscv/option-arch-01a.d | 8 +- >> gas/testsuite/gas/riscv/priv-reg-access-all.s | 292 ++++++++++ >> gas/testsuite/gas/riscv/priv-reg-fail-fext.d | 2 +- >> .../gas/riscv/priv-reg-fail-read-only-01.d | 2 +- >> .../gas/riscv/priv-reg-fail-read-only-01.s | 267 +-------- >> .../gas/riscv/priv-reg-fail-read-only-02.d | 2 +- >> .../gas/riscv/priv-reg-fail-rv32-only.d | 2 +- >> .../gas/riscv/priv-reg-fail-version-1p10.d | 2 +- >> .../gas/riscv/priv-reg-fail-version-1p11.d | 2 +- >> .../gas/riscv/priv-reg-fail-version-1p12.d | 11 + >> .../gas/riscv/priv-reg-fail-version-1p12.l | 24 + >> .../gas/riscv/priv-reg-fail-version-1p9p1.d | 2 +- >> gas/testsuite/gas/riscv/priv-reg-fail-zkr.d | 2 +- >> .../gas/riscv/priv-reg-version-1p10.d | 534 +++++++++--------- >> .../gas/riscv/priv-reg-version-1p11.d | 534 +++++++++--------- >> .../gas/riscv/priv-reg-version-1p12.d | 275 +++++++++ > > Comparing with the spec I find SCONTEXT still is numbered 0x7aa in here, > when the doc says 0x5a8. Was this overlooked? Or is the doc wrong? > > Jan You pointed out that right but still, this is intentional. In PATCH 3/6 commit message, I wrote: > (except moved "scontext" and > new "mscontext"). Because `scontext' is a part of debug specification (NOT privileged specification), I thought it should be controlled by debug specification version option like one proposed by Nelson Chu: Note that debug specification 1.0 (with moved `scontext' and new `mscontext') is not ratified yet. Tsukasa > >> .../gas/riscv/priv-reg-version-1p9p1.d | 534 +++++++++--------- >> gas/testsuite/gas/riscv/priv-reg.s | 294 +--------- >> 22 files changed, 1421 insertions(+), 1374 deletions(-) >> create mode 100644 gas/testsuite/gas/riscv/priv-reg-access-all.s >> create mode 100644 gas/testsuite/gas/riscv/priv-reg-fail-version-1p12.d >> create mode 100644 gas/testsuite/gas/riscv/priv-reg-fail-version-1p12.l >> create mode 100644 gas/testsuite/gas/riscv/priv-reg-version-1p12.d >> >> >> base-commit: d20236e748ab70e9243960850eef64838f1b9721 >