On 23/07/2025 12:57 pm, Teddy Astie wrote: > Le 23/07/2025 à 13:16, Andrew Cooper a écrit : >> On 23/07/2025 10:05 am, Teddy Astie wrote: >>> do_sched_op(SCHEDOP_yield) just calls vcpu_yield(). Remove the indirection >>> through the hypercall handler and use the function directly. >>> >>> Perform the same for SCHEDOP_block. >>> >>> Not a functional change. >>> >>> Signed-off-by: Teddy Astie <teddy.as...@vates.tech> >>> --- >>> v2: >>> - For SCHEDOP_block case: export and use vcpu_block_enable_events instead >> You need to adjust the commit message for this change, now that you're >> exporting vcpu_block_enable_events(). >> >> With that adjusted, Reviewed-by: Andrew Cooper <andrew.coop...@citrix.com> >> >> If there are no other changes needed, you can propose some updated >> wording here and it can be adjusted on commit. >> > Are you ok with > > Export vcpu_block_enable_events() to perform the same for SCHEDOP_block. > > (in place of "Perform the same for SCHEDOP_block") > > ?
Yeah, that looks fine. ~Andrew