On Sat, Nov 19, 2022, David Woodhouse wrote: > + /* > + * Use kvm_gpc_activate() here because if the runstate > + * area was configured in 32-bit mode and only extends > + * to the second page now because the guest changed to > + * 64-bit mode, the second GPC won't have been set up. > + */ > + if (kvm_gpc_activate(v->kvm, gpc2, NULL, KVM_HOST_USES_PFN, > + gpc1->gpa + user_len1, user_len2)) I believe kvm_gpc_activate() needs to be converted from write_lock_irq() to write_lock_irqsave() for this to be safe. Side topic, why do all of these flows disable IRQs?
