On Wed, Jan 30, 2019 at 05:24:41PM +0100, Paolo Bonzini wrote: > On 23/01/19 23:39, Sean Christopherson wrote: > > } else if (is_protmode(vcpu)) { > > Since we are at it, this "if (is_protmode(vcpu))" should probably be > removed, shouldn't it? Yep. It could be converted to a WARN if we wanted to be paranoid, but that seems a bit gratuitous.