All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] include: sbi: Change SBI spec version to 3.0
@ 2025-05-16 12:28 Anup Patel
  2025-05-16 18:06 ` Atish Patra
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Anup Patel @ 2025-05-16 12:28 UTC (permalink / raw)
  To: Atish Patra; +Cc: Andrew Jones, Anup Patel, opensbi, Anup Patel

Now that SBI v3.0 specification is frozen, change runtime SBI version
implemented by OpenSBI to v3.0. Also, mark extensions defined by the
SBI v3.0 specification as non-experimental.

Signed-off-by: Anup Patel <apatel@ventanamicro.com>
---
 include/sbi/sbi_ecall.h  | 2 +-
 lib/sbi/sbi_ecall_dbtr.c | 1 -
 lib/sbi/sbi_ecall_fwft.c | 1 -
 lib/sbi/sbi_ecall_mpxy.c | 1 -
 lib/sbi/sbi_ecall_sse.c  | 1 -
 5 files changed, 1 insertion(+), 5 deletions(-)

diff --git a/include/sbi/sbi_ecall.h b/include/sbi/sbi_ecall.h
index 88a69bb3..15400e5a 100644
--- a/include/sbi/sbi_ecall.h
+++ b/include/sbi/sbi_ecall.h
@@ -13,7 +13,7 @@
 #include <sbi/sbi_types.h>
 #include <sbi/sbi_list.h>
 
-#define SBI_ECALL_VERSION_MAJOR		2
+#define SBI_ECALL_VERSION_MAJOR		3
 #define SBI_ECALL_VERSION_MINOR		0
 #define SBI_OPENSBI_IMPID		1
 
diff --git a/lib/sbi/sbi_ecall_dbtr.c b/lib/sbi/sbi_ecall_dbtr.c
index d28fd396..9c496c42 100644
--- a/lib/sbi/sbi_ecall_dbtr.c
+++ b/lib/sbi/sbi_ecall_dbtr.c
@@ -69,7 +69,6 @@ struct sbi_ecall_extension ecall_dbtr = {
 	.name			= "dbtr",
 	.extid_start		= SBI_EXT_DBTR,
 	.extid_end		= SBI_EXT_DBTR,
-	.experimental		= true,
 	.handle			= sbi_ecall_dbtr_handler,
 	.register_extensions	= sbi_ecall_dbtr_register_extensions,
 };
diff --git a/lib/sbi/sbi_ecall_fwft.c b/lib/sbi/sbi_ecall_fwft.c
index 3519285e..2277405e 100644
--- a/lib/sbi/sbi_ecall_fwft.c
+++ b/lib/sbi/sbi_ecall_fwft.c
@@ -45,7 +45,6 @@ struct sbi_ecall_extension ecall_fwft = {
 	.name			= "fwft",
 	.extid_start		= SBI_EXT_FWFT,
 	.extid_end		= SBI_EXT_FWFT,
-	.experimental		= true,
 	.register_extensions	= sbi_ecall_fwft_register_extensions,
 	.handle			= sbi_ecall_fwft_handler,
 };
diff --git a/lib/sbi/sbi_ecall_mpxy.c b/lib/sbi/sbi_ecall_mpxy.c
index 03ac71b2..0a5e3525 100644
--- a/lib/sbi/sbi_ecall_mpxy.c
+++ b/lib/sbi/sbi_ecall_mpxy.c
@@ -67,7 +67,6 @@ struct sbi_ecall_extension ecall_mpxy = {
 	.name			= "mpxy",
 	.extid_start		= SBI_EXT_MPXY,
 	.extid_end		= SBI_EXT_MPXY,
-	.experimental		= true,
 	.register_extensions	= sbi_ecall_mpxy_register_extensions,
 	.handle			= sbi_ecall_mpxy_handler,
 };
diff --git a/lib/sbi/sbi_ecall_sse.c b/lib/sbi/sbi_ecall_sse.c
index 303c0f2d..b7dbf072 100644
--- a/lib/sbi/sbi_ecall_sse.c
+++ b/lib/sbi/sbi_ecall_sse.c
@@ -59,7 +59,6 @@ struct sbi_ecall_extension ecall_sse = {
 	.name			= "sse",
 	.extid_start		= SBI_EXT_SSE,
 	.extid_end		= SBI_EXT_SSE,
-	.experimental		= true,
 	.register_extensions	= sbi_ecall_sse_register_extensions,
 	.handle			= sbi_ecall_sse_handler,
 };
-- 
2.43.0


-- 
opensbi mailing list
opensbi@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/opensbi

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] include: sbi: Change SBI spec version to 3.0
  2025-05-16 12:28 [PATCH] include: sbi: Change SBI spec version to 3.0 Anup Patel
@ 2025-05-16 18:06 ` Atish Patra
  2025-05-19 12:38 ` Clément Léger
  2025-05-20  8:22 ` Anup Patel
  2 siblings, 0 replies; 4+ messages in thread
