diff options
author | Anup Patel <anup.patel@wdc.com> | 2020-12-28 15:55:18 +0300 |
---|---|---|
committer | Anup Patel <anup@brainfault.org> | 2021-01-07 07:23:14 +0300 |
commit | 80bc5065bb67f9d118b83f64cbe96f2e3e1bc0c6 (patch) | |
tree | 034b077172404c2a0b991bb63158af916f9cfea7 /lib/sbi/sbi_ecall_legacy.c | |
parent | 7dcb1e1753e9c5daec0580779ea8c31778bff152 (diff) | |
download | opensbi-80bc5065bb67f9d118b83f64cbe96f2e3e1bc0c6.tar.xz |
lib: sbi: Replace args with trap registers in ecall handler
We had added args pointer in ecall handler to ensure that ecall
handler only implements functionality and does not deal with
SBI calling convention. This also helped us to keep SBI calling
convention related code in one place at sbi_ecall_handler().
The Keystone Enclavce project needs access to the trap regsiters
in their ecall handler so that they can context switch enclaves
in custom SBI calls. To help the Keystone Enclave project, we
replace the args pointer in ecall handler parameter with a const
pointer to trap registers.
Signed-off-by: Anup Patel <anup.patel@wdc.com>
Reviewed-by: Atish Patra <atish.patra@wdc.com>
Diffstat (limited to 'lib/sbi/sbi_ecall_legacy.c')
-rw-r--r-- | lib/sbi/sbi_ecall_legacy.c | 25 |
1 files changed, 13 insertions, 12 deletions
diff --git a/lib/sbi/sbi_ecall_legacy.c b/lib/sbi/sbi_ecall_legacy.c index 7844fbb..8afeb00 100644 --- a/lib/sbi/sbi_ecall_legacy.c +++ b/lib/sbi/sbi_ecall_legacy.c @@ -43,7 +43,8 @@ static int sbi_load_hart_mask_unpriv(ulong *pmask, ulong *hmask, } static int sbi_ecall_legacy_handler(unsigned long extid, unsigned long funcid, - unsigned long *args, unsigned long *out_val, + const struct sbi_trap_regs *regs, + unsigned long *out_val, struct sbi_trap_info *out_trap) { int ret = 0; @@ -54,13 +55,13 @@ static int sbi_ecall_legacy_handler(unsigned long extid, unsigned long funcid, switch (extid) { case SBI_EXT_0_1_SET_TIMER: #if __riscv_xlen == 32 - sbi_timer_event_start((((u64)args[1] << 32) | (u64)args[0])); + sbi_timer_event_start((((u64)regs->a1 << 32) | (u64)regs->a0)); #else - sbi_timer_event_start((u64)args[0]); + sbi_timer_event_start((u64)regs->a0); #endif break; case SBI_EXT_0_1_CONSOLE_PUTCHAR: - sbi_putc(args[0]); + sbi_putc(regs->a0); break; case SBI_EXT_0_1_CONSOLE_GETCHAR: ret = sbi_getc(); @@ -69,13 +70,13 @@ static int sbi_ecall_legacy_handler(unsigned long extid, unsigned long funcid, sbi_ipi_clear_smode(); break; case SBI_EXT_0_1_SEND_IPI: - ret = sbi_load_hart_mask_unpriv((ulong *)args[0], + ret = sbi_load_hart_mask_unpriv((ulong *)regs->a0, &hmask, out_trap); if (ret != SBI_ETRAP) ret = sbi_ipi_send_smode(hmask, 0); break; case SBI_EXT_0_1_REMOTE_FENCE_I: - ret = sbi_load_hart_mask_unpriv((ulong *)args[0], + ret = sbi_load_hart_mask_unpriv((ulong *)regs->a0, &hmask, out_trap); if (ret != SBI_ETRAP) { SBI_TLB_INFO_INIT(&tlb_info, 0, 0, 0, 0, @@ -84,21 +85,21 @@ static int sbi_ecall_legacy_handler(unsigned long extid, unsigned long funcid, } break; case SBI_EXT_0_1_REMOTE_SFENCE_VMA: - ret = sbi_load_hart_mask_unpriv((ulong *)args[0], + ret = sbi_load_hart_mask_unpriv((ulong *)regs->a0, &hmask, out_trap); if (ret != SBI_ETRAP) { - SBI_TLB_INFO_INIT(&tlb_info, args[1], args[2], 0, 0, + SBI_TLB_INFO_INIT(&tlb_info, regs->a1, regs->a2, 0, 0, SBI_TLB_FLUSH_VMA, source_hart); ret = sbi_tlb_request(hmask, 0, &tlb_info); } break; case SBI_EXT_0_1_REMOTE_SFENCE_VMA_ASID: - ret = sbi_load_hart_mask_unpriv((ulong *)args[0], + ret = sbi_load_hart_mask_unpriv((ulong *)regs->a0, &hmask, out_trap); if (ret != SBI_ETRAP) { - SBI_TLB_INFO_INIT(&tlb_info, args[1], args[2], args[3], - 0, SBI_TLB_FLUSH_VMA_ASID, - source_hart); + SBI_TLB_INFO_INIT(&tlb_info, regs->a1, + regs->a2, regs->a3, 0, + SBI_TLB_FLUSH_VMA_ASID, source_hart); ret = sbi_tlb_request(hmask, 0, &tlb_info); } break; |