>>> On 07.01.19 at 13:02, <[email protected]> wrote: > @@ -1472,10 +1468,7 @@ static int hvm_load_cpu_msrs(struct domain *d, > hvm_domain_context_t *h) > return -EOPNOTSUPP; > /* Checking finished */ > > - if ( hvm_funcs.load_msr ) > - err = hvm_funcs.load_msr(v, ctxt); > - > - for ( i = 0; !err && i < ctxt->count; ++i ) > + for ( i = 0; i < ctxt->count; ++i ) > { > switch ( ctxt->msr[i].index ) > {
err can become set to non-zero in the loop body further down from here, so I don't think you should alter the loop header. Jan _______________________________________________ Xen-devel mailing list [email protected] https://lists.xenproject.org/mailman/listinfo/xen-devel