From: Atish Patra @ 2025-05-16 18:06 UTC (permalink / raw)
  To: Anup Patel; +Cc: Andrew Jones, Anup Patel, opensbi

On 5/16/25 5:28 AM, Anup Patel wrote:
> Now that SBI v3.0 specification is frozen, change runtime SBI version
> implemented by OpenSBI to v3.0. Also, mark extensions defined by the
> SBI v3.0 specification as non-experimental.
> 
> Signed-off-by: Anup Patel <apatel@ventanamicro.com>
> ---
>   include/sbi/sbi_ecall.h  | 2 +-
>   lib/sbi/sbi_ecall_dbtr.c | 1 -
>   lib/sbi/sbi_ecall_fwft.c | 1 -
>   lib/sbi/sbi_ecall_mpxy.c | 1 -
>   lib/sbi/sbi_ecall_sse.c  | 1 -
>   5 files changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/include/sbi/sbi_ecall.h b/include/sbi/sbi_ecall.h
> index 88a69bb3..15400e5a 100644
> --- a/include/sbi/sbi_ecall.h
> +++ b/include/sbi/sbi_ecall.h
> @@ -13,7 +13,7 @@
>   #include <sbi/sbi_types.h>
>   #include <sbi/sbi_list.h>
>   
> -#define SBI_ECALL_VERSION_MAJOR		2
> +#define SBI_ECALL_VERSION_MAJOR		3
>   #define SBI_ECALL_VERSION_MINOR		0
>   #define SBI_OPENSBI_IMPID		1
>   
> diff --git a/lib/sbi/sbi_ecall_dbtr.c b/lib/sbi/sbi_ecall_dbtr.c
> index d28fd396..9c496c42 100644
> --- a/lib/sbi/sbi_ecall_dbtr.c
> +++ b/lib/sbi/sbi_ecall_dbtr.c
> @@ -69,7 +69,6 @@ struct sbi_ecall_extension ecall_dbtr = {
>   	.name			= "dbtr",
>   	.extid_start		= SBI_EXT_DBTR,
>   	.extid_end		= SBI_EXT_DBTR,
> -	.experimental		= true,
>   	.handle			= sbi_ecall_dbtr_handler,
>   	.register_extensions	= sbi_ecall_dbtr_register_extensions,
>   };
> diff --git a/lib/sbi/sbi_ecall_fwft.c b/lib/sbi/sbi_ecall_fwft.c
> index 3519285e..2277405e 100644
> --- a/lib/sbi/sbi_ecall_fwft.c
> +++ b/lib/sbi/sbi_ecall_fwft.c
> @@ -45,7 +45,6 @@ struct sbi_ecall_extension ecall_fwft = {
>   	.name			= "fwft",
>   	.extid_start		= SBI_EXT_FWFT,
>   	.extid_end		= SBI_EXT_FWFT,
> -	.experimental		= true,
>   	.register_extensions	= sbi_ecall_fwft_register_extensions,
>   	.handle			= sbi_ecall_fwft_handler,
>   };
> diff --git a/lib/sbi/sbi_ecall_mpxy.c b/lib/sbi/sbi_ecall_mpxy.c
> index 03ac71b2..0a5e3525 100644
> --- a/lib/sbi/sbi_ecall_mpxy.c
> +++ b/lib/sbi/sbi_ecall_mpxy.c
> @@ -67,7 +67,6 @@ struct sbi_ecall_extension ecall_mpxy = {
>   	.name			= "mpxy",
>   	.extid_start		= SBI_EXT_MPXY,
>   	.extid_end		= SBI_EXT_MPXY,
> -	.experimental		= true,
>   	.register_extensions	= sbi_ecall_mpxy_register_extensions,
>   	.handle			= sbi_ecall_mpxy_handler,
>   };
> diff --git a/lib/sbi/sbi_ecall_sse.c b/lib/sbi/sbi_ecall_sse.c
> index 303c0f2d..b7dbf072 100644
> --- a/lib/sbi/sbi_ecall_sse.c
> +++ b/lib/sbi/sbi_ecall_sse.c
> @@ -59,7 +59,6 @@ struct sbi_ecall_extension ecall_sse = {
>   	.name			= "sse",
>   	.extid_start		= SBI_EXT_SSE,
>   	.extid_end		= SBI_EXT_SSE,
> -	.experimental		= true,
>   	.register_extensions	= sbi_ecall_sse_register_extensions,
>   	.handle			= sbi_ecall_sse_handler,
>   };

