On 04/02/2015 10:44, Christian Borntraeger wrote: > +static void kvm_s390_set_crycb_format(struct kvm *kvm) > +{ > + kvm->arch.crypto.crycbd = (__u32)(unsigned long) kvm->arch.crypto.crycb; > + > + if (kvm_s390_apxa_installed()) > + kvm->arch.crypto.crycbd |= CRYCB_FORMAT2; > + else > + kvm->arch.crypto.crycbd |= CRYCB_FORMAT1; > +} > + > static int kvm_s390_crypto_init(struct kvm *kvm) > { > if (!test_vfacility(76)) > @@ -663,8 +709,7 @@ static int kvm_s390_crypto_init(struct kvm *kvm) > if (!kvm->arch.crypto.crycb) > return -ENOMEM; > > - kvm->arch.crypto.crycbd = (__u32) (unsigned long) kvm->arch.crypto.crycb | > - CRYCB_FORMAT1; > + kvm_s390_set_crycb_format(kvm); Could this also be part of the "things" that KVM can choose to enable, even though it's not a facility? Paolo -- 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