On Fri, Nov 26, 2021 at 9:42 PM Paolo Bonzini <pbonzini@xxxxxxxxxx> wrote: > > On 11/26/21 16:40, Anup Patel wrote: > > +static int kvm_sbi_ext_expevend_handler(struct kvm_vcpu *vcpu, > > + struct kvm_run *run, > > + unsigned long *out_val, > > + struct kvm_cpu_trap *utrap, > > + bool *exit) > > Doesn't really matter what this is used for, it's a handler that > forwards. So you can name it kvm_sbi_ext_forward_handler. Sure, I will rename this to kvm_sbi_ext_forward_handler() Regards, Anup > > Paolo > > > +{ > > + /* > > + * Both SBI experimental and vendor extensions are > > + * unconditionally forwarded to userspace. > > + */ > > + kvm_riscv_vcpu_sbi_forward(vcpu, run); > > + *exit = true; > > + return 0; > > +} > > + >