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 1FDC63858D37 for ; Mon, 21 Feb 2022 13:49:08 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 1FDC63858D37 Message-ID: Date: Mon, 21 Feb 2022 22:49:05 +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 Cc: Vineet Gupta , Binutils References: <77d4e5d6-5059-3f44-1ee9-8e2a6bfd3af3@suse.com> <268f323e-9f7a-2803-4b71-46506952fb52@irq.a4lg.com> From: Tsukasa OI In-Reply-To: <268f323e-9f7a-2803-4b71-46506952fb52@irq.a4lg.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-6.6 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:49:10 -0000 Ah, I forgot to note. The original Privileged Architecture 1.12 patchsets: (v1) (v2) are superseded by the latest one: (CSR only v1) ... as Privileged Architecture 1.12 itself is implemented as a part of hypervisor support (this is merged into master): Thanks, Tsukasa On 2022/02/21 22:38, Tsukasa OI via Binutils wrote: > 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 >> >