Re: [PATCH v4 5/6] Minor cosmetic changes in SBI section of OS/A-base platform


atishp@...
 

On Sat, 2021-08-07 at 11:43 +0530, Anup Patel wrote:
We don't need a table for required SBI extensions and UEFI interfaces
implemented by UEFI firmware.

Signed-off-by: Anup Patel <anup.patel@...>
Reviewed-by: Atish Patra <atish.patra@...>

---
 riscv-platform-spec.adoc | 49 ++++++++++++++++++----------------------
 1 file changed, 22 insertions(+), 27 deletions(-)

diff --git a/riscv-platform-spec.adoc b/riscv-platform-spec.adoc
index 9b29180..e4a8bdc 100644
--- a/riscv-platform-spec.adoc
+++ b/riscv-platform-spec.adoc
@@ -449,35 +449,32 @@ mentioned in the requirements.
 
 
 ==== Runtime Services
-===== SBI
-
-- Required SBI spec version is 0.3 or higher.
-- Required SBI extensions -
 
-[cols="1,2", width=80%, align="left", options="header"]
-|===
-|EXTENSION     |    CONDITIONS
-|SBI TIME      |    if **stimecmp** CSR not available
-|SBI IPI       |
-|SBI RFENCE    |
-|SBI HSM       |
-|SBI SRST      |   
-|SBI PMU       |
-|===
-
-- Required responsibilities of M-Mode runtime also includes -
-    ** Interrupt and Exception Delegation
-    ** Misaligned Load & Stores handling
-    ** Missing CSRs emulation
-    ** PMP Configuration
+===== SBI
 
-- Wherever applicable firmware must implement UEFI interfaces over
similar
-interfaces and services present in the SBI specification. For example,
UEFI
-runtime services must implement ResetSystem() via SBI Reset extension.
+* The M-mode runtime must implement SBI specification version 0.3
+  or higher
+* Required SBI extensions include:
+** SBI TIME
+** SBI IPI
+** SBI RFENCE
+** SBI HSM
+** SBI SRST
+** SBI PMU
+* Required features of the M-Mode runtime also includes
+** Delegate interrupts and exceptions to S-mode
+** Handle misaligned load and store
+** Emulate missing CSRs
+** Configure PMP CSRs
 
 ===== UEFI
-- OS should prioritize calling the UEFI interfaces before the SBI or
Platform
-specific mechanisms.
+
+* Wherever applicable UEFI firmware must implement UEFI interfaces
over
+  similar interfaces and services present in the SBI specification.
For
+  example, the UEFI ResetSystem() service must be implemented via the
+  SBI System Reset Extension.
+* The operating system should prioritize calling the UEFI interfaces
before
+  the SBI or platform specific mechanisms.
 
 // Server extension for OS-A Platform
 === Server Extension
@@ -676,8 +673,6 @@ process.
 |===
 
 ===== Runtime services
-====== SBI
-*TBD*
 
 ====== UEFI
 The UEFI run time services listed below are required to be
implemented.
--
Regards,
Atish

Join {tech-unixplatformspec@lists.riscv.org to automatically receive all group messages.