riscv / riscv-isa-manual

RISC-V Instruction Set Manual
https://riscv.org/
Creative Commons Attribution 4.0 International
3.65k stars 634 forks source link

Hypervisor chapter uses term HSLEN but I think it means HSXLEN #1693

Open james-ball-qualcomm opened 5 hours ago

james-ball-qualcomm commented 5 hours ago

Quote from chapter below. Should be HSXLEN, not HSLEN I believe.

The hstatus register is an HSXLEN-bit read/write register formatted as shown in Hypervisor status register (hstatus) when HSLEN=32 when HSXLEN=32 and Hypervisor status register (hstatus) when HSXLEN=64. when HSXLEN=64. The hstatus register provides facilities analogous to the mstatus register for tracking and controlling the exception behavior of a VS-mode guest. Hypervisor status register (hstatus) when HSLEN=32

images/bytefield/hstatusreg-rv32.edn

gfavor commented 5 hours ago

You're correct re the "HSLEN" typo. Please go ahead and create a PR to fix that. Thanks.

james-ball-qualcomm commented 4 hours ago

Cool! My first PR against the new ISA manuals!