On Tue, May 14, 2013 at 03:13:33PM +0100, Marc Zyngier wrote: > Implements helpers for dealing with the EL2 syndrome register as > well as accessing the vcpu registers. > > Reviewed-by: Christopher Covington <[email protected]> > Signed-off-by: Marc Zyngier <[email protected]> ... > +static inline bool kvm_vcpu_dabt_isvalid(const struct kvm_vcpu *vcpu) > +{ > + return !!(kvm_vcpu_get_hsr(vcpu) & ESR_EL2_ISV); > +}
You kept these '!!' ;). BTW, would the compiler handle the conversion between the integer and bool here? Either way is fine by me: Reviewed-by: Catalin Marinas <[email protected]> -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
