Re: [PATCH v1 10/15] xen/riscv: implement SBI legacy SET_TIMER support for guests

[Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>]

On 1/8/26 11:45 AM, Jan Beulich wrote:
> On 24.12.2025 18:03, Oleksii Kurochko wrote:
> > Add handling of the SBI_EXT_0_1_SET_TIMER function ID to the legacy
> > extension ecall handler. The handler now programs the vCPU’s virtual
> > timer via vtimer_set_timer() and returns SBI_SUCCESS.
> >
> > This enables guests using the legacy SBI timer interface to schedule
> > timer events correctly.
> >
> > Signed-off-by: Oleksii Kurochko <oleksii.kurochko@xxxxxxxxx>
> Acked-by: Jan Beulich <jbeulich@xxxxxxxx>
Thanks.

> What about the more modern timer extension, though?
Handling will be the same, as the API is identical:
  struct sbiret sbi_set_timer(uint64_t stime_value)

The only additional work needed is to add handling for the new extension
with EID|0x54494D45| (“TIME”), which was introduced in SBI v0.2.

Interestingly, we currently report to the guest that OpenSBI v0.2 is
available, so technically this modern timer extension should be usable.
However, the guest still tries to use the Legacy extension, as it has not
yet received an indication in Xen that the EID|0x54494D45| (“TIME”) isn't
implemented.

~ Oleksii