Message ID | 20240219074733.122080-1-weijiang.yang@intel.com |
---|---|
Headers |
Return-Path: <linux-kernel+bounces-70866-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:693c:2685:b0:108:e6aa:91d0 with SMTP id mn5csp1135864dyc; Sun, 18 Feb 2024 23:52:03 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCWoiD7Ftx85NWtu4MeTWPtFn1AOYD3E57lCvbwm2cG84MqNeBVsYG6luteLQxPF4kowFg39g0/MtknYYqx5mUuG48pf0A== X-Google-Smtp-Source: AGHT+IEJ6M2yLU3T8g/XgEXsIwDasVIJ/GUcE2OdrcZQ7XHNuXiSiYSbKz/bvUnLmF2GR5QHt1k/ X-Received: by 2002:a05:620a:f94:b0:787:6bfa:bde with SMTP id b20-20020a05620a0f9400b007876bfa0bdemr955958qkn.55.1708329123554; Sun, 18 Feb 2024 23:52:03 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1708329123; cv=pass; d=google.com; s=arc-20160816; b=BZL+s4vFOtXOy7/NzYBqQZBvCVfbvTCHuatWCC8fXFaaEXXh9LqBPETHfj6lJCh6Pf qDNfAnywCszGT7ylE0arwCBZfGQAAhL1JkyQQ5F4GRw5nAScjqYnkHs050dA79Wt3YNT PDmdoLVe3U780/bxZygjuIa8rhDhpe7yURcOz115COenhyZqYMTzGhBFToP5LYqikCwX J4cXc+/qpSQWn4ZqXqqxv0ISDzTPY+gMk/+cDKTL7REUnxZuiP2vQ1bQcfV5tU1+sas/ cib9JS8hyIZfTOdksRj709zpgxW/HSd13Jrx+vlHPq7JQxiLLtW0Qnvl/96JxTEN9rBT N/kg== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:cc:to :from:dkim-signature; bh=O77cuq5fdIGKR/DRKl3lgH6K0GvAOOS2/Vb03MHAxuY=; fh=6bYtQoKQxNJSYrJA1a9vseXH6qHZpRYO7L/7krtpXA0=; b=pgng97Mf2XRKdY0wZ23MEgzmOSzjcYfg8OjwHD/SBOohNhvM1n8tNyjHO130CcY6Yv Q5fP49eA686Wj0ovdHkJSf6/lELshaLT5f4qNaJPCT8+kG4KXK2r3g7H6oyeNwKY4reb vbGdsrzretUkZprDFIoW2OpcA+J4fTCp7fDwrh5VevwJZInrSPKMz/euqlMKq2evl77n Q6yo3yN9fM6bXqdz4giM7kfizXdbnXFILqa6vLszGvr5QmlMQOV3We/E6MqKrShjQdec F2RNlvg9ODhugmTbk+JeT0x85eUWZwu3BwjUwFvA08SaOnKMsTd0SkaRp20kJqwdtH7e tlTg==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=WLHSy36a; arc=pass (i=1 spf=pass spfdomain=intel.com dkim=pass dkdomain=intel.com dmarc=pass fromdomain=intel.com); spf=pass (google.com: domain of linux-kernel+bounces-70866-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-70866-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [147.75.199.223]) by mx.google.com with ESMTPS id y22-20020a05620a09d600b007873624b6ecsi5853977qky.139.2024.02.18.23.52.03 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 18 Feb 2024 23:52:03 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-70866-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223; Authentication-Results: mx.google.com; dkim=pass header.i=@intel.com header.s=Intel header.b=WLHSy36a; arc=pass (i=1 spf=pass spfdomain=intel.com dkim=pass dkdomain=intel.com dmarc=pass fromdomain=intel.com); spf=pass (google.com: domain of linux-kernel+bounces-70866-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) smtp.mailfrom="linux-kernel+bounces-70866-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 069351C211A9 for <ouuuleilei@gmail.com>; Mon, 19 Feb 2024 07:52:03 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 866742D60C; Mon, 19 Feb 2024 07:47:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="WLHSy36a" Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.16]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9721020DC3; Mon, 19 Feb 2024 07:47:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=198.175.65.16 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708328867; cv=none; b=VXLEPOz2lOseLXmDPM0wNCBOdkhTlRduyuppqMZc8yb77i+txmBHX3gwfr/ZJF06AOm3ljklX4CcmhG+xCVgDM3rL30rzGZ1gfQ7EbYruc7apWASxgKwu1pmEhfpO9Si4+tWvJC3FK7DPoUOUlYiUqXKyHv0RIIw0jkm0wU9rEQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708328867; c=relaxed/simple; bh=/YWn5/ObdOfhNg+Pusr/TniRTi2o6Z52ssQzMtoyqGQ=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=HbuZm1rVYidddWBG71VaImkwU2VUVP41/JQCbSKsj1G22Ln8xild4FQGDrEhXRZsyop9uKHW3MYg8Z2lsd4yElRJAtDA0PYXYF2Hrea1PBZGCWLGOHPSn3xeTTLOquUb5R7o1PLqVx5FnDRmCiyd7gDsQ4ccEvY9xQzwUWV2JsY= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=WLHSy36a; arc=none smtp.client-ip=198.175.65.16 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1708328864; x=1739864864; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=/YWn5/ObdOfhNg+Pusr/TniRTi2o6Z52ssQzMtoyqGQ=; b=WLHSy36a346dQPOy3TF2bRDmVIpo8cMRMagUKrXva9jJ1dZ+iPZ72lqa AXtRB9kN3XXS5vRJ6Hz7oS8gq5IDJuRtWPS92EiSG2p8AGCE7bnVXzrmj qq/znBK5803vApy7n+FJmvsqSWlpxKa1+KEsAosvCHpN3V67XrCV8yiqq ug3iMHR4by9n2cxvSqiVZewWx6H3wGxEW8mLPP1UdWsQhSa6H0jLgTOMA nIQklRKtz/pLq5Z5PbzOSYVz1FVqGLW64bZq7oGzdAhqS5WonXyQ4jQ/F /oUNMxnvCD2uqm9kprKZXWwngnepJL+rb/wF5LPnnWU3Dn9VtM5g/EUKA A==; X-IronPort-AV: E=McAfee;i="6600,9927,10988"; a="2535005" X-IronPort-AV: E=Sophos;i="6.06,170,1705392000"; d="scan'208";a="2535005" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orvoesa108.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Feb 2024 23:47:43 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10988"; a="826966056" X-IronPort-AV: E=Sophos;i="6.06,170,1705392000"; d="scan'208";a="826966056" Received: from jf.jf.intel.com (HELO jf.intel.com) ([10.165.9.183]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 18 Feb 2024 23:47:43 -0800 From: Yang Weijiang <weijiang.yang@intel.com> To: seanjc@google.com, pbonzini@redhat.com, dave.hansen@intel.com, x86@kernel.org, kvm@vger.kernel.org, linux-kernel@vger.kernel.org Cc: peterz@infradead.org, chao.gao@intel.com, rick.p.edgecombe@intel.com, mlevitsk@redhat.com, john.allen@amd.com, weijiang.yang@intel.com Subject: [PATCH v10 00/27] Enable CET Virtualization Date: Sun, 18 Feb 2024 23:47:06 -0800 Message-ID: <20240219074733.122080-1-weijiang.yang@intel.com> X-Mailer: git-send-email 2.43.0 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1791312918946612227 X-GMAIL-MSGID: 1791312918946612227 |
Series |
Enable CET Virtualization
|
|
Message
Yang, Weijiang
Feb. 19, 2024, 7:47 a.m. UTC
Control-flow Enforcement Technology (CET) is a kind of CPU feature used to prevent Return/CALL/Jump-Oriented Programming (ROP/COP/JOP) attacks. It provides two sub-features(SHSTK,IBT) to defend against ROP/COP/JOP style control-flow subversion attacks. Shadow Stack (SHSTK): A shadow stack is a second stack used exclusively for control transfer operations. The shadow stack is separate from the data/normal stack and can be enabled individually in user and kernel mode. When shadow stack is enabled, CALL pushes the return address on both the data and shadow stack. RET pops the return address from both stacks and compares them. If the return addresses from the two stacks do not match, the processor generates a #CP. Indirect Branch Tracking (IBT): IBT introduces new instruction(ENDBRANCH)to mark valid target addresses of indirect branches (CALL, JMP etc...). If an indirect branch is executed and the next instruction is _not_ an ENDBRANCH, the processor generates a #CP. These instruction behaves as a NOP on platforms that doesn't support CET. Dependency: ===================== CET native series for user mode shadow stack has already been merged in v6.6 mainline kernel. The first 7 kernel patches are prerequisites for this KVM patch series since guest CET user mode and supervisor mode states depends on kernel FPU framework to properly save/restore the states whenever FPU context switch is required, e.g., after VM-Exit and before vCPU thread exits to userspace. In this series, guest supervisor SHSTK mitigation solution isn't introduced for Intel platform therefore guest SSS_CET bit of CPUID(0x7,1):EDX[bit18] is cleared. Check SDM (Vol 1, Section 17.2.3) for details. CET states management: ====================== KVM cooperates with host kernel FPU framework to manage guest CET registers. With CET supervisor mode state support in this series, KVM can save/restore full guest CET xsave-managed states. CET user mode and supervisor mode xstates, i.e., MSR_IA32_{U_CET,PL3_SSP} and MSR_IA32_PL{0,1,2}, depend on host FPU framework to swap guest and host xstates. On VM-Exit, guest CET xstates are saved to guest fpu area and host CET xstates are loaded from task/thread context before vCPU returns to userspace, vice-versa on VM-Entry. See details in kvm_{load,put}_guest_fpu(). So guest CET xstates management depends on CET xstate bits(U_CET/S_CET bit) set in host XSS MSR. CET supervisor mode states are grouped into two categories : XSAVE-managed and non-XSAVE-managed, the former includes MSR_IA32_PL{0,1,2}_SSP and are controlled by CET supervisor mode bit(S_CET bit) in XSS, the later consists of MSR_IA32_S_CET and MSR_IA32_INTR_SSP_TBL. VMX introduces new VMCS fields, {GUEST|HOST}_{S_CET,SSP,INTR_SSP_TABL}, to facilitate guest/host non-XSAVES-managed states. When VMX CET entry/exit load bits are set, guest/host MSR_IA32_{S_CET,INTR_SSP_TBL,SSP} are loaded from equivalent fields at VM-Exit/Entry. With these new fields, such supervisor states require no addtional KVM save/reload actions. Tests: ====================== This series passed basic CET user shadow stack test and kernel IBT test in L1 and L2 guest. The patch series _has_ impact to existing vmx test cases in KVM-unit-tests,the failures have been fixed here[1]. One new selftest app[2] is introduced for testing CET MSRs accessibilities. Note, this series hasn't been tested on AMD platform yet. To run user SHSTK test and kernel IBT test in guest, an CET capable platform is required, e.g., Sapphire Rapids server, and follow below steps to build the binaries: 1. Host kernel: Apply this series to mainline kernel (>= v6.6) and build. 2. Guest kernel: Pull kernel (>= v6.6), opt-in CONFIG_X86_KERNEL_IBT and CONFIG_X86_USER_SHADOW_STACK options. Build with CET enabled gcc versions (>= 8.5.0). 3. Apply CET QEMU patches[3] before build mainline QEMU. Check kernel selftest test_shadow_stack_64 output: [INFO] new_ssp = 7f8c82100ff8, *new_ssp = 7f8c82101001 [INFO] changing ssp from 7f8c82900ff0 to 7f8c82100ff8 [INFO] ssp is now 7f8c82101000 [OK] Shadow stack pivot [OK] Shadow stack faults [INFO] Corrupting shadow stack [INFO] Generated shadow stack violation successfully [OK] Shadow stack violation test [INFO] Gup read -> shstk access success [INFO] Gup write -> shstk access success [INFO] Violation from normal write [INFO] Gup read -> write access success [INFO] Violation from normal write [INFO] Gup write -> write access success [INFO] Cow gup write -> write access success [OK] Shadow gup test [INFO] Violation from shstk access [OK] mprotect() test [SKIP] Userfaultfd unavailable. [OK] 32 bit test Check kernel IBT with dmesg | grep CET: CET detected: Indirect Branch Tracking enabled Changes in v10: ===================== 1. Add Reviewed-by tags from Chao and Rick. [Chao, Rick] 2. Use two bit flags to check CET guarded instructions in KVM emulator. [Chao] 3. Refine reset handling of xsave-managed guest FPU states. [Chao] 4. Add nested CET MSR sync when entry/exit-load-bit is not set. [Chao] 5. Other minor changes per comments from Chao and Rick. 6. Rebased on https://github.com/kvm-x86/linux commit: c0f8b0752b09 [1]: KVM-unit-tests fixup: https://lore.kernel.org/all/20230913235006.74172-1-weijiang.yang@intel.com/ [2]: Selftest for CET MSRs: https://lore.kernel.org/all/20230914064201.85605-1-weijiang.yang@intel.com/ [3]: QEMU patch: https://lore.kernel.org/all/20230720111445.99509-1-weijiang.yang@intel.com/ [4]: v9 patchset: https://lore.kernel.org/all/20240124024200.102792-1-weijiang.yang@intel.com/ Patch 1-7: Fixup patches for kernel xstate and enable CET supervisor xstate. Patch 8-11: Cleanup patches for KVM. Patch 12-15: Enable KVM XSS MSR support. Patch 16: Fault check for CR4.CET setting. Patch 17: Report CET MSRs to userspace. Patch 18: Introduce CET VMCS fields. Patch 19: Add SHSTK/IBT to KVM-governed framework.(to be deprecated) Patch 20: Emulate CET MSR access. Patch 21: Handle SSP at entry/exit to SMM. Patch 22: Set up CET MSR interception. Patch 23: Initialize host constant supervisor state. Patch 24: Enable CET virtualization settings. Patch 25-26: Add CET nested support. Patch 27: KVM emulation handling for branch instructions Sean Christopherson (4): x86/fpu/xstate: Always preserve non-user xfeatures/flags in __state_perm KVM: x86: Rework cpuid_get_supported_xcr0() to operate on vCPU data KVM: x86: Report XSS as to-be-saved if there are supported features KVM: x86: Load guest FPU state when access XSAVE-managed MSRs Yang Weijiang (23): x86/fpu/xstate: Refine CET user xstate bit enabling x86/fpu/xstate: Add CET supervisor mode state support x86/fpu/xstate: Introduce XFEATURE_MASK_KERNEL_DYNAMIC xfeature set x86/fpu/xstate: Introduce fpu_guest_cfg for guest FPU configuration x86/fpu/xstate: Create guest fpstate with guest specific config x86/fpu/xstate: Warn if kernel dynamic xfeatures detected in normal fpstate KVM: x86: Rename kvm_{g,s}et_msr()* to menifest emulation operations KVM: x86: Refine xsave-managed guest register/MSR reset handling KVM: x86: Add kvm_msr_{read,write}() helpers KVM: x86: Refresh CPUID on write to guest MSR_IA32_XSS KVM: x86: Initialize kvm_caps.supported_xss KVM: x86: Add fault checks for guest CR4.CET setting KVM: x86: Report KVM supported CET MSRs as to-be-saved KVM: VMX: Introduce CET VMCS fields and control bits KVM: x86: Use KVM-governed feature framework to track "SHSTK/IBT enabled" KVM: VMX: Emulate read and write to CET MSRs KVM: x86: Save and reload SSP to/from SMRAM KVM: VMX: Set up interception for CET MSRs KVM: VMX: Set host constant supervisor states to VMCS fields KVM: x86: Enable CET virtualization for VMX and advertise to userspace KVM: nVMX: Introduce new VMX_BASIC bit for event error_code delivery to L1 KVM: nVMX: Enable CET support for nested guest KVM: x86: Don't emulate instructions guarded by CET arch/x86/include/asm/fpu/types.h | 16 +- arch/x86/include/asm/fpu/xstate.h | 11 +- arch/x86/include/asm/kvm_host.h | 12 +- arch/x86/include/asm/msr-index.h | 1 + arch/x86/include/asm/vmx.h | 8 + arch/x86/include/uapi/asm/kvm_para.h | 1 + arch/x86/kernel/fpu/core.c | 53 +++-- arch/x86/kernel/fpu/xstate.c | 44 ++++- arch/x86/kernel/fpu/xstate.h | 3 + arch/x86/kvm/cpuid.c | 80 ++++++-- arch/x86/kvm/emulate.c | 46 +++-- arch/x86/kvm/governed_features.h | 2 + arch/x86/kvm/smm.c | 12 +- arch/x86/kvm/smm.h | 2 +- arch/x86/kvm/vmx/capabilities.h | 10 + arch/x86/kvm/vmx/nested.c | 120 ++++++++++-- arch/x86/kvm/vmx/nested.h | 5 + arch/x86/kvm/vmx/vmcs12.c | 6 + arch/x86/kvm/vmx/vmcs12.h | 14 +- arch/x86/kvm/vmx/vmx.c | 112 ++++++++++- arch/x86/kvm/vmx/vmx.h | 9 +- arch/x86/kvm/x86.c | 280 ++++++++++++++++++++++++--- arch/x86/kvm/x86.h | 28 +++ 23 files changed, 761 insertions(+), 114 deletions(-) base-commit: c0f8b0752b0988e5116c78e8b6c3cfdf89806e45