From 129a484b6fec7a7cbdae33c6f53da195ed7acb8c Mon Sep 17 00:00:00 2001 From: Tariq Kurd Date: Tue, 17 Dec 2024 17:37:06 +0100 Subject: [PATCH] Split sstatus.CRG into CRGS/CRGU --- src/cheri-pte-ext.adoc | 49 ++++++++++++++++++++-------------- src/insns/load_exceptions.adoc | 2 +- 2 files changed, 30 insertions(+), 21 deletions(-) diff --git a/src/cheri-pte-ext.adoc b/src/cheri-pte-ext.adoc index aa430e9e..6219571f 100644 --- a/src/cheri-pte-ext.adoc +++ b/src/cheri-pte-ext.adoc @@ -12,11 +12,11 @@ format. NOTE: There is no explicit mechanism for enabling or disabling {cheri_pte_ext_name}. A VM-enabled legacy (non-CHERI) OS running in {cheri_int_mode_name} will not load or store capabilities, and so the default state of CW=0 causing loaded capabilities to have their tags cleared, and stored capabilities with their tags set to cause a page fault, won't occur. -A CHERI-aware OS running a VM-enabled OS is strongly recommended to support {cheri_pte_ext_name}, and the minimum level of support is to set CW to 1 in all PTEs intended for storing capabilities (i.e. anonymous mappings) and leave <>.CRG and CRG in all PTEs set to 0, which will allow capabilities with their tags set to be loaded and stored successfully. +A CHERI-aware OS running a VM-enabled OS is strongly recommended to support {cheri_pte_ext_name}, and the minimum level of support is to set CW to 1 in all PTEs intended for storing capabilities (i.e. anonymous mappings) and leave <>.CRGS/CRGU and CRG in all PTEs set to 0, which will allow capabilities with their tags set to be loaded and stored successfully. Therefore when implementing any RV64 virtual memory translation scheme (_Sv39_, _Sv48_ or _Sv57_) and {cheri_base_ext_name}, implementing {cheri_pte_ext_name} is strongly recommended. -NOTE: It is possible to detect the presence of {cheri_pte_ext_name} in software, by configuring a page table entry without programming CW and without setting <>.CRG, and testing for an exception on storing a tagged capability. +NOTE: It is possible to detect the presence of {cheri_pte_ext_name} in software, by configuring a page table entry without programming CW and without setting <>.CRGS/CRGU, and testing for an exception on storing a tagged capability. NOTE: _Sv32_ (for RV32) does not have any spare PTE bits, and so this extension cannot be implemented. @@ -108,11 +108,15 @@ If the CW bit is clear then: NOTE: The tag bit of the stored capability is checked _after_ it is potentially cleared <>. +NOTE: Two capability revocation generation bits (CRG) are implemented in <>, +one for kernel-space code running in supervisor mode (CRGS) and one for user-space code running +in user mode (CRGU). The relevant bit to use is only controlled by the current operating mode. + ** The same behavior as when CRG is clear, allowing software interpretation of this state. ** When a capability store or AMO instruction is executed and the tag bit of the capability being written is set, the -implementation sets the CW bit and assigns the CRG bit equal to <>.CRG. +implementation sets the CW bit and assigns the CRG bit equal to <>.CRGS/CRGU. + The PTE update must be atomic with respect to other accesses to the PTE, and must atomically check @@ -135,19 +139,21 @@ When CW is set, the CRG bit indicates the current generation of the virtual memo regards to the ongoing capability revocation cycle. Two schemes are permitted: * A load page fault exception is raised when a capability load or AMO instruction is executed -with <> granted and the virtual page's CRG bit does not equal <>.CRG. +with <> granted and the virtual page's CRG bit does not equal <>.CRGS/CRGU. * A load page fault exception is raised when a capability load or AMO instruction is executed -with <> granted and the virtual page's CRG bit does not equal <>.CRG +with <> granted and the virtual page's CRG bit does not equal <>.CRGS/CRGU. and the capability read from memory optionally has its tag set^1^. [[pte_cw_crg_load_summary]] .Summary of Load CW and CRG behavior in the PTEs -[%autowidth,float="center",align="center",cols="<,<,<",options="header"] +[%autowidth,float="center",align="center",cols="<,<,<,<",options="header"] |=== -|PTE.CW |PTE.CRG |Load/AMO -| 0 | X | Clear loaded tag -| 1 |≠ <>.CRG | Page fault, or page fault if tag is set^1^ -| 1 |= <>.CRG | Normal operation +|PTE.CW |Mode |PTE.CRG |Load/AMO +| 0 | S/U | X | Clear loaded tag +| 1 | S |≠ <>.CRGS | Page fault, or page fault if tag is set^1^ +| 1 | U |≠ <>.CRGU | Page fault, or page fault if tag is set^1^ +| 1 | S |= <>.CRGS | Normal operation +| 1 | U |= <>.CRGU | Normal operation |=== [[pte_cw_crg_store_summary]] @@ -181,18 +187,18 @@ The decision about whether to take exceptions on capability stores with the tag These cause PTE Accessed and Dirty updates to be done in software, via the exception handler, or by a hardware mechanism respectively. * If only _Svade_ is implemented, or enabled through henvcfg.ADUE or menvcfg.ADUE, then take a page fault. -* If only _Svadu_ is implemented, or enabled through henvcfg.ADUE or menvcfg.ADUE, then do the hardware update of setting PTE.CW=1 and setting PTE.CRG=<>.CRG as described in <>. +* If only _Svadu_ is implemented, or enabled through henvcfg.ADUE or menvcfg.ADUE, then do the hardware update of setting PTE.CW=1 and setting PTE.CRG=<>.CRGS/CRGU as described in <>. [#xstatus_pte] === Extending the Supervisor (sstatus) and Virtual Supervisor (vsstatus) Status Registers The <> and <> CSRs are extended to include the new Capability Read Generation (CRG) bit as shown. -When V=1 <>.CRG is in effect. +When V=1 <>.CRGS/CRGU is in effect. -<>.CRG also exists. Reading or writing it is equivalent to reading or writing <>.CRG. +<>.CRGS/CRGU also exist. Reading or writing them is equivalent to reading or writing <>.CRGS/CRGU. -NOTE: As there is no M-mode translation available in RISC-V, there is no current software use for <>.CRG. +NOTE: As there is no M-mode translation available in RISC-V, there is no current software use for <>.CRGS/CRGU or an M-mode equivalent bit. It is _only_ included not to break the rule that <> is required to be a subset of <>. @@ -232,8 +238,9 @@ It is _only_ included not to break the rule that <> is r {bits: 1, name: 'WPRI'}, {bits: 1, name: 'MPELP'}, {bits: 1, name: 'MDT'}, - {bits: 19, name: 'WPRI'}, - {bits: 1, name: 'CRG'}, + {bits: 18, name: 'WPRI'}, + {bits: 1, name: 'CRGU'}, + {bits: 1, name: 'CRGS'}, {bits: 1, name: 'SD'}, ], config:{lanes: 4, hspace:1024}} .... @@ -262,8 +269,9 @@ It is _only_ included not to break the rule that <> is r {bits: 1, name: 'SDT'}, {bits: 7, name: 'WPRI'}, {bits: 2, name: 'UXL[1:0]'}, - {bits: 28, name: 'WPRI'}, - {bits: 1, name: 'CRG'}, + {bits: 27, name: 'WPRI'}, + {bits: 1, name: 'CRGU'}, + {bits: 1, name: 'CRGS'}, {bits: 1, name: 'SD'}, ], config:{lanes: 4, hspace:1024}} .... @@ -289,8 +297,9 @@ It is _only_ included not to break the rule that <> is r {bits: 1, name: 'SUM'}, {bits: 12, name: 'WPRI'}, {bits: 2, name: 'UXL[1:0]'}, - {bits: 28, name: 'WPRI'}, - {bits: 1, name: 'CRG'}, + {bits: 27, name: 'WPRI'}, + {bits: 1, name: 'CRGU'}, + {bits: 1, name: 'CRGS'}, {bits: 1, name: 'SD'} ], config:{lanes: 4, hspace:1024}} .... diff --git a/src/insns/load_exceptions.adoc b/src/insns/load_exceptions.adoc index de2b69eb..e4c8b178 100644 --- a/src/insns/load_exceptions.adoc +++ b/src/insns/load_exceptions.adoc @@ -25,7 +25,7 @@ listed below; in this case, _CHERI data fault_ is reported in the <> or + If {cheri_pte_ext_name} is implemented, and virtual memory is enabled, then the state of <>.CW and <>.CRG from the current virtual memory page, -together with <>.CRG may cause a CHERI <> page fault exception +together with <>.CRGS/CRGU may cause a CHERI <> page fault exception in addition to a normal RISC-V page fault exception. See <> for the exception reporting in this case. +