On Mon, May 24, 2021, Jim Mattson wrote: > On Mon, May 24, 2021 at 4:10 PM Sean Christopherson <seanjc@xxxxxxxxxx> wrote: > > > > On Mon, May 24, 2021, Paolo Bonzini wrote: > > > On 24/05/21 18:39, Jim Mattson wrote: > > > > Without this patch, the accompanying selftest never wakes up from HLT > > > > in L2. If you can get the selftest to work without this patch, feel > > > > free to drop it. > > > > > > Ok, that's a pretty good reason. I'll try to debug it. > > > > I don't think there's any debug necessary, the hack of unconditionally calling > > kvm_check_nested_events() in kvm_vcpu_running() ... > > We don't unconditionally call kvm_check_nested_events() in > kvm_vcpu_running(). We still call kvm_check_nested_events() only when > is_guest_mode(vcpu). The only change introduced in this patch is that > we stop ignoring the result. Doh, sorry, bad use of "unconditionally". I meant "unconditionally when in L2". :-)
