Re: [PATCH v8 3/9] xen: add bitmap to indicate per-domain state changes

[Jan Beulich <jbeulich@suse.com>]

On 04.02.2025 12:34, Juergen Gross wrote:
> @@ -1075,6 +1086,26 @@ static void clear_global_virq_handlers(struct domain *d)
>      }
>  }
>  
> +struct domain *lock_dom_exc_handler(void)
> +{
> +    struct domain *d;
> +
> +    d = get_global_virq_handler(VIRQ_DOM_EXC);

The Arm dom0less crash(es?) found by CI are apparently a result of this possibly
returning NULL (hardware_domain == NULL). Returning NULL from here also in this
case (and not just ...

> +    if ( unlikely(!get_domain(d)) )
> +        return NULL;

... in this one) would at least deal with the use from domain_changed_state(). I
didn't closely check other uses. Thoughts?

Jan