[PATCH v2 0/3] RISC-V: Prepare Privileged Architecture 1.12

[Tsukasa OI]

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:

<https://sourceware.org/pipermail/binutils/2021-August/117568.html>

Note that debug specification 1.0 (with moved `scontext' and new
`mscontext') is not ratified yet.

<https://github.com/riscv/riscv-debug-spec/commit/b659d7dc7f578e1a2a76f9e62a5eec0f2d80045c>

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
>