* [PATCH v6 2/4] dt-bindings: riscv: Add Svade and Svadu Entries [not found] <20240628093711.11716-1-yongxuan.wang@sifive.com> @ 2024-06-28 9:37 ` Yong-Xuan Wang 2024-06-28 16:19 ` Conor Dooley 0 siblings, 1 reply; 5+ messages in thread From: Yong-Xuan Wang @ 2024-06-28 9:37 UTC (permalink / raw) To: linux-kernel, linux-riscv, kvm-riscv, kvm Cc: greentime.hu, vincent.chen, Yong-Xuan Wang, Conor Dooley, Rob Herring, Krzysztof Kozlowski, Paul Walmsley, Palmer Dabbelt, Albert Ou, devicetree Add entries for the Svade and Svadu extensions to the riscv,isa-extensions property. Signed-off-by: Yong-Xuan Wang <yongxuan.wang@sifive.com> --- .../devicetree/bindings/riscv/extensions.yaml | 28 +++++++++++++++++++ 1 file changed, 28 insertions(+) diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml index 468c646247aa..c3d053ce7783 100644 --- a/Documentation/devicetree/bindings/riscv/extensions.yaml +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml @@ -153,6 +153,34 @@ properties: ratified at commit 3f9ed34 ("Add ability to manually trigger workflow. (#2)") of riscv-time-compare. + - const: svade + description: | + The standard Svade supervisor-level extension for SW-managed PTE A/D + bit updates as ratified in the 20240213 version of the privileged + ISA specification. + + Both Svade and Svadu extensions control the hardware behavior when + the PTE A/D bits need to be set. The default behavior for the four + possible combinations of these extensions in the device tree are: + 1) Neither Svade nor Svadu present in DT => It is technically + unknown whether the platform uses Svade or Svadu. Supervisor may + assume Svade to be present and enabled or it can discover based + on mvendorid, marchid, and mimpid. + 2) Only Svade present in DT => Supervisor must assume Svade to be + always enabled. (Obvious) + 3) Only Svadu present in DT => Supervisor must assume Svadu to be + always enabled. (Obvious) + 4) Both Svade and Svadu present in DT => Supervisor must assume + Svadu turned-off at boot time. To use Svadu, supervisor must + explicitly enable it using the SBI FWFT extension. + + - const: svadu + description: | + The standard Svadu supervisor-level extension for hardware updating + of PTE A/D bits as ratified at commit c1abccf ("Merge pull request + #25 from ved-rivos/ratified") of riscv-svadu. Please refer to Svade + dt-binding description for more details. + - const: svinval description: The standard Svinval supervisor-level extension for fine-grained -- 2.17.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v6 2/4] dt-bindings: riscv: Add Svade and Svadu Entries 2024-06-28 9:37 ` [PATCH v6 2/4] dt-bindings: riscv: Add Svade and Svadu Entries Yong-Xuan Wang @ 2024-06-28 16:19 ` Conor Dooley 2024-06-29 13:09 ` Jessica Clarke 0 siblings, 1 reply; 5+ messages in thread From: Conor Dooley @ 2024-06-28 16:19 UTC (permalink / raw) To: Yong-Xuan Wang Cc: linux-kernel, linux-riscv, kvm-riscv, kvm, greentime.hu, vincent.chen, Rob Herring, Krzysztof Kozlowski, Paul Walmsley, Palmer Dabbelt, Albert Ou, devicetree [-- Attachment #1: Type: text/plain, Size: 3149 bytes --] On Fri, Jun 28, 2024 at 05:37:06PM +0800, Yong-Xuan Wang wrote: > Add entries for the Svade and Svadu extensions to the riscv,isa-extensions > property. > > Signed-off-by: Yong-Xuan Wang <yongxuan.wang@sifive.com> > --- > .../devicetree/bindings/riscv/extensions.yaml | 28 +++++++++++++++++++ > 1 file changed, 28 insertions(+) > > diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml > index 468c646247aa..c3d053ce7783 100644 > --- a/Documentation/devicetree/bindings/riscv/extensions.yaml > +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml > @@ -153,6 +153,34 @@ properties: > ratified at commit 3f9ed34 ("Add ability to manually trigger > workflow. (#2)") of riscv-time-compare. > > + - const: svade > + description: | > + The standard Svade supervisor-level extension for SW-managed PTE A/D > + bit updates as ratified in the 20240213 version of the privileged > + ISA specification. > + > + Both Svade and Svadu extensions control the hardware behavior when > + the PTE A/D bits need to be set. The default behavior for the four > + possible combinations of these extensions in the device tree are: > + 1) Neither Svade nor Svadu present in DT => > It is technically > + unknown whether the platform uses Svade or Svadu. Supervisor may > + assume Svade to be present and enabled or it can discover based > + on mvendorid, marchid, and mimpid. I would just write "for backwards compatibility, if neither Svade nor Svadu appear in the devicetree the supervisor may assume Svade to be present and enabled". If there are systems that this behaviour causes problems for, we can deal with them iff they appear. I don't think looking at m*id would be sufficient here anyway, since the firmware can have an impact. I'd just drop that part entirely. > + 2) Only Svade present in DT => Supervisor must assume Svade to be > + always enabled. (Obvious) nit: I'd drop the "(Obvious)" comments from here. Cheers, Conor. > + 3) Only Svadu present in DT => Supervisor must assume Svadu to be > + always enabled. (Obvious) > + 4) Both Svade and Svadu present in DT => Supervisor must assume > + Svadu turned-off at boot time. To use Svadu, supervisor must > + explicitly enable it using the SBI FWFT extension. > + > + - const: svadu > + description: | > + The standard Svadu supervisor-level extension for hardware updating > + of PTE A/D bits as ratified at commit c1abccf ("Merge pull request > + #25 from ved-rivos/ratified") of riscv-svadu. Please refer to Svade > + dt-binding description for more details. > + > - const: svinval > description: > The standard Svinval supervisor-level extension for fine-grained > -- > 2.17.1 > [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v6 2/4] dt-bindings: riscv: Add Svade and Svadu Entries 2024-06-28 16:19 ` Conor Dooley @ 2024-06-29 13:09 ` Jessica Clarke 2024-06-30 14:09 ` Conor Dooley 0 siblings, 1 reply; 5+ messages in thread From: Jessica Clarke @ 2024-06-29 13:09 UTC (permalink / raw) To: Conor Dooley Cc: Yong-Xuan Wang, LKML, linux-riscv, kvm-riscv, kvm, Greentime Hu, Vincent Chen, Rob Herring, Krzysztof Kozlowski, Paul Walmsley, Palmer Dabbelt, Albert Ou, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS On 28 Jun 2024, at 17:19, Conor Dooley <conor@kernel.org> wrote: > > On Fri, Jun 28, 2024 at 05:37:06PM +0800, Yong-Xuan Wang wrote: >> Add entries for the Svade and Svadu extensions to the riscv,isa-extensions >> property. >> >> Signed-off-by: Yong-Xuan Wang <yongxuan.wang@sifive.com> >> --- >> .../devicetree/bindings/riscv/extensions.yaml | 28 +++++++++++++++++++ >> 1 file changed, 28 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml >> index 468c646247aa..c3d053ce7783 100644 >> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml >> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml >> @@ -153,6 +153,34 @@ properties: >> ratified at commit 3f9ed34 ("Add ability to manually trigger >> workflow. (#2)") of riscv-time-compare. >> >> + - const: svade >> + description: | >> + The standard Svade supervisor-level extension for SW-managed PTE A/D >> + bit updates as ratified in the 20240213 version of the privileged >> + ISA specification. >> + >> + Both Svade and Svadu extensions control the hardware behavior when >> + the PTE A/D bits need to be set. The default behavior for the four >> + possible combinations of these extensions in the device tree are: >> + 1) Neither Svade nor Svadu present in DT => > >> It is technically >> + unknown whether the platform uses Svade or Svadu. Supervisor may >> + assume Svade to be present and enabled or it can discover based >> + on mvendorid, marchid, and mimpid. > > I would just write "for backwards compatibility, if neither Svade nor > Svadu appear in the devicetree the supervisor may assume Svade to be > present and enabled". If there are systems that this behaviour causes > problems for, we can deal with them iff they appear. I don't think > looking at m*id would be sufficient here anyway, since the firmware can > have an impact. I'd just drop that part entirely. Older QEMU falls into that category, as do Bluespec’s soft-cores (which ours are derived from at Cambridge). I feel that, in reality, one should be prepared to handle both trapping and atomic updates if writing an OS that aims to support case 1. Jess ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v6 2/4] dt-bindings: riscv: Add Svade and Svadu Entries 2024-06-29 13:09 ` Jessica Clarke @ 2024-06-30 14:09 ` Conor Dooley 2024-07-11 10:34 ` Yong-Xuan Wang 0 siblings, 1 reply; 5+ messages in thread From: Conor Dooley @ 2024-06-30 14:09 UTC (permalink / raw) To: Jessica Clarke Cc: Yong-Xuan Wang, LKML, linux-riscv, kvm-riscv, kvm, Greentime Hu, Vincent Chen, Rob Herring, Krzysztof Kozlowski, Paul Walmsley, Palmer Dabbelt, Albert Ou, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS [-- Attachment #1: Type: text/plain, Size: 2875 bytes --] On Sat, Jun 29, 2024 at 02:09:34PM +0100, Jessica Clarke wrote: > On 28 Jun 2024, at 17:19, Conor Dooley <conor@kernel.org> wrote: > > > > On Fri, Jun 28, 2024 at 05:37:06PM +0800, Yong-Xuan Wang wrote: > >> Add entries for the Svade and Svadu extensions to the riscv,isa-extensions > >> property. > >> > >> Signed-off-by: Yong-Xuan Wang <yongxuan.wang@sifive.com> > >> --- > >> .../devicetree/bindings/riscv/extensions.yaml | 28 +++++++++++++++++++ > >> 1 file changed, 28 insertions(+) > >> > >> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml > >> index 468c646247aa..c3d053ce7783 100644 > >> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml > >> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml > >> @@ -153,6 +153,34 @@ properties: > >> ratified at commit 3f9ed34 ("Add ability to manually trigger > >> workflow. (#2)") of riscv-time-compare. > >> > >> + - const: svade > >> + description: | > >> + The standard Svade supervisor-level extension for SW-managed PTE A/D > >> + bit updates as ratified in the 20240213 version of the privileged > >> + ISA specification. > >> + > >> + Both Svade and Svadu extensions control the hardware behavior when > >> + the PTE A/D bits need to be set. The default behavior for the four > >> + possible combinations of these extensions in the device tree are: > >> + 1) Neither Svade nor Svadu present in DT => > > > >> It is technically > >> + unknown whether the platform uses Svade or Svadu. Supervisor may > >> + assume Svade to be present and enabled or it can discover based > >> + on mvendorid, marchid, and mimpid. > > > > I would just write "for backwards compatibility, if neither Svade nor > > Svadu appear in the devicetree the supervisor may assume Svade to be > > present and enabled". If there are systems that this behaviour causes > > problems for, we can deal with them iff they appear. I don't think > > looking at m*id would be sufficient here anyway, since the firmware can > > have an impact. I'd just drop that part entirely. > > Older QEMU falls into that category, as do Bluespec’s soft-cores (which > ours are derived from at Cambridge). I feel that, in reality, one > should be prepared to handle both trapping and atomic updates if > writing an OS that aims to support case 1. I guess that is actually what we should put in then, to use an approximation of your wording, something like Neither Svade nor Svadu present in DT => Supervisor software should be prepared to handle either hardware updating of the PTE A/D bits or page faults when they need updated ? [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 228 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v6 2/4] dt-bindings: riscv: Add Svade and Svadu Entries 2024-06-30 14:09 ` Conor Dooley @ 2024-07-11 10:34 ` Yong-Xuan Wang 0 siblings, 0 replies; 5+ messages in thread From: Yong-Xuan Wang @ 2024-07-11 10:34 UTC (permalink / raw) To: Conor Dooley Cc: Jessica Clarke, LKML, linux-riscv, kvm-riscv, kvm, Greentime Hu, Vincent Chen, Rob Herring, Krzysztof Kozlowski, Paul Walmsley, Palmer Dabbelt, Albert Ou, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS Hi Conor and Jessica, On Sun, Jun 30, 2024 at 10:09 PM Conor Dooley <conor@kernel.org> wrote: > > On Sat, Jun 29, 2024 at 02:09:34PM +0100, Jessica Clarke wrote: > > On 28 Jun 2024, at 17:19, Conor Dooley <conor@kernel.org> wrote: > > > > > > On Fri, Jun 28, 2024 at 05:37:06PM +0800, Yong-Xuan Wang wrote: > > >> Add entries for the Svade and Svadu extensions to the riscv,isa-extensions > > >> property. > > >> > > >> Signed-off-by: Yong-Xuan Wang <yongxuan.wang@sifive.com> > > >> --- > > >> .../devicetree/bindings/riscv/extensions.yaml | 28 +++++++++++++++++++ > > >> 1 file changed, 28 insertions(+) > > >> > > >> diff --git a/Documentation/devicetree/bindings/riscv/extensions.yaml b/Documentation/devicetree/bindings/riscv/extensions.yaml > > >> index 468c646247aa..c3d053ce7783 100644 > > >> --- a/Documentation/devicetree/bindings/riscv/extensions.yaml > > >> +++ b/Documentation/devicetree/bindings/riscv/extensions.yaml > > >> @@ -153,6 +153,34 @@ properties: > > >> ratified at commit 3f9ed34 ("Add ability to manually trigger > > >> workflow. (#2)") of riscv-time-compare. > > >> > > >> + - const: svade > > >> + description: | > > >> + The standard Svade supervisor-level extension for SW-managed PTE A/D > > >> + bit updates as ratified in the 20240213 version of the privileged > > >> + ISA specification. > > >> + > > >> + Both Svade and Svadu extensions control the hardware behavior when > > >> + the PTE A/D bits need to be set. The default behavior for the four > > >> + possible combinations of these extensions in the device tree are: > > >> + 1) Neither Svade nor Svadu present in DT => > > > > > >> It is technically > > >> + unknown whether the platform uses Svade or Svadu. Supervisor may > > >> + assume Svade to be present and enabled or it can discover based > > >> + on mvendorid, marchid, and mimpid. > > > > > > I would just write "for backwards compatibility, if neither Svade nor > > > Svadu appear in the devicetree the supervisor may assume Svade to be > > > present and enabled". If there are systems that this behaviour causes > > > problems for, we can deal with them iff they appear. I don't think > > > looking at m*id would be sufficient here anyway, since the firmware can > > > have an impact. I'd just drop that part entirely. > > > > Older QEMU falls into that category, as do Bluespec’s soft-cores (which > > ours are derived from at Cambridge). I feel that, in reality, one > > should be prepared to handle both trapping and atomic updates if > > writing an OS that aims to support case 1. > > I guess that is actually what we should put in then, to use an > approximation of your wording, something like > Neither Svade nor Svadu present in DT => Supervisor software should be > prepared to handle either hardware updating of the PTE A/D bits or page > faults when they need updated > ? Thank you! I will update in the next version. Regards, Yong-Xuan ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-07-11 10:35 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20240628093711.11716-1-yongxuan.wang@sifive.com>
2024-06-28 9:37 ` [PATCH v6 2/4] dt-bindings: riscv: Add Svade and Svadu Entries Yong-Xuan Wang
2024-06-28 16:19 ` Conor Dooley
2024-06-29 13:09 ` Jessica Clarke
2024-06-30 14:09 ` Conor Dooley
2024-07-11 10:34 ` Yong-Xuan Wang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).