On Sun, May 04, 2014 at 10:48:52PM +0530, Aneesh Kumar K.V wrote: > This patch make sure we inherit the LE bit correctly in different case > so that we can run Little Endian distro in PR mode [snip] > +static void kvmppc_set_lpcr_pr(struct kvm_vcpu *vcpu, u64 new_lpcr) > +{ > + /* > + * If ILE (interrupt little-endian) has changed, update the > + * MSR_LE bit in the intr_msr for each vcpu in this vcore. > + */ > + if ((new_lpcr & LPCR_ILE) != (vcpu->arch.intr_msr & MSR_LE)) { Since LPCR_ILE != MSR_LE, this condition is always going to be true. I suggest you remove this if statement and just do the body unconditionally. Paul. -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html