LGTM.
Reviewed-by: Atish Patra <atishp@rivosinc.com>

-- 
opensbi mailing list
opensbi@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/opensbi

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] include: sbi: Change SBI spec version to 3.0
  2025-05-16 12:28 [PATCH] include: sbi: Change SBI spec version to 3.0 Anup Patel
  2025-05-16 18:06 ` Atish Patra
@ 2025-05-19 12:38 ` Clément Léger
  2025-05-20  8:22 ` Anup Patel
  2 siblings, 0 replies; 4+ messages in thread
From: Clément Léger @ 2025-05-19 12:38 UTC (permalink / raw)
  To: Anup Patel, Atish Patra; +Cc: Andrew Jones, Anup Patel, opensbi



On 16/05/2025 14:28, Anup Patel wrote:
> Now that SBI v3.0 specification is frozen, change runtime SBI version
> implemented by OpenSBI to v3.0. Also, mark extensions defined by the
> SBI v3.0 specification as non-experimental.
> 
> Signed-off-by: Anup Patel <apatel@ventanamicro.com>
> ---
>  include/sbi/sbi_ecall.h  | 2 +-
>  lib/sbi/sbi_ecall_dbtr.c | 1 -
>  lib/sbi/sbi_ecall_fwft.c | 1 -
>  lib/sbi/sbi_ecall_mpxy.c | 1 -
>  lib/sbi/sbi_ecall_sse.c  | 1 -
>  5 files changed, 1 insertion(+), 5 deletions(-)
> 
> diff --git a/include/sbi/sbi_ecall.h b/include/sbi/sbi_ecall.h
> index 88a69bb3..15400e5a 100644
> --- a/include/sbi/sbi_ecall.h
> +++ b/include/sbi/sbi_ecall.h
> @@ -13,7 +13,7 @@
>  #include <sbi/sbi_types.h>
>  #include <sbi/sbi_list.h>
>  
> -#define SBI_ECALL_VERSION_MAJOR		2
> +#define SBI_ECALL_VERSION_MAJOR		3
>  #define SBI_ECALL_VERSION_MINOR		0
>  #define SBI_OPENSBI_IMPID		1
>  
> diff --git a/lib/sbi/sbi_ecall_dbtr.c b/lib/sbi/sbi_ecall_dbtr.c
> index d28fd396..9c496c42 100644
> --- a/lib/sbi/sbi_ecall_dbtr.c
> +++ b/lib/sbi/sbi_ecall_dbtr.c
> @@ -69,7 +69,6 @@ struct sbi_ecall_extension ecall_dbtr = {
>  	.name			= "dbtr",
>  	.extid_start		= SBI_EXT_DBTR,
>  	.extid_end		= SBI_EXT_DBTR,
> -	.experimental		= true,
>  	.handle			= sbi_ecall_dbtr_handler,
>  	.register_extensions	= sbi_ecall_dbtr_register_extensions,
>  };
> diff --git a/lib/sbi/sbi_ecall_fwft.c b/lib/sbi/sbi_ecall_fwft.c
> index 3519285e..2277405e 100644
> --- a/lib/sbi/sbi_ecall_fwft.c
> +++ b/lib/sbi/sbi_ecall_fwft.c
> @@ -45,7 +45,6 @@ struct sbi_ecall_extension ecall_fwft = {
>  	.name			= "fwft",
>  	.extid_start		= SBI_EXT_FWFT,
>  	.extid_end		= SBI_EXT_FWFT,
> -	.experimental		= true,
>  	.register_extensions	= sbi_ecall_fwft_register_extensions,
>  	.handle			= sbi_ecall_fwft_handler,
>  };
> diff --git a/lib/sbi/sbi_ecall_mpxy.c b/lib/sbi/sbi_ecall_mpxy.c
> index 03ac71b2..0a5e3525 100644
> --- a/lib/sbi/sbi_ecall_mpxy.c
> +++ b/lib/sbi/sbi_ecall_mpxy.c
> @@ -67,7 +67,6 @@ struct sbi_ecall_extension ecall_mpxy = {
>  	.name			= "mpxy",
>  	.extid_start		= SBI_EXT_MPXY,
>  	.extid_end		= SBI_EXT_MPXY,
> -	.experimental		= true,
>  	.register_extensions	= sbi_ecall_mpxy_register_extensions,
>  	.handle			= sbi_ecall_mpxy_handler,
>  };
> diff --git a/lib/sbi/sbi_ecall_sse.c b/lib/sbi/sbi_ecall_sse.c
> index 303c0f2d..b7dbf072 100644
> --- a/lib/sbi/sbi_ecall_sse.c
> +++ b/lib/sbi/sbi_ecall_sse.c
> @@ -59,7 +59,6 @@ struct sbi_ecall_extension ecall_sse = {
>  	.name			= "sse",
>  	.extid_start		= SBI_EXT_SSE,
>  	.extid_end		= SBI_EXT_SSE,
> -	.experimental		= true,
>  	.register_extensions	= sbi_ecall_sse_register_extensions,
>  	.handle			= sbi_ecall_sse_handler,
>  };

Hi Anup,

Reviewed-by: Clément Léger <cleger@rivosinc.com>

Thanks,

Clément


-- 
opensbi mailing list
opensbi@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/opensbi

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] include: sbi: Change SBI spec version to 3.0
  2025-05-16 12:28 [PATCH] include: sbi: Change SBI spec version to 3.0 Anup Patel
  2025-05-16 18:06 ` Atish Patra
  2025-05-19 12:38 ` Clément Léger
@ 2025-05-20  8:22 ` Anup Patel
  2 siblings, 0 replies; 4+ messages in thread
