lkml.org 
[lkml]   [2015]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 3.10 16/17] kvm: vmx: handle invvpid vm exit gracefully
    Date
    3.10-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Petr Matousek <pmatouse@redhat.com>

    commit a642fc305053cc1c6e47e4f4df327895747ab485 upstream.

    On systems with invvpid instruction support (corresponding bit in
    IA32_VMX_EPT_VPID_CAP MSR is set) guest invocation of invvpid
    causes vm exit, which is currently not handled and results in
    propagation of unknown exit to userspace.

    Fix this by installing an invvpid vm exit handler.

    This is CVE-2014-3646.

    Cc: stable@vger.kernel.org
    Signed-off-by: Petr Matousek <pmatouse@redhat.com>
    Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
    [wangkai: Backport to 3.10: adjust context]
    Signed-off-by: Wang Kai <morgan.wang@huawei.com>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
    ---
    arch/x86/include/uapi/asm/vmx.h | 2 ++
    arch/x86/kvm/vmx.c | 9 ++++++++-
    2 files changed, 10 insertions(+), 1 deletion(-)

    --- a/arch/x86/include/uapi/asm/vmx.h
    +++ b/arch/x86/include/uapi/asm/vmx.h
    @@ -67,6 +67,7 @@
    #define EXIT_REASON_EPT_MISCONFIG 49
    #define EXIT_REASON_INVEPT 50
    #define EXIT_REASON_PREEMPTION_TIMER 52
    +#define EXIT_REASON_INVVPID 53
    #define EXIT_REASON_WBINVD 54
    #define EXIT_REASON_XSETBV 55
    #define EXIT_REASON_APIC_WRITE 56
    @@ -112,6 +113,7 @@
    { EXIT_REASON_EOI_INDUCED, "EOI_INDUCED" }, \
    { EXIT_REASON_INVALID_STATE, "INVALID_STATE" }, \
    { EXIT_REASON_INVD, "INVD" }, \
    + { EXIT_REASON_INVVPID, "INVVPID" }, \
    { EXIT_REASON_INVPCID, "INVPCID" }, \
    { EXIT_REASON_PREEMPTION_TIMER, "PREEMPTION_TIMER" }

    --- a/arch/x86/kvm/vmx.c
    +++ b/arch/x86/kvm/vmx.c
    @@ -6248,6 +6248,12 @@ static int handle_invept(struct kvm_vcpu
    return 1;
    }

    +static int handle_invvpid(struct kvm_vcpu *vcpu)
    +{
    + kvm_queue_exception(vcpu, UD_VECTOR);
    + return 1;
    +}
    +
    /*
    * The exit handlers return 1 if the exit was handled fully and guest execution
    * may resume. Otherwise they set the kvm_run parameter to indicate what needs
    @@ -6293,6 +6299,7 @@ static int (*const kvm_vmx_exit_handlers
    [EXIT_REASON_MWAIT_INSTRUCTION] = handle_invalid_op,
    [EXIT_REASON_MONITOR_INSTRUCTION] = handle_invalid_op,
    [EXIT_REASON_INVEPT] = handle_invept,
    + [EXIT_REASON_INVVPID] = handle_invvpid,
    };

    static const int kvm_vmx_max_exit_handlers =
    @@ -6519,7 +6526,7 @@ static bool nested_vmx_exit_handled(stru
    case EXIT_REASON_VMPTRST: case EXIT_REASON_VMREAD:
    case EXIT_REASON_VMRESUME: case EXIT_REASON_VMWRITE:
    case EXIT_REASON_VMOFF: case EXIT_REASON_VMON:
    - case EXIT_REASON_INVEPT:
    + case EXIT_REASON_INVEPT: case EXIT_REASON_INVVPID:
    /*
    * VMX instructions trap unconditionally. This allows L1 to
    * emulate them for its L2 guest, i.e., allows 3-level nesting!



    \
     
     \ /
      Last update: 2015-02-09 10:21    [W:4.203 / U:0.260 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site