diff mbox series

[1/1] include: sbi: SBI function IDs for RFENCE extension

Message ID 20210309113122.63148-1-xypron.glpk@gmx.de
State Accepted
Headers show
Series [1/1] include: sbi: SBI function IDs for RFENCE extension | expand

Commit Message

Heinrich Schuchardt March 9, 2021, 11:31 a.m. UTC
The SBI function IDs for RFENCE extension must match the SBI specification.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
The same bug exists in the Linux kernel:
https://lore.kernel.org/linux-riscv/20210306054801.18263-1-xypron.glpk@gmx.de/
---
 include/sbi/sbi_ecall_interface.h | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

--
2.30.1

Comments

Atish Patra March 9, 2021, 7:22 p.m. UTC | #1
On Tue, Mar 9, 2021 at 3:31 AM Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> The SBI function IDs for RFENCE extension must match the SBI specification.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> The same bug exists in the Linux kernel:
> https://lore.kernel.org/linux-riscv/20210306054801.18263-1-xypron.glpk@gmx.de/
> ---
>  include/sbi/sbi_ecall_interface.h | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/include/sbi/sbi_ecall_interface.h b/include/sbi/sbi_ecall_interface.h
> index 7fcb373..559a33e 100644
> --- a/include/sbi/sbi_ecall_interface.h
> +++ b/include/sbi/sbi_ecall_interface.h
> @@ -48,10 +48,10 @@
>  #define SBI_EXT_RFENCE_REMOTE_FENCE_I          0x0
>  #define SBI_EXT_RFENCE_REMOTE_SFENCE_VMA       0x1
>  #define SBI_EXT_RFENCE_REMOTE_SFENCE_VMA_ASID  0x2
> -#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA      0x3
> -#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA_VMID 0x4
> -#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA      0x5
> -#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA_ASID 0x6
> +#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA_VMID 0x3
> +#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA      0x4
> +#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA_ASID 0x5
> +#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA      0x6
>
>  /* SBI function IDs for HSM extension */
>  #define SBI_EXT_HSM_HART_START                 0x0
> --
> 2.30.1
>
>
> --
> opensbi mailing list
> opensbi@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi


Reviewed-by: Atish Patra <atish.patra@wdc.com>
Anup Patel March 12, 2021, 4:06 a.m. UTC | #2
> -----Original Message-----
> From: Atish Patra <atishp@atishpatra.org>
> Sent: 10 March 2021 00:53
> To: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Cc: OpenSBI <opensbi@lists.infradead.org>; Anup Patel
> <Anup.Patel@wdc.com>; Atish Patra <Atish.Patra@wdc.com>
> Subject: Re: [PATCH 1/1] include: sbi: SBI function IDs for RFENCE extension
> 
> On Tue, Mar 9, 2021 at 3:31 AM Heinrich Schuchardt <xypron.glpk@gmx.de>
> wrote:
> >
> > The SBI function IDs for RFENCE extension must match the SBI
> specification.
> >
> > Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> > ---
> > The same bug exists in the Linux kernel:
> > https://lore.kernel.org/linux-riscv/20210306054801.18263-1-
> xypron.glpk@gmx.de/
> > ---
> >  include/sbi/sbi_ecall_interface.h | 8 ++++----
> >  1 file changed, 4 insertions(+), 4 deletions(-)
> >
> > diff --git a/include/sbi/sbi_ecall_interface.h
> b/include/sbi/sbi_ecall_interface.h
> > index 7fcb373..559a33e 100644
> > --- a/include/sbi/sbi_ecall_interface.h
> > +++ b/include/sbi/sbi_ecall_interface.h
> > @@ -48,10 +48,10 @@
> >  #define SBI_EXT_RFENCE_REMOTE_FENCE_I          0x0
> >  #define SBI_EXT_RFENCE_REMOTE_SFENCE_VMA       0x1
> >  #define SBI_EXT_RFENCE_REMOTE_SFENCE_VMA_ASID  0x2
> > -#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA      0x3
> > -#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA_VMID 0x4
> > -#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA      0x5
> > -#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA_ASID 0x6
> > +#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA_VMID 0x3
> > +#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA      0x4
> > +#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA_ASID 0x5
> > +#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA      0x6
> >
> >  /* SBI function IDs for HSM extension */
> >  #define SBI_EXT_HSM_HART_START                 0x0
> > --
> > 2.30.1
> >
> >
> > --
> > opensbi mailing list
> > opensbi@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/opensbi
> 
> 
> Reviewed-by: Atish Patra <atish.patra@wdc.com>
> 

Applied this patch to the riscv/opensbi repo

Thanks,
Anup
diff mbox series

Patch

diff --git a/include/sbi/sbi_ecall_interface.h b/include/sbi/sbi_ecall_interface.h
index 7fcb373..559a33e 100644
--- a/include/sbi/sbi_ecall_interface.h
+++ b/include/sbi/sbi_ecall_interface.h
@@ -48,10 +48,10 @@ 
 #define SBI_EXT_RFENCE_REMOTE_FENCE_I		0x0
 #define SBI_EXT_RFENCE_REMOTE_SFENCE_VMA	0x1
 #define SBI_EXT_RFENCE_REMOTE_SFENCE_VMA_ASID	0x2
-#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA	0x3
-#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA_VMID	0x4
-#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA	0x5
-#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA_ASID	0x6
+#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA_VMID	0x3
+#define SBI_EXT_RFENCE_REMOTE_HFENCE_GVMA	0x4
+#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA_ASID	0x5
+#define SBI_EXT_RFENCE_REMOTE_HFENCE_VVMA	0x6

 /* SBI function IDs for HSM extension */
 #define SBI_EXT_HSM_HART_START			0x0