From: Anup Patel @ 2025-05-20  8:22 UTC (permalink / raw)
  To: Anup Patel; +Cc: Atish Patra, Andrew Jones, opensbi

On Fri, May 16, 2025 at 5:58 PM Anup Patel <apatel@ventanamicro.com> wrote:
>
> Now that SBI v3.0 specification is frozen, change runtime SBI version
> implemented by OpenSBI to v3.0. Also, mark extensions defined by the
> SBI v3.0 specification as non-experimental.
>
> Signed-off-by: Anup Patel <apatel@ventanamicro.com>

Applied this patch to the riscv/opensbi repo.

Thanks,
Anup

> ---
>  include/sbi/sbi_ecall.h  | 2 +-
>  lib/sbi/sbi_ecall_dbtr.c | 1 -
>  lib/sbi/sbi_ecall_fwft.c | 1 -
>  lib/sbi/sbi_ecall_mpxy.c | 1 -
>  lib/sbi/sbi_ecall_sse.c  | 1 -
>  5 files changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/include/sbi/sbi_ecall.h b/include/sbi/sbi_ecall.h
> index 88a69bb3..15400e5a 100644
> --- a/include/sbi/sbi_ecall.h
> +++ b/include/sbi/sbi_ecall.h
> @@ -13,7 +13,7 @@
>  #include <sbi/sbi_types.h>
>  #include <sbi/sbi_list.h>
>
> -#define SBI_ECALL_VERSION_MAJOR                2
> +#define SBI_ECALL_VERSION_MAJOR                3
>  #define SBI_ECALL_VERSION_MINOR                0
>  #define SBI_OPENSBI_IMPID              1
>
> diff --git a/lib/sbi/sbi_ecall_dbtr.c b/lib/sbi/sbi_ecall_dbtr.c
> index d28fd396..9c496c42 100644
> --- a/lib/sbi/sbi_ecall_dbtr.c
> +++ b/lib/sbi/sbi_ecall_dbtr.c
> @@ -69,7 +69,6 @@ struct sbi_ecall_extension ecall_dbtr = {
>         .name                   = "dbtr",
>         .extid_start            = SBI_EXT_DBTR,
>         .extid_end              = SBI_EXT_DBTR,
> -       .experimental           = true,
>         .handle                 = sbi_ecall_dbtr_handler,
>         .register_extensions    = sbi_ecall_dbtr_register_extensions,
>  };
> diff --git a/lib/sbi/sbi_ecall_fwft.c b/lib/sbi/sbi_ecall_fwft.c
> index 3519285e..2277405e 100644
> --- a/lib/sbi/sbi_ecall_fwft.c
> +++ b/lib/sbi/sbi_ecall_fwft.c
> @@ -45,7 +45,6 @@ struct sbi_ecall_extension ecall_fwft = {
>         .name                   = "fwft",
>         .extid_start            = SBI_EXT_FWFT,
>         .extid_end              = SBI_EXT_FWFT,
> -       .experimental           = true,
>         .register_extensions    = sbi_ecall_fwft_register_extensions,
>         .handle                 = sbi_ecall_fwft_handler,
>  };
> diff --git a/lib/sbi/sbi_ecall_mpxy.c b/lib/sbi/sbi_ecall_mpxy.c
> index 03ac71b2..0a5e3525 100644
> --- a/lib/sbi/sbi_ecall_mpxy.c
> +++ b/lib/sbi/sbi_ecall_mpxy.c
> @@ -67,7 +67,6 @@ struct sbi_ecall_extension ecall_mpxy = {
>         .name                   = "mpxy",
>         .extid_start            = SBI_EXT_MPXY,
>         .extid_end              = SBI_EXT_MPXY,
> -       .experimental           = true,
>         .register_extensions    = sbi_ecall_mpxy_register_extensions,
>         .handle                 = sbi_ecall_mpxy_handler,
>  };
> diff --git a/lib/sbi/sbi_ecall_sse.c b/lib/sbi/sbi_ecall_sse.c
> index 303c0f2d..b7dbf072 100644
> --- a/lib/sbi/sbi_ecall_sse.c
> +++ b/lib/sbi/sbi_ecall_sse.c
> @@ -59,7 +59,6 @@ struct sbi_ecall_extension ecall_sse = {
>         .name                   = "sse",
>         .extid_start            = SBI_EXT_SSE,
>         .extid_end              = SBI_EXT_SSE,
> -       .experimental           = true,
>         .register_extensions    = sbi_ecall_sse_register_extensions,
>         .handle                 = sbi_ecall_sse_handler,
>  };
> --
> 2.43.0
>

-- 
opensbi mailing list
opensbi@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/opensbi

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-05-20  8:22 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-16 12:28 [PATCH] include: sbi: Change SBI spec version to 3.0 Anup Patel
2025-05-16 18:06 ` Atish Patra
2025-05-19 12:38 ` Clément Léger
2025-05-20  8:22 ` Anup Patel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.