On 4/20/22 14:05, Heiko Carstens wrote:
+static inline bool kvm_s390_topology_changed(struct kvm_vcpu *vcpu) +{ + if (!test_kvm_facility(vcpu->kvm, 11)) + return false; + + /* A new vCPU has been hotplugged */ + if (vcpu->arch.prev_cpu == S390_KVM_TOPOLOGY_NEW_CPU) + return true; + + /* The real CPU backing up the vCPU moved to another socket */ + if (cpumask_test_cpu(vcpu->cpu, + topology_core_cpumask(vcpu->arch.prev_cpu))) + return true; + + return false; +}This seems to be wrong. I'd guess that you need if (cpumask_test_cpu(vcpu->cpu, topology_core_cpumask(vcpu->arch.prev_cpu))) --> return false; --> return true;
/o\ I do not know if I can do this right at the end! Thanks, I send an update immediately Pierre -- Pierre Morel IBM Lab Boeblingen
