[PATCH v3] lib: sbi: Refine the settings for switching to Virtual Supervisor Mode.

[Pope B.Lei]

Dear all:

    I apologize for starting a new topic, but I'm unsure how to
proceed without doing so. Below is the sequence of commands I've used.
Could you please suggest any improvements?

Generating the Patch:

bashCopy code
git format-patch -v3 -M -s origin/master

Sending the Patch:

vbnetCopy code
git send-mail --to opensbi at lists.infradead.org
v3-0001-lib-sbi-Refine-the-settings-for-switching-to-Virt.patch

Thank you for your guidance!

On Wed, Apr 10, 2024 at 10:05 AM Pope B.Lei <popeblei at gmail.com> wrote:
>
> Although Mstatus.MPV is set, before executing mret, access to VS mode
> registers should use the actual register addresses, not the pseudonyms
> of S registers.
>
> Signed-off-by: Pope B.Lei <popeblei at gmail.com>
> ---
> V1 -> V2: Simplify the if condition
> V2 -> V3: Format indentation to tabs.
>  lib/sbi/sbi_hart.c | 15 +++++++++++----
>  1 file changed, 11 insertions(+), 4 deletions(-)
>
> diff --git a/lib/sbi/sbi_hart.c b/lib/sbi/sbi_hart.c
> index 3d13694..ea75e74 100644
> --- a/lib/sbi/sbi_hart.c
> +++ b/lib/sbi/sbi_hart.c
> @@ -1041,10 +1041,17 @@ sbi_hart_switch_mode(unsigned long arg0, unsigned long arg1,
>         csr_write(CSR_MEPC, next_addr);
>
>         if (next_mode == PRV_S) {
> -               csr_write(CSR_STVEC, next_addr);
> -               csr_write(CSR_SSCRATCH, 0);
> -               csr_write(CSR_SIE, 0);
> -               csr_write(CSR_SATP, 0);
> +               if (next_virt) {
> +                       csr_write(CSR_VSTVEC, next_addr);
> +                       csr_write(CSR_VSSCRATCH, 0);
> +                       csr_write(CSR_VSIE, 0);
> +                       csr_write(CSR_VSATP, 0);
> +               } else {
> +                       csr_write(CSR_STVEC, next_addr);
> +                       csr_write(CSR_SSCRATCH, 0);
> +                       csr_write(CSR_SIE, 0);
> +                       csr_write(CSR_SATP, 0);
> +               }
>         } else if (next_mode == PRV_U) {
>                 if (misa_extension('N')) {
>                         csr_write(CSR_UTVEC, next_addr);
> --
> 2.35.2.windows.1
>