Message ID | 20240625210933.1620802-2-samuel.holland@sifive.com (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | riscv: Userspace pointer masking and tagged address ABI | expand |
Context | Check | Description |
---|---|---|
conchuod/vmtest-fixes-PR | fail | merge-conflict |
On Tue, Jun 25, 2024 at 02:09:12PM -0700, Samuel Holland wrote: > The RISC-V Pointer Masking specification defines three extensions: > Smmpm, Smnpm, and Ssnpm. Document the behavior of these extensions as > following the current draft of the specification, which is 1.0.0-rc2. You say draft, but the actual extension has already completed public review, right? > > Signed-off-by: Samuel Holland <samuel.holland@sifive.com> > --- > > Changes in v2: > - Update pointer masking specification version reference > > .../devicetree/bindings/riscv/extensions.yaml | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml > index cfed80ad5540..b6aeedc53676 100644 > --- a/Documentation/devicetree/bindings/riscv/extensions.yaml > +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml > @@ -128,6 +128,18 @@ properties: > changes to interrupts as frozen at commit ccbddab ("Merge pull > request #42 from riscv/jhauser-2023-RC4") of riscv-aia. > > + - const: smmpm > + description: | > + The standard Smmpm extension for M-mode pointer masking as defined > + at commit 654a5c4a7725 ("Update PDF and version number.") of > + riscv-j-extension. > + > + - const: smnpm > + description: | > + The standard Smnpm extension for next-mode pointer masking as defined > + at commit 654a5c4a7725 ("Update PDF and version number.") of > + riscv-j-extension. > + > - const: smstateen > description: | > The standard Smstateen extension for controlling access to CSRs > @@ -147,6 +159,12 @@ properties: > and mode-based filtering as ratified at commit 01d1df0 ("Add ability > to manually trigger workflow. (#2)") of riscv-count-overflow. > > + - const: ssnpm > + description: | > + The standard Ssnpm extension for next-mode pointer masking as defined > + at commit 654a5c4a7725 ("Update PDF and version number.") of > + riscv-j-extension. > + > - const: sstc > description: | > The standard Sstc supervisor-level extension for time compare as > -- > 2.44.1 >
Hi Conor, On 2024-06-26 11:01 AM, Conor Dooley wrote: > On Tue, Jun 25, 2024 at 02:09:12PM -0700, Samuel Holland wrote: >> The RISC-V Pointer Masking specification defines three extensions: >> Smmpm, Smnpm, and Ssnpm. Document the behavior of these extensions as >> following the current draft of the specification, which is 1.0.0-rc2. > > You say draft, but the actual extension has already completed public > review, right? Correct. The spec is frozen, and public review is complete. Here's the tracking ticket for details: https://jira.riscv.org/browse/RVS-1111 I use the word draft because it is still an -rc version, but I can reword this if you prefer. Regards, Samuel >> Signed-off-by: Samuel Holland <samuel.holland@sifive.com> >> --- >> >> Changes in v2: >> - Update pointer masking specification version reference >> >> .../devicetree/bindings/riscv/extensions.yaml | 18 ++++++++++++++++++ >> 1 file changed, 18 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml >> index cfed80ad5540..b6aeedc53676 100644 >> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml >> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml >> @@ -128,6 +128,18 @@ properties: >> changes to interrupts as frozen at commit ccbddab ("Merge pull >> request #42 from riscv/jhauser-2023-RC4") of riscv-aia. >> >> + - const: smmpm >> + description: | >> + The standard Smmpm extension for M-mode pointer masking as defined >> + at commit 654a5c4a7725 ("Update PDF and version number.") of >> + riscv-j-extension. >> + >> + - const: smnpm >> + description: | >> + The standard Smnpm extension for next-mode pointer masking as defined >> + at commit 654a5c4a7725 ("Update PDF and version number.") of >> + riscv-j-extension. >> + >> - const: smstateen >> description: | >> The standard Smstateen extension for controlling access to CSRs >> @@ -147,6 +159,12 @@ properties: >> and mode-based filtering as ratified at commit 01d1df0 ("Add ability >> to manually trigger workflow. (#2)") of riscv-count-overflow. >> >> + - const: ssnpm >> + description: | >> + The standard Ssnpm extension for next-mode pointer masking as defined >> + at commit 654a5c4a7725 ("Update PDF and version number.") of >> + riscv-j-extension. >> + >> - const: sstc >> description: | >> The standard Sstc supervisor-level extension for time compare as >> -- >> 2.44.1 >>
On Wed, Jun 26, 2024 at 11:14:27AM -0500, Samuel Holland wrote: > Hi Conor, > > On 2024-06-26 11:01 AM, Conor Dooley wrote: > > On Tue, Jun 25, 2024 at 02:09:12PM -0700, Samuel Holland wrote: > >> The RISC-V Pointer Masking specification defines three extensions: > >> Smmpm, Smnpm, and Ssnpm. Document the behavior of these extensions as > >> following the current draft of the specification, which is 1.0.0-rc2. > > > > You say draft, but the actual extension has already completed public > > review, right? > > Correct. The spec is frozen, and public review is complete. Here's the tracking > ticket for details: https://jira.riscv.org/browse/RVS-1111 > > I use the word draft because it is still an -rc version, but I can reword this > if you prefer. No, it's fine. I just was double checking the state of the extension before acking the patch. It'd be good, in the future to note what the status is, given the policy is to not accept things that are at least frozen. Acked-by: Conor Dooley <conor.dooley@microchip.com> Thanks, Conor.
diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml index cfed80ad5540..b6aeedc53676 100644 --- a/Documentation/devicetree/bindings/riscv/extensions.yaml +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml @@ -128,6 +128,18 @@ properties: changes to interrupts as frozen at commit ccbddab ("Merge pull request #42 from riscv/jhauser-2023-RC4") of riscv-aia. + - const: smmpm + description: | + The standard Smmpm extension for M-mode pointer masking as defined + at commit 654a5c4a7725 ("Update PDF and version number.") of + riscv-j-extension. + + - const: smnpm + description: | + The standard Smnpm extension for next-mode pointer masking as defined + at commit 654a5c4a7725 ("Update PDF and version number.") of + riscv-j-extension. + - const: smstateen description: | The standard Smstateen extension for controlling access to CSRs @@ -147,6 +159,12 @@ properties: and mode-based filtering as ratified at commit 01d1df0 ("Add ability to manually trigger workflow. (#2)") of riscv-count-overflow. + - const: ssnpm + description: | + The standard Ssnpm extension for next-mode pointer masking as defined + at commit 654a5c4a7725 ("Update PDF and version number.") of + riscv-j-extension. + - const: sstc description: | The standard Sstc supervisor-level extension for time compare as
The RISC-V Pointer Masking specification defines three extensions: Smmpm, Smnpm, and Ssnpm. Document the behavior of these extensions as following the current draft of the specification, which is 1.0.0-rc2. Signed-off-by: Samuel Holland <samuel.holland@sifive.com> --- Changes in v2: - Update pointer masking specification version reference .../devicetree/bindings/riscv/extensions.yaml | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+)