Avi Kivity wrote:
Ingo Molnar wrote:
* Avi Kivity <avi@xxxxxxxxxxxx> wrote:IIRC the accounting code clears it, but yes, it may not have been called
This bit can go; for the external module I can add it back inhm:
external-module-compat.h. No need to pollute mainline with backward
compatibility stuff.
static inline void kvm_guest_enter(void)
{
current->flags |= PF_VCPU;
}
static inline void kvm_guest_exit(void)
{
}
shouldnt PF_VCPU be cleared in kvm_guest_exit()?
at all, so clearing it here is needed.
No, It must not be cleared here because we can't enter in the accounting code
between kvm_guest_enter(void) and kvm_guest_exit(void).
This is why the accounting code clears it.