On Sun, Sep 14, 2025 at 9:49 AM Jan Beulich <jbeul...@suse.com> wrote:
>
> On 14.09.2025 01:24, Tamas K Lengyel wrote:
> >>> +static inline bool vm_event_is_enabled(struct vcpu *v)
> >>> +{
> >>> +#ifdef CONFIG_VM_EVENT
> >>> + return v->arch.vm_event != NULL;
> >>
> >> Is "enabled" (in the function name) a good description of this condition,
> >> Tamas?
> >
> > Sure, sounds fine to me.
>
> That is the pointer alone being non-NULL identifies "enabled"? And not
> just e.g. "active" or "available" (can be enabled with further things
> set up)?
Nope, just that the struct is non-NULL means vm_event is enabled.
There is no meaningful distinction of enabled vs active vs available
in the contexts this is being checked.
Tamas
