On 26.01.2022 02:03, Stefano Stabellini wrote: > Are you guys OK with something like this?
With proper proof that this isn't going to regress anything else, maybe. But ... > --- a/xen/include/xsm/dummy.h > +++ b/xen/include/xsm/dummy.h > @@ -92,7 +92,9 @@ static always_inline int xsm_default_action( > return 0; > /* fall through */ > case XSM_PRIV: > - if ( is_control_domain(src) ) > + if ( is_control_domain(src) || > + src->domain_id == DOMID_IDLE || > + src->domain_id == DOMID_XEN ) > return 0; ... my first question would be under what circumstances you might observe DOMID_XEN here and hence why this check is there. Jan
