xsm: streamline xsm_default_action() The privileges being strongly ordered is better reflected by using fall through within the respective switch statement. Signed-off-by: Jan Beulich --- a/xen/include/xsm/dummy.h +++ b/xen/include/xsm/dummy.h @@ -59,20 +59,14 @@ static always_inline int xsm_default_act switch ( action ) { case XSM_HOOK: return 0; - case XSM_DM_PRIV: - if ( src->is_privileged ) - return 0; - if ( target && src->target == target ) - return 0; - return -EPERM; case XSM_TARGET: if ( src == target ) return 0; - if ( src->is_privileged ) - return 0; + /* fall through */ + case XSM_DM_PRIV: if ( target && src->target == target ) return 0; - return -EPERM; + /* fall through */ case XSM_PRIV: if ( src->is_privileged ) return 0;