From nobody Wed Dec 17 00:38:35 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 02B92C77B76 for ; Sat, 22 Apr 2023 02:19:43 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229572AbjDVCTk (ORCPT ); Fri, 21 Apr 2023 22:19:40 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:50998 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229533AbjDVCTd (ORCPT ); Fri, 21 Apr 2023 22:19:33 -0400 Received: from DM5PR00CU002.outbound.protection.outlook.com (mail-centralusazon11021020.outbound.protection.outlook.com [52.101.62.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C3B791FFE; Fri, 21 Apr 2023 19:19:31 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ESpR1mOm/iWIuGYFAnPr6BuUOOR4BVnM2PhSvvTAPfvwlEdtupTQ1+di9OpNsJLomcrAA+9oQ3/h00LqRtOb9FEqtZj0uZYxSLyNOhVaWieuLYoisu56/WpgaQ2hiezt0xlooRU4XBBnm7PR1SXw/ARBqRoiSJMWLgP6WbUe/UrzvnTtJy+tE/QUww06h6z8ijqWBDcUX/WO4+1tbtBu8ze/9C2Zzao9VMAsAQ1InHka842Pb8avN0jN9NbPGSh4psu8FIhtf4iMfH4KE9awoC+Eb6BBvg61lXuDHXy60bd1DFXuTwjnHBYCeMBb+QpT1i6QY1RiwHS8nUwyeaVvqw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=HU2Bt79BFoMpAhMMCkLW9mbW/eTQdPppArjP7XzLuN8=; b=QMf3rPuFfA+4MCn1dnnZpXx5L4cO+7kU5T1cd9LadTBClGZcHtEQ0VeMYlL1gbUkSWwXxWq0W7i/ktMdbgnOhYCuIBimU6lQBhurYKCorV9Ljv76HNsK/IS7ZKcSgSujWNhrzIP+RmFxegIVclYc6fvGPDVT557Q36DDG5V3dkajgV/zN241pK+VAn12cgvYm7XAjnyST9IKK7KwT5ZzXIdgEBgydS/Ec4FY9hLcqd3KeFYr49Qn7n+qCiYxvlCOtU74K7JfXVJso/Da/MwhnV11ZLf4LuGzlV/0dWVIQwgF1Mvwrjdd3yS0/8oA7vMJoF9/Gn0/az3f4QgPqOUTJA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=HU2Bt79BFoMpAhMMCkLW9mbW/eTQdPppArjP7XzLuN8=; b=WeOot1TGj8V2pt3cTRa+XdFax1toYG+6UcS3/FUX+Xr/981rPmNEnNktycLPXKkzrR9nccYK5uXXTpDlrG76R+A0NpbS3/r7047uYI931mRriK3zBNV0nXScn0AAl/nVxc+Y3l0EReSlgPmHapQNkOYrEil/nPcXUjDJLhMNWxA= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; Received: from BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) by DM6PR21MB1418.namprd21.prod.outlook.com (2603:10b6:5:25c::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6340.11; Sat, 22 Apr 2023 02:19:29 +0000 Received: from BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30]) by BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30%5]) with mapi id 15.20.6340.014; Sat, 22 Apr 2023 02:19:29 +0000 From: Dexuan Cui To: ak@linux.intel.com, arnd@arndb.de, bp@alien8.de, brijesh.singh@amd.com, dan.j.williams@intel.com, dave.hansen@linux.intel.com, haiyangz@microsoft.com, hpa@zytor.com, jane.chu@oracle.com, kirill.shutemov@linux.intel.com, kys@microsoft.com, linux-arch@vger.kernel.org, linux-hyperv@vger.kernel.org, luto@kernel.org, mingo@redhat.com, peterz@infradead.org, rostedt@goodmis.org, sathyanarayanan.kuppuswamy@linux.intel.com, seanjc@google.com, tglx@linutronix.de, tony.luck@intel.com, wei.liu@kernel.org, x86@kernel.org, mikelley@microsoft.com Cc: linux-kernel@vger.kernel.org, Tianyu.Lan@microsoft.com, Dexuan Cui Subject: [PATCH v5 1/6] x86/tdx: Retry TDVMCALL_MAP_GPA() when needed Date: Fri, 21 Apr 2023 19:17:30 -0700 Message-Id: <20230422021735.27698-2-decui@microsoft.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20230422021735.27698-1-decui@microsoft.com> References: <20230422021735.27698-1-decui@microsoft.com> X-ClientProxiedBy: CY5PR03CA0035.namprd03.prod.outlook.com (2603:10b6:930:8::20) To BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL0PR2101MB1092:EE_|DM6PR21MB1418:EE_ X-MS-Office365-Filtering-Correlation-Id: b2f1ba7a-504f-4a0f-6f88-08db42d8004c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: WohpIpcIDJgiYVOKvUMfuGMt0v7DI4iXgLPZfSt9zrWz2/bqgq11VTBdHZ/mVJTp1ycH4h7Ae64HZSRg9SN08Dd0oAFL+8CAXTkYHMCzO6UxlK/ctvEut59v4FuLijq8wHfs0GC2vU/fuUryx0wAk92j+q8j8gLloG3ivxTWfNy4iQwoAVa1XRkuF/MZvqEUX+XCNUSyOqS0gHsaIe5aTzDddeiP/mW9B/ZRfHbofdj/nG6SxTGYpQlEvhT6VsIYv/rLmfFYkaWq+D25XzRoU1Jbc8g1zAJdU6lr9m6SVbsjjpNUv8ru159hXZ4HQe3u3W+Tqq1Qs9yxab/+iCCXSf/smOLeilDIIXo7xwysS814b+SJaHhW2GdW0XK6ge5Hswy7IdWCiWRpPnSltk/knLdF98WILvYEhtvUpdc/ZJn4l8dGXrLnJ2IJ84UXiC+w8514vRROPQqKN7kjgWi0szpyDV5nya/ibBiapMu05ymExvqq9sAtHr7X07oZk3An3EfEQXA20SUU83uksXG69dxAKdAvr13Xy4s7GbFr4DQTnrnDtkns8vYMuXVM+IzHBIVBfAOFQ1ExlWXxSAUk6SpYS2USq3cAjh7GAkAN0KFYOSwLWAQ6Ynn5ZuSXwsbZuBvyPK0yaJwkHWLBf428dw== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR2101MB1092.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(346002)(376002)(366004)(396003)(39860400002)(136003)(451199021)(83380400001)(2616005)(86362001)(107886003)(6512007)(6506007)(1076003)(36756003)(52116002)(478600001)(6486002)(6666004)(66476007)(4326008)(66946007)(41300700001)(82950400001)(82960400001)(921005)(66556008)(316002)(786003)(6636002)(8936002)(8676002)(38100700002)(10290500003)(7416002)(186003)(2906002)(5660300002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?Djw7lO7OjEgI/JlKl/f7d7VjZiSQeGGuOaEj5NR1mF38Vm8pZcH9DHqcKdO+?= =?us-ascii?Q?iWSvkxyKsdK9jxEoyJ6CsQ8ovgM2ydzwrZUxe5//CPyY015V5EJhz+tQlHZZ?= =?us-ascii?Q?1e4Drgf+w+2HtvXB+rS4Mgltjws2NvawYZZpRuIX/fs5ygB4AjkkbuLLlw5U?= =?us-ascii?Q?2TIsJrqe9krvO2XoGWkFbdKcg2DVCr5KSqqCpQy78K6DYHmJ4aPMqPbGKDc+?= =?us-ascii?Q?X/IU7UZjvcPFkc8pPzKPNh4abZznRapcq0l138fXKWMPoWwlkHPlzjJtthAV?= =?us-ascii?Q?aV+g3CLiQ20rn6UHzZo7W2kLnGSkZE8LqhhFZqBD77MlO+4fXIgkVp4UV+z3?= =?us-ascii?Q?XQ9oiUWVUAh1Jmw/2JyNXF2MqdoNOCaiFDtNqH8H3E6dHwpr14ofJriAY2y2?= =?us-ascii?Q?oJdHi8am4X+ghFkKIM22a7lKUOkbtuiPuqmKeJIUhGHcVVkcTQAhWbPE8ZOx?= =?us-ascii?Q?Tr3Lso2wBBAEASqDspvclTiX+2MD2IRUrfUJpvKmuaXH4rqFWjS9zgwX3T+f?= =?us-ascii?Q?42XQi8ltDZQjyyT7C3FYociZ/9tUhCJz+SHvzN4Ckmp/fjLuf6lCfVaszUN/?= =?us-ascii?Q?WtYGulpSILtdX5TQXFMzLJuqkadyk5XP+3TSIsmv5agh/Jbd21KfogIfcgSY?= =?us-ascii?Q?76mHHUBj+Uebo2N23rEcGJgWvjjIFII2/1yl+veuF46ryNJPieBhXk/KO/up?= =?us-ascii?Q?yptYNkJqOEyULXhtRD3x/PhH3WuW1EtlYUlkBO2c6TJ8fuTNXqhIfmW3p9yu?= =?us-ascii?Q?KT5zecTxmrQanmc7UGphMeJwqM64wRLoZ34WOOH1+gh7TNIqNg+PtmST76ml?= =?us-ascii?Q?0Vuiadn6jSsN4ZxZWCqWr6rSA/cgAR9/rf7USkNPonaQMF7fu/qzs4sH4BLr?= =?us-ascii?Q?F8A2iIQnQ+ATKTUCjVO8nEOmJQo8P7ODsYPrVAV2NRbn2MCaUvituEMRYpYC?= =?us-ascii?Q?6glJybcV16QabbqSXzKnYNGvqCqtKcEJYsPIx+N7njRoWkbk8/BrUa/5/CoZ?= =?us-ascii?Q?iUGy1tH6J9S8eDsyaDthSKcOO/4BEs6PN9zs6PkiK1PxrJG6DKsvsmMmaIOe?= =?us-ascii?Q?nj+RPbGkMFFxEQ4bSLKEwuRImbYb9ZhsP8/WjsDLrSLcQnlpiE6lQAymOq8o?= =?us-ascii?Q?YQNlBLm4iJG/hpqTPpGaN9r+C7d5VkB81Hw0pIYSY4REJePkPoOXlpNJW8bz?= =?us-ascii?Q?b9cVO2fQeqQh9rFVhOvc8XebZ1t/9XJcE3w44bx1FQpDwYP36z2PEyjNg88t?= =?us-ascii?Q?3VBCrQmBthfkTbSTMbGwBvzHEH/IynMabmnU8rb1zK7oz+3uzeWk9XKrHt92?= =?us-ascii?Q?C9fhh748dTG1lRIkrpiXer+9ARjsDnaHF9YKoOugVQADSOLDEyBSdKtV9Lj0?= =?us-ascii?Q?LOXwx6UP6dFXin+74f4mXssfNmGskB/eD/kc0zYbjubnIlQrHb0+zKzQg24Y?= =?us-ascii?Q?dnYEu5ogauS8B9jFx/n6Wx/FYDCRx1VvivHqyMqyBCI39SmJ/EjSGzU9BoC7?= =?us-ascii?Q?6H7P+ngrAo9WTPPy/LBAK2w/hGHBaWcWwAvy2SIfA6frwCWv4TdMYB4trtH4?= =?us-ascii?Q?7fssml5l0Os2yNJbHhLXYslJwfHtKAQzK0hmHSIZ7ksJ1MoiRdm0kIx6y79g?= =?us-ascii?Q?LisFru8xfcLbZIl9eKsMIaM6w9SIw0gi2xQ3/bemPnLt?= X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: b2f1ba7a-504f-4a0f-6f88-08db42d8004c X-MS-Exchange-CrossTenant-AuthSource: BL0PR2101MB1092.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2023 02:19:29.0360 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 406ZXY5dPxNoENfWqc2Spy4EPi7fg1slOfqN9MHPZOojr5r2KBtp2vT300Ddnn2iJtnsSrYJpKPwUh5/F//i/A== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR21MB1418 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" GHCI spec for TDX 1.0 says that the MapGPA call may fail with the R10 error code =3D TDG.VP.VMCALL_RETRY (1), and the guest must retry this operation for the pages in the region starting at the GPA specified in R11. When a TDX guest runs on Hyper-V, Hyper-V returns the retry error when hyperv_init() -> swiotlb_update_mem_attributes() -> set_memory_decrypted() decrypts up to 1GB of swiotlb bounce buffers. Acked-by: Kirill A. Shutemov Reviewed-by: Michael Kelley Signed-off-by: Dexuan Cui --- arch/x86/coco/tdx/tdx.c | 64 +++++++++++++++++++++++++++++++++-------- 1 file changed, 52 insertions(+), 12 deletions(-) Changes in v2: Used __tdx_hypercall() directly in tdx_map_gpa(). Added a max_retry_cnt of 1000. Renamed a few variables, e.g., r11 -> map_fail_paddr. Changes in v3: Changed max_retry_cnt from 1000 to 3. Changes in v4: __tdx_hypercall(&args, TDX_HCALL_HAS_OUTPUT) -> __tdx_hypercall_ret() Added Kirill's Acked-by. Changes in v5: Added Michael's Reviewed-by. diff --git a/arch/x86/coco/tdx/tdx.c b/arch/x86/coco/tdx/tdx.c index 4c4c6db39eca..5574c91541a2 100644 --- a/arch/x86/coco/tdx/tdx.c +++ b/arch/x86/coco/tdx/tdx.c @@ -28,6 +28,8 @@ #define TDVMCALL_MAP_GPA 0x10001 #define TDVMCALL_REPORT_FATAL_ERROR 0x10003 =20 +#define TDVMCALL_STATUS_RETRY 1 + /* MMIO direction */ #define EPT_READ 0 #define EPT_WRITE 1 @@ -788,14 +790,15 @@ static bool try_accept_one(phys_addr_t *start, unsign= ed long len, } =20 /* - * Inform the VMM of the guest's intent for this physical page: shared with - * the VMM or private to the guest. The VMM is expected to change its map= ping - * of the page in response. + * Notify the VMM about page mapping conversion. More info about ABI + * can be found in TDX Guest-Host-Communication Interface (GHCI), + * section "TDG.VP.VMCALL". */ -static bool tdx_enc_status_changed(unsigned long vaddr, int numpages, bool= enc) +static bool tdx_map_gpa(phys_addr_t start, phys_addr_t end, bool enc) { - phys_addr_t start =3D __pa(vaddr); - phys_addr_t end =3D __pa(vaddr + numpages * PAGE_SIZE); + int max_retry_cnt =3D 3, retry_cnt =3D 0; + struct tdx_hypercall_args args; + u64 map_fail_paddr, ret; =20 if (!enc) { /* Set the shared (decrypted) bits: */ @@ -803,12 +806,49 @@ static bool tdx_enc_status_changed(unsigned long vadd= r, int numpages, bool enc) end |=3D cc_mkdec(0); } =20 - /* - * Notify the VMM about page mapping conversion. More info about ABI - * can be found in TDX Guest-Host-Communication Interface (GHCI), - * section "TDG.VP.VMCALL" - */ - if (_tdx_hypercall(TDVMCALL_MAP_GPA, start, end - start, 0, 0)) + while (1) { + memset(&args, 0, sizeof(args)); + args.r10 =3D TDX_HYPERCALL_STANDARD; + args.r11 =3D TDVMCALL_MAP_GPA; + args.r12 =3D start; + args.r13 =3D end - start; + + ret =3D __tdx_hypercall_ret(&args); + if (ret !=3D TDVMCALL_STATUS_RETRY) + break; + /* + * The guest must retry the operation for the pages in the + * region starting at the GPA specified in R11. Make sure R11 + * contains a sane value. + */ + map_fail_paddr =3D args.r11; + if (map_fail_paddr < start || map_fail_paddr >=3D end) + return false; + + if (map_fail_paddr =3D=3D start) { + retry_cnt++; + if (retry_cnt > max_retry_cnt) + return false; + } else { + retry_cnt =3D 0; + start =3D map_fail_paddr; + } + } + + return !ret; +} + +/* + * Inform the VMM of the guest's intent for this physical page: shared with + * the VMM or private to the guest. The VMM is expected to change its mapp= ing + * of the page in response. + */ +static bool tdx_enc_status_changed(unsigned long vaddr, int numpages, bool= enc) +{ + phys_addr_t start =3D __pa(vaddr); + phys_addr_t end =3D __pa(vaddr + numpages * PAGE_SIZE); + + if (!tdx_map_gpa(start, end, enc)) return false; =20 /* private->shared conversion requires only MapGPA call */ --=20 2.25.1 From nobody Wed Dec 17 00:38:35 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id A1E50C77B61 for ; Sat, 22 Apr 2023 02:19:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229624AbjDVCTq (ORCPT ); Fri, 21 Apr 2023 22:19:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51010 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229447AbjDVCTe (ORCPT ); Fri, 21 Apr 2023 22:19:34 -0400 Received: from DM5PR00CU002.outbound.protection.outlook.com (mail-centralusazon11021020.outbound.protection.outlook.com [52.101.62.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id ED4DC2100; Fri, 21 Apr 2023 19:19:32 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EFTzJ5fwlU/ZS5gw4ZnQFVIQ4EKOpACdqqcdJpNpWSOuKtPw15vWca5jvvLzPyQ/JMbXf+80n9CJwn1BxF32S6qdP4s+724UchfWgSDBdHG3MMWwDi/FUvZEFLv4B0nFMaUmakFNbQzY3NzKHkLtGWkM1rFSORz0l8jCTdLCmQMjdTb5WDsMFQNxzUU96G6GHVOWc5s9YIVaAqbQMrbGw7MUt8t3nNWPjGGHjnLmfMwH3CnZ5h5NmZL7mzZDINp71f1yyf22FHisV3nfgQ8ybh9nVafWlrBNid6XA+jJGO8ZVmZ//rFjVjF8K2dieFZZydRxunPYKdQSMMzPKiNR9g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=HNLlz0cSXewC7JIgVUN898xyrQRDEhO+WjYNVIJeH9k=; b=VHc3TauNPMrATcIlPhdyRRGY3vYNWkVqJHaGnheHfw1/xEELCfN99cVjVco3P3XBPgW06OJTGZ8stdGdPsZIhq6gkikO/mU/xhMS8/NOLerADRg5T7TieMSPOi+60Rc9Hz8FV1JLzXcBwj2MUjlf/H2MuRRzIXKBxA/JDQNPEiAUJkvXH5IKD0wpYmy7HaPsMRmMUmJsN3oyXmaEDt/4hGySkr/JQ1943V2VCe3KBU2VHzZsFSu8kXCxZbfoaYfr+CkCFEGHR1DUTX3OGgglkdDjEm6jjuwiHX1IOQVKJw4cKYTpUWCreFB5UwLMdGHEIrLWGvTag9tl9uonDeUDJA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=HNLlz0cSXewC7JIgVUN898xyrQRDEhO+WjYNVIJeH9k=; b=Q3KIm5JE22U7xZTzs//w7OjNE8DZf3BP63taPYJ8Wj2AMq5lIbiw1eNl4wX0xRHzSAsjLKRotpB7vCTQZX3TwRzlVNods0CJbT+rF/bAho38mqrsSlbtbE9mVWmDl8gLN2M9NoTusR9Eu6v9xyFmsbFFylYeodp5iU5TU3tohe4= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; Received: from BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) by DM6PR21MB1418.namprd21.prod.outlook.com (2603:10b6:5:25c::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6340.11; Sat, 22 Apr 2023 02:19:31 +0000 Received: from BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30]) by BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30%5]) with mapi id 15.20.6340.014; Sat, 22 Apr 2023 02:19:31 +0000 From: Dexuan Cui To: ak@linux.intel.com, arnd@arndb.de, bp@alien8.de, brijesh.singh@amd.com, dan.j.williams@intel.com, dave.hansen@linux.intel.com, haiyangz@microsoft.com, hpa@zytor.com, jane.chu@oracle.com, kirill.shutemov@linux.intel.com, kys@microsoft.com, linux-arch@vger.kernel.org, linux-hyperv@vger.kernel.org, luto@kernel.org, mingo@redhat.com, peterz@infradead.org, rostedt@goodmis.org, sathyanarayanan.kuppuswamy@linux.intel.com, seanjc@google.com, tglx@linutronix.de, tony.luck@intel.com, wei.liu@kernel.org, x86@kernel.org, mikelley@microsoft.com Cc: linux-kernel@vger.kernel.org, Tianyu.Lan@microsoft.com, Dexuan Cui Subject: [PATCH v5 2/6] x86/tdx: Support vmalloc() for tdx_enc_status_changed() Date: Fri, 21 Apr 2023 19:17:31 -0700 Message-Id: <20230422021735.27698-3-decui@microsoft.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20230422021735.27698-1-decui@microsoft.com> References: <20230422021735.27698-1-decui@microsoft.com> X-ClientProxiedBy: CY5PR03CA0035.namprd03.prod.outlook.com (2603:10b6:930:8::20) To BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL0PR2101MB1092:EE_|DM6PR21MB1418:EE_ X-MS-Office365-Filtering-Correlation-Id: fb4c005f-8932-4a13-6d76-08db42d80195 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: xNUbyANkpj0JXqU3Y0m2rVU+Mwc+q91EDaO75aqBvViyoNSMwazNQxVOQ5nGlfwuesrFdVJi1wmNxrbVohywi6KwqgfGIW9hIQQmEZNFVVbhUBJ8yhVb7QLPNoJNkpBBLG2sNuxG06MaYnftbsQFen+lt/ypmGylb4NJcjy3K7YypUvbdS9Bq2W9mnqKaUlwgRi4Nboen6RDrDJUhZkmTzdce3/CoA2MYv5pD7G15wcl0Fsael8mUHE0do6iKFKdsF4+QUIHRnypzKYXG3d0CJku/2FCACSnM6MwwaR6bwVtQgimR1XDYw9b1DtFPB2OZ1gWEYCDA9gV6VaHwJk66ggfIswkoNTJWVvMLQ99gdXDyhrPc4ctalOL6SpkwcM/Tpt+FRsiIK56gCuCVKKZplijQa3iQun3gOqFPfLNk9kW0tc99b93DBZSXFvjgcD3cDV4lOlF1xMqKPiz/cNQ/xSLV1ymjK9oNHc+JRoRqL/6jGuE9w+srO6iWHptl8tZGZxD9bINb9MR6KKKbfgnNfp10EFvCn6CCBdRT3NujvBN8vyRH6RLiSsFCZ1cu8DZsXiGRDh8hf11NjOYrYhmdfQ+d06TmVVn2rOqusVyARoe5K0LcA8RpZzECUBHkzK7YSiqVvJ6dz3IFXBtPfAxug== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR2101MB1092.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(346002)(376002)(366004)(396003)(39860400002)(136003)(451199021)(83380400001)(2616005)(86362001)(107886003)(6512007)(6506007)(1076003)(36756003)(52116002)(478600001)(6486002)(66476007)(4326008)(66946007)(41300700001)(82950400001)(82960400001)(921005)(66556008)(316002)(786003)(6636002)(8936002)(8676002)(38100700002)(10290500003)(7416002)(186003)(2906002)(5660300002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?qT6XG4nBUrHjmAW/bBkKT69rApOFLQE0tvRu6z32i4YRO0MbypGzYaQxTPWr?= =?us-ascii?Q?xi9sFC6V0B8k5EkgSgDYYuVv4z2YT47yLTTDWxylpFFWcre3yg7IPnLe1i/u?= =?us-ascii?Q?g5XO6qLyspHRXLtzljaBJnS93qbpj35m0JV0OFqV7g3UFV3HbRc0Uwdgo9Zo?= =?us-ascii?Q?E64hiYn1ghbl4tnP0wGipgT41OAMhtYKGzymaFbKXXQswNlT+iHrTjtaS9Rq?= =?us-ascii?Q?8nN+Hw5AEeQGvDOTDJqRKj42K5bsF2D8C2OKVmWKrnhmk5SvfofmV/bNff9n?= =?us-ascii?Q?AC3KyXc4Wju1F1XHYC72yi5Hn8/QJvz37eh2RD3eWKMLI7HRHKv8Mzu3UwL5?= =?us-ascii?Q?fLu/DWWbbMtFPfuMiHL/r+XrKued5fLw4t5cGN60o2huRIxiXM/e9mOQyHIL?= =?us-ascii?Q?F3ijLIjii6E0luKajtjpZ5BbptDhmNwzqVELfxLiT6DLDv0bsUhUHCyGyIln?= =?us-ascii?Q?BMYDP9EJxvUIfASOilZSYnMli/NM4TFiT2dXg2W7yA04BjDGpYm9l+w0EqT0?= =?us-ascii?Q?GTSgGaQY/VyVdMJ5Hgl+xkX2GKF5vnPDeL6/ONtcuFBYj5qySnoNOg0LbOB3?= =?us-ascii?Q?jqD1CHCfOURc5uBLGgNMo0zoEYWIainRs0ENDIInqfeJz7eVekWxZf4uOh1d?= =?us-ascii?Q?nRo7RQFSTm2+4QrYkDTXfezcMW0zMRtx7SsY1+ZmUiju0sXt7D3a56jR54w6?= =?us-ascii?Q?vXu8YodlwbxAgdEKaS5LfO6qDBTfHo0UV7Qf9xgv5NErKaTPvSaCGDUUk/zg?= =?us-ascii?Q?lWNelNctY/RdjBzF4tNUQwZxROZKFp0qj+Ep6t26/vQmXuKWyL4bDx/cze6k?= =?us-ascii?Q?hydhiMpGBYxqs9XSFkTNJEzGhRt5JfmBERDLQlPAuzkukG05ISM+isv8i5+6?= =?us-ascii?Q?l5aS899PqOa9/EsO97JozjodFY0ZAe39UZhfxiQ6n4EwrySEDaRHjhvjOYOg?= =?us-ascii?Q?OrDbGxukIznhMEsTd7UQ/hrQdg+ewMJwhpc/A6LAcdC5WhWw9V6XIq4TMRQl?= =?us-ascii?Q?Y+BM574CssCK7isWH1rJTxEz8GmZF0c0aTUQnyKXagObPGc4jxYsYHB3m40u?= =?us-ascii?Q?8td5boAznAZhgsLzs7m09VmYLa4xauAW1LbsTBzFOQUouHhnIE8pVXYkKdXe?= =?us-ascii?Q?lEkJs1QTzjGcYSB31rJ9WwgaDEBuPngwCzBgAsCYw0YAzAvAe6ALniFtbfk/?= =?us-ascii?Q?8fnt1G6PmV7OQk3ccQR5chmg3We4FSD9BQ5jnqF50sfK3NCEmzClNVNnWXPS?= =?us-ascii?Q?DolL3RputIjeYcVV0COFhzj/2Ff07UAmLlAQuj92+jhX2WaOt8BFRUSr5WMv?= =?us-ascii?Q?tqPymNq3oxgBKE+PktavYnxqPPGU3ca7MAbehQ23h0MKqxCib8xvfwtzyDsM?= =?us-ascii?Q?8yA+OWMKQgww+wLyAiEaV6YTz01RPhvnZwfadOQ0MYtu/ZNhxTtzMyYibNSJ?= =?us-ascii?Q?segZoxqpl7iyihw2jbKm7ufah/mCKAwb11vNWKhf0uFPvYi/FvqWqmKBDZuN?= =?us-ascii?Q?yNjA4UvaV4XqShWTgzdEN1lnwYoCM4sUfkY1Xsu/vXkkHk5VcNQS3jNvyKmx?= =?us-ascii?Q?ano+caa2dVBbG+Y8c/g+JH7SMWnqK14958XE/Fv9DLqo5WKOgsYY8TiPgKHY?= =?us-ascii?Q?lU3hw3ahJNVXZwzirSyykibU2AWChzQCqTUHTN2OT0b2?= X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: fb4c005f-8932-4a13-6d76-08db42d80195 X-MS-Exchange-CrossTenant-AuthSource: BL0PR2101MB1092.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2023 02:19:31.3592 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: f30hbCgf03KS6aX4qmLZw7O5IeM71jQWN58wZJCGdqfAmMJCHkQbs0gLkaizHoemFTKIGmNNU2E3R7Ay2b2T8Q== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR21MB1418 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" When a TDX guest runs on Hyper-V, the hv_netvsc driver's netvsc_init_buf() allocates buffers using vzalloc(), and needs to share the buffers with the host OS by calling set_memory_decrypted(), which is not working for vmalloc() yet. Add the support by handling the pages one by one. Co-developed-by: Kirill A. Shutemov Signed-off-by: Kirill A. Shutemov Reviewed-by: Michael Kelley Signed-off-by: Dexuan Cui --- arch/x86/coco/tdx/tdx.c | 76 ++++++++++++++++++++++++++++------------- 1 file changed, 52 insertions(+), 24 deletions(-) Changes in v2: Changed tdx_enc_status_changed() in place. Hi, Dave, I checked the huge vmalloc mapping code, but still don't know how to get the underlying huge page info (if huge page is in use) and try to use PG_LEVEL_2M/1G in try_accept_page() for vmalloc: I checked is_vm_area_hugepages() and __vfree() -> __vunmap(), and I think the underlying page allocation info is internal to the mm code, and there is no mm API to for me get the info in tdx_enc_status_changed(). Changes in v3: No change since v2. Changes in v4: Added Kirill's Co-developed-by since Kirill helped to improve the code by adding tdx_enc_status_changed_phys(). Thanks Kirill for the clarification on load_unaligned_zeropad()! The vzalloc() usage in drivers/net/hyperv/netvsc.c: netvsc_init_buf() remains the same. It may not worth it to "allocate a vmalloc region, allocate pages manually", because we have to consider the worst case where the system is sufferiing from severe memory fragmentation and we can only allocate multiple single pages. We may not want to complicate the code in netvsc_init_buf(). We'll support NIC SR-IOV for TDX VMs on Hyper-V, so the netvsc send/recv buffers won't be used when the VF NIC is up. Changes in v5: Added Kirill's Signed-off-by. Added Michael's Reviewed-by. diff --git a/arch/x86/coco/tdx/tdx.c b/arch/x86/coco/tdx/tdx.c index 5574c91541a2..731be50b3d09 100644 --- a/arch/x86/coco/tdx/tdx.c +++ b/arch/x86/coco/tdx/tdx.c @@ -7,6 +7,7 @@ #include #include #include +#include #include #include #include @@ -789,6 +790,34 @@ static bool try_accept_one(phys_addr_t *start, unsigne= d long len, return true; } =20 +static bool try_accept_page(phys_addr_t start, phys_addr_t end) +{ + /* + * For shared->private conversion, accept the page using + * TDX_ACCEPT_PAGE TDX module call. + */ + while (start < end) { + unsigned long len =3D end - start; + + /* + * Try larger accepts first. It gives chance to VMM to keep + * 1G/2M SEPT entries where possible and speeds up process by + * cutting number of hypercalls (if successful). + */ + + if (try_accept_one(&start, len, PG_LEVEL_1G)) + continue; + + if (try_accept_one(&start, len, PG_LEVEL_2M)) + continue; + + if (!try_accept_one(&start, len, PG_LEVEL_4K)) + return false; + } + + return true; +} + /* * Notify the VMM about page mapping conversion. More info about ABI * can be found in TDX Guest-Host-Communication Interface (GHCI), @@ -838,6 +867,19 @@ static bool tdx_map_gpa(phys_addr_t start, phys_addr_t= end, bool enc) return !ret; } =20 +static bool tdx_enc_status_changed_phys(phys_addr_t start, phys_addr_t end, + bool enc) +{ + if (!tdx_map_gpa(start, end, enc)) + return false; + + /* private->shared conversion requires only MapGPA call */ + if (!enc) + return true; + + return try_accept_page(start, end); +} + /* * Inform the VMM of the guest's intent for this physical page: shared with * the VMM or private to the guest. The VMM is expected to change its mapp= ing @@ -845,37 +887,23 @@ static bool tdx_map_gpa(phys_addr_t start, phys_addr_= t end, bool enc) */ static bool tdx_enc_status_changed(unsigned long vaddr, int numpages, bool= enc) { - phys_addr_t start =3D __pa(vaddr); - phys_addr_t end =3D __pa(vaddr + numpages * PAGE_SIZE); + unsigned long start =3D vaddr; + unsigned long end =3D start + numpages * PAGE_SIZE; =20 - if (!tdx_map_gpa(start, end, enc)) + if (offset_in_page(start) !=3D 0) return false; =20 - /* private->shared conversion requires only MapGPA call */ - if (!enc) - return true; + if (!is_vmalloc_addr((void *)start)) + return tdx_enc_status_changed_phys(__pa(start), __pa(end), enc); =20 - /* - * For shared->private conversion, accept the page using - * TDX_ACCEPT_PAGE TDX module call. - */ while (start < end) { - unsigned long len =3D end - start; + phys_addr_t start_pa =3D slow_virt_to_phys((void *)start); + phys_addr_t end_pa =3D start_pa + PAGE_SIZE; =20 - /* - * Try larger accepts first. It gives chance to VMM to keep - * 1G/2M SEPT entries where possible and speeds up process by - * cutting number of hypercalls (if successful). - */ - - if (try_accept_one(&start, len, PG_LEVEL_1G)) - continue; - - if (try_accept_one(&start, len, PG_LEVEL_2M)) - continue; - - if (!try_accept_one(&start, len, PG_LEVEL_4K)) + if (!tdx_enc_status_changed_phys(start_pa, end_pa, enc)) return false; + + start +=3D PAGE_SIZE; } =20 return true; --=20 2.25.1 From nobody Wed Dec 17 00:38:35 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 02A4DC7618E for ; Sat, 22 Apr 2023 02:19:48 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229595AbjDVCTn (ORCPT ); Fri, 21 Apr 2023 22:19:43 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51042 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229548AbjDVCTh (ORCPT ); Fri, 21 Apr 2023 22:19:37 -0400 Received: from DM5PR00CU002.outbound.protection.outlook.com (mail-centralusazon11021020.outbound.protection.outlook.com [52.101.62.20]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id ACCE62736; Fri, 21 Apr 2023 19:19:35 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gRLrZnfaeVF/GsLWuf3hud6okirM9ImirG2nkNv8Tyq+Fabv9lJaeegB2RoIFB7OhnTKQyf7YLMHhyeizyu4KbZc4K0XzlDqhBst3ZmZpBubHMtpLFeuoggCdXGd3mJQb9wsHEIbGI3GoDBWV5oZXTVmLO4gYx56T4sIg+xfdesfuhhL69Y2CBDrulL3lHczCgr/AezCxubsx66Ub2PHUMvBM7ubi70aHP5nK+0Wbh3+kGw/34SWwYZW04tVmAT69TgWeWkJiGLa3p/R8Y1V5xbSNlg8N39tW6dedCkHo9sxPwqJWpOnCpQ7+C8CAYRtvmbi+ucvFtuzaHKQZIFD/w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=SBwoM/LjwXJvKSX67F+QWcIsJfdYlo2EH5x5qfdR5fI=; b=iyzM3l84scMzdtGBRRxShywRk2D8gv1KGVyIxw/b2FzWmAqh/cagw6K0qr8owdVIpVw+AReO1GhLM/h0CrlyqWQj/kqy2Ce71ZliQB051meMnEuTntzhFrWOVfLY2vF6Vid9laHawW/CDo8FHJVfYq3hPW7Hc6KY/e1Gir7haDFOGbJHQDWO/f/jglWAdxZDlhiyN+Ma5ksJqRCsOvXZ2Zp95ZAx5syibimZw6waA+JE6hiunk3euWNqYiFJUfBz2vXS/rknd4h7khIxiX6KF+DtiRcb5+FNM1qS292tL35pujRPpkRW4ZMZ8lYth/uCHrH+ydDy98QgCMu26QXW2w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=SBwoM/LjwXJvKSX67F+QWcIsJfdYlo2EH5x5qfdR5fI=; b=Wc4QwVR8Ep6jkOO9r+M+UAoLbMJakLiuF5gbnxI1yXzloJE+VoixqC6S/yVXgSa1FYejfaIkRjdSj1l0+uBUUU0ZaFliPCcTjmzzEwrRY3BKvzM4z9W5hRirR+iU60XQMibGhbtvTLmVEJc2tcsrY/vSxPC/Zhr5/yg245BAA2s= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; Received: from BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) by DM6PR21MB1418.namprd21.prod.outlook.com (2603:10b6:5:25c::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6340.11; Sat, 22 Apr 2023 02:19:33 +0000 Received: from BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30]) by BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30%5]) with mapi id 15.20.6340.014; Sat, 22 Apr 2023 02:19:33 +0000 From: Dexuan Cui To: ak@linux.intel.com, arnd@arndb.de, bp@alien8.de, brijesh.singh@amd.com, dan.j.williams@intel.com, dave.hansen@linux.intel.com, haiyangz@microsoft.com, hpa@zytor.com, jane.chu@oracle.com, kirill.shutemov@linux.intel.com, kys@microsoft.com, linux-arch@vger.kernel.org, linux-hyperv@vger.kernel.org, luto@kernel.org, mingo@redhat.com, peterz@infradead.org, rostedt@goodmis.org, sathyanarayanan.kuppuswamy@linux.intel.com, seanjc@google.com, tglx@linutronix.de, tony.luck@intel.com, wei.liu@kernel.org, x86@kernel.org, mikelley@microsoft.com Cc: linux-kernel@vger.kernel.org, Tianyu.Lan@microsoft.com, Dexuan Cui Subject: [PATCH v5 3/6] x86/hyperv: Add hv_isolation_type_tdx() to detect TDX guests Date: Fri, 21 Apr 2023 19:17:32 -0700 Message-Id: <20230422021735.27698-4-decui@microsoft.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20230422021735.27698-1-decui@microsoft.com> References: <20230422021735.27698-1-decui@microsoft.com> X-ClientProxiedBy: CY5PR03CA0035.namprd03.prod.outlook.com (2603:10b6:930:8::20) To BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL0PR2101MB1092:EE_|DM6PR21MB1418:EE_ X-MS-Office365-Filtering-Correlation-Id: 486e1cf6-2c77-4cfc-070c-08db42d802fb X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: +y6RW3Xh9d+suZBQQ+Sskin1l0kK8pjKSRSvpXVCspNFmPc7SwKc3u+31nJwBCpa5AqiTq44ImwsV0ZzOM+VeaeiDiXuH2qdJwFqozw9XbqTChHAY0Gg08Qp2YMqYJwzGu7byB/07Z7wO0lBzl++EsQ92R32YgBitTssYMQFZ+0tkFNECPjSOf3Ycz/T5F6XAqqRAlsnFdQsAtmdNSdgxoEdxgQUMcYWJw7LarRBuHfE/85go2OZpnyn+5qN+BovqpIdVAZdAcvR31aUkVBxSmL1CcVgpgRts0qcwE9TfwdySGfUjHCo4hSISYCD3y/NW/JnzXI6qSRtjoR9ArLHU8+j75CnRaKFuj3qrGgzWuWw2I8MhYF3eTl/+/KL/c4zFZxgcdqj/acdtav6c+DcWvWs4njguaFqWgcMiGMMU6OuILXDkbQcQx1PYH0uu7fMYCzrOYtkfCz1y5Ty7DJR8AttIkEFjxRu0fUMQ6SNeQh6vFBDRLEY6E1lXykjeAOD6/0CerlPMOgX+3Gcr/GNtmsCxMJnll1IBQIeFXT78bguRCRLLNNwAks34pRaXbnzSSU7m5VPnnviLXyeAugVrpLVFNENKgaUeF1LfgZDr/LviaDfjE9YwZBy54VvjXyiEORecpqlhh3itqOJMZkApA== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR2101MB1092.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(346002)(376002)(366004)(396003)(39860400002)(136003)(451199021)(83380400001)(2616005)(86362001)(107886003)(6512007)(6506007)(1076003)(36756003)(52116002)(478600001)(6486002)(66476007)(4326008)(66946007)(41300700001)(82950400001)(82960400001)(921005)(66556008)(316002)(786003)(6636002)(8936002)(8676002)(38100700002)(10290500003)(7416002)(186003)(2906002)(5660300002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?xhdu4EcfaQ5bdp+xmJBzeYPP3iCwWctpI9n+mwrSzi+dFQSgEbccjxA47AF8?= =?us-ascii?Q?u23uk6tq/BZ60apzWIH1FtGxriX85JmCLUIzSpxY9ApvVHpETgiFkE+jd0/E?= =?us-ascii?Q?bPPy1YfIUIiNcH9V6xKoRkkB7z7rRIqd08c9RKSdWf/SsmueYp/8wyU1xJoa?= =?us-ascii?Q?baTX1INB8fLLP/hZAmVfvl3mvDQm+6kiPPaMFbhRoDaclu9707UsS9EhRdGF?= =?us-ascii?Q?tZFFjZFUQHOHYkC4jac/wSLa/hQMcNIeN6WPjLv4jdhtmVum58mvR2LcFeYR?= =?us-ascii?Q?M9cZTL7sA61lHrSrjhRnLJSFDPsGE4JWY/WFyH+q3e3IwSVPmu4qB4GRDXjy?= =?us-ascii?Q?O2YujJDB2ocxGnzsBpK6l8IbwD7O4cOuPjQlf9gEdp+OQ0BvDWcF4pz5bRST?= =?us-ascii?Q?Vo+w4mv0Hw9675FSQbbyQ4NBzNdY81cNL+mYDWH7iJVl7LWSBo9ArkldyQ7f?= =?us-ascii?Q?ESkCGkVKbqo2nzAH+n+vMp1mPq5mKFzN0VGVU9CsYQdK/tpZHDQlR0V27XGv?= =?us-ascii?Q?zlC6CEhgaq5zO3OPq1jpKHT8qF2HbGm0rlLmLQsF+yoJ6z1GvMSALool3yxG?= =?us-ascii?Q?QX7YD2+bRO+NsRGFE3PMdgZdXXy8nsan6HzkQD7zTr6hX0XTRZuMcuXljpwP?= =?us-ascii?Q?uSSP4Rx9awhR/dqO7MEaR690hofdsYGjtZrDkN1BPsIET9lcqjeibLaiWXfD?= =?us-ascii?Q?XtYTwjy1HcQ7YkrAEq+50iPXWt8BY3XNnp4lXv4p5+XwwqahG8UdQvVZdH4O?= =?us-ascii?Q?lhLuFkiFxKAWZ8NNu9PDIDo4JUyw9XHYXD6agPfNAonLFHhXIsdhWbNk3Uxi?= =?us-ascii?Q?/TLnMW7CRqZQ1TnQExFP0s3UHYa7sGIWZdyY0kwaRPiYo6Yg1Xhymth7hChh?= =?us-ascii?Q?lEM7yZvM6/d8LKF4tfWbPlQn97sydqVLRSxcNB45NNsY/6pGyEY4O7gfYhVO?= =?us-ascii?Q?iJvalzR3/qYVJV6OgZAyOMAcQhb/F0F+nqlutww/2VRIMwivl0Dku4moPJZr?= =?us-ascii?Q?wiLcuij186HD49U9DSKeSTjwUAKMBfbbjHb5Eh3y2AzgUxY3gCUR7Yhe0ttS?= =?us-ascii?Q?Idpy2zulzQAhokzsDUrwaQ3gHcC8k+Vml4WxzvbXD34kpN05mEuwdkntO7Gn?= =?us-ascii?Q?/Ynoawxnid5gtEDTmp0mjAYs3xGkDn/CwXjipgMYLUE0cMbCpaj4P8y+xwX1?= =?us-ascii?Q?dcokrijr/I2Bwly3XouMb0082CdWC7zcEtAOld/25ebbDkSdEQYJt9WxqeOp?= =?us-ascii?Q?Bytcrbmwhybjm8jlyeIX/3T/e1KWJjZn4O5gHUk+Pq3//wnykFoaemOxghkX?= =?us-ascii?Q?o0dbD8+redjqtydHRcDok2bsVmrA4bTvd2VhNmu69NygG9lozQzEYhEXpEeR?= =?us-ascii?Q?632t29EW4QefbF/VRtBXv8vP7DHa1cI1liW9WdFNy+GNNWNx8V5bdB3cUCQQ?= =?us-ascii?Q?3HIfTBjxMDPAIy5BLy9bChRQEKR6wwPc/7XmrBgDmiXN6WrfrTmhYjVRL5T1?= =?us-ascii?Q?Bn+tyTDMwc3BMgZDRvs0eLQnJww45UIpWrFv/PXEOQ1YhFUeHGngMcDOdvEQ?= =?us-ascii?Q?0ODcPUHWDZ44LWT1AR4M7ufBsgHKJU973pbYEP0ntEvkhVxCvlqaYbnD81cP?= =?us-ascii?Q?FXA0WuYKso0EJ0sdpuboS+GZZs4SZ/Vf4HRvzbfR3GVL?= X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: 486e1cf6-2c77-4cfc-070c-08db42d802fb X-MS-Exchange-CrossTenant-AuthSource: BL0PR2101MB1092.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2023 02:19:33.4969 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: vNBs4IREa7vyIsq/gjao+qYGLZEyXEgQRuSkNbiT072ASDkC1zNQjWH2oF9FgBBTSoJXAp6Xm5TW5Rq/37UrZw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR21MB1418 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" No logic change to SNP/VBS guests. hv_isolation_type_tdx() wil be used to instruct a TDX guest on Hyper-V to do some TDX-specific operations, e.g. hv_do_hypercall() should use __tdx_hypercall(), and a TDX guest on Hyper-V should handle the Hyper-V Event/Message/Monitor pages specially. Reviewed-by: Kuppuswamy Sathyanarayanan Reviewed-by: Michael Kelley Signed-off-by: Dexuan Cui --- arch/x86/hyperv/ivm.c | 6 ++++++ arch/x86/include/asm/hyperv-tlfs.h | 3 ++- arch/x86/include/asm/mshyperv.h | 3 +++ arch/x86/kernel/cpu/mshyperv.c | 2 ++ drivers/hv/hv_common.c | 6 ++++++ include/asm-generic/mshyperv.h | 1 + 6 files changed, 20 insertions(+), 1 deletion(-) Changes in v2: Added "#ifdef CONFIG_INTEL_TDX_GUEST and #endif" for hv_isolation_type_tdx() in arch/x86/hyperv/ivm.c. Simplified the changes in ms_hyperv_init_platform(). Changes in v3: Added Kuppuswamy's Reviewed-by. Changes in v4: A minor rebase to Michael's v7 DDA patchset. Changes in v5: Added Michael's Reviewed-by. diff --git a/arch/x86/hyperv/ivm.c b/arch/x86/hyperv/ivm.c index 127d5b7b63de..3658ade4f412 100644 --- a/arch/x86/hyperv/ivm.c +++ b/arch/x86/hyperv/ivm.c @@ -400,6 +400,7 @@ bool hv_is_isolation_supported(void) } =20 DEFINE_STATIC_KEY_FALSE(isolation_type_snp); +DEFINE_STATIC_KEY_FALSE(isolation_type_tdx); =20 /* * hv_isolation_type_snp - Check system runs in the AMD SEV-SNP based @@ -409,3 +410,8 @@ bool hv_isolation_type_snp(void) { return static_branch_unlikely(&isolation_type_snp); } + +bool hv_isolation_type_tdx(void) +{ + return static_branch_unlikely(&isolation_type_tdx); +} diff --git a/arch/x86/include/asm/hyperv-tlfs.h b/arch/x86/include/asm/hype= rv-tlfs.h index b4fb75bd1013..338f383c721c 100644 --- a/arch/x86/include/asm/hyperv-tlfs.h +++ b/arch/x86/include/asm/hyperv-tlfs.h @@ -169,7 +169,8 @@ enum hv_isolation_type { HV_ISOLATION_TYPE_NONE =3D 0, HV_ISOLATION_TYPE_VBS =3D 1, - HV_ISOLATION_TYPE_SNP =3D 2 + HV_ISOLATION_TYPE_SNP =3D 2, + HV_ISOLATION_TYPE_TDX =3D 3 }; =20 /* Hyper-V specific model specific registers (MSRs) */ diff --git a/arch/x86/include/asm/mshyperv.h b/arch/x86/include/asm/mshyper= v.h index e3cef98a0142..de7ceae9e65e 100644 --- a/arch/x86/include/asm/mshyperv.h +++ b/arch/x86/include/asm/mshyperv.h @@ -22,6 +22,7 @@ union hv_ghcb; =20 DECLARE_STATIC_KEY_FALSE(isolation_type_snp); +DECLARE_STATIC_KEY_FALSE(isolation_type_tdx); =20 typedef int (*hyperv_fill_flush_list_func)( struct hv_guest_mapping_flush_list *flush, @@ -38,6 +39,8 @@ extern u64 hv_current_partition_id; =20 extern union hv_ghcb * __percpu *hv_ghcb_pg; =20 +extern bool hv_isolation_type_tdx(void); + int hv_call_deposit_pages(int node, u64 partition_id, u32 num_pages); int hv_call_add_logical_proc(int node, u32 lp_index, u32 acpi_id); int hv_call_create_vp(int node, u64 partition_id, u32 vp_index, u32 flags); diff --git a/arch/x86/kernel/cpu/mshyperv.c b/arch/x86/kernel/cpu/mshyperv.c index ff348ebb6ae2..a87fb934cd4b 100644 --- a/arch/x86/kernel/cpu/mshyperv.c +++ b/arch/x86/kernel/cpu/mshyperv.c @@ -405,6 +405,8 @@ static void __init ms_hyperv_init_platform(void) =20 if (hv_get_isolation_type() =3D=3D HV_ISOLATION_TYPE_SNP) static_branch_enable(&isolation_type_snp); + else if (hv_get_isolation_type() =3D=3D HV_ISOLATION_TYPE_TDX) + static_branch_enable(&isolation_type_tdx); } =20 if (hv_max_functions_eax >=3D HYPERV_CPUID_NESTED_FEATURES) { diff --git a/drivers/hv/hv_common.c b/drivers/hv/hv_common.c index 6d40b6c7b23b..c55db7ea6580 100644 --- a/drivers/hv/hv_common.c +++ b/drivers/hv/hv_common.c @@ -271,6 +271,12 @@ bool __weak hv_isolation_type_snp(void) } EXPORT_SYMBOL_GPL(hv_isolation_type_snp); =20 +bool __weak hv_isolation_type_tdx(void) +{ + return false; +} +EXPORT_SYMBOL_GPL(hv_isolation_type_tdx); + void __weak hv_setup_vmbus_handler(void (*handler)(void)) { } diff --git a/include/asm-generic/mshyperv.h b/include/asm-generic/mshyperv.h index afcd9ae9588c..83e56ebe0cb7 100644 --- a/include/asm-generic/mshyperv.h +++ b/include/asm-generic/mshyperv.h @@ -58,6 +58,7 @@ extern void * __percpu *hyperv_pcpu_output_arg; extern u64 hv_do_hypercall(u64 control, void *inputaddr, void *outputaddr); extern u64 hv_do_fast_hypercall8(u16 control, u64 input8); extern bool hv_isolation_type_snp(void); +extern bool hv_isolation_type_tdx(void); =20 /* Helper functions that provide a consistent pattern for checking Hyper-V= hypercall status. */ static inline int hv_result(u64 status) --=20 2.25.1 From nobody Wed Dec 17 00:38:35 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id F0908C77B61 for ; Sat, 22 Apr 2023 02:20:00 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229648AbjDVCT7 (ORCPT ); Fri, 21 Apr 2023 22:19:59 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51092 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229573AbjDVCTk (ORCPT ); Fri, 21 Apr 2023 22:19:40 -0400 Received: from DM5PR00CU002.outbound.protection.outlook.com (mail-centralusazon11021017.outbound.protection.outlook.com [52.101.62.17]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 8DF75273A; Fri, 21 Apr 2023 19:19:38 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Hxn4HZncE84mN4Vqy1NhOoTbEbQcZsnXMEK9sC2NGhQa1iV9/C0erjqmHRe2RSJU/WkQ27hIBVNfdjL+APttZ3//isjQZyIbCkZURzCGqdPXHWAPkB6MYfBjaN8mCoZGSI/oRqmn+z85BwgpfxAHQx8Pt+ke5N7j2/ADmFe8AFXNgujwtLtov1klE/O6LnsZ6KNfBlwce+DQvw9P00R2RsjgSjOAxHXfyOheX19A0Hn66uNq9c48TGW3kb6wLVp55e/OOqL3Suc59BQRY1JlARZGxA9Z0ZO7A0mrxzFcTobgtj7wb8Aa/K0NlI+Qxy4wEIobjJuPzJo9JQtN8NUPwg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=THRGJuuH1SKLqlYToOEMCASmgDqLPN0zK7bQWcKleIc=; b=K0CKDi1SlDkMvtht1G7md9TmS1/Y3Yv74yg0vjA0qnCANChpD0+dFHVMnTz/YlcwocPmM3NUTF/ymVSIqduCBv+WRBgP7zN9TaV51sZbDumP8cIw0YmnsHOr3IwmdXkz7aV7XznPcO455Olg6zlYjGqzxd0ek3+MiUCxKSmXykqZp6/ARwmHGHz+z5RsGyPNvco7ULWGZ2DNOBZTqAkuG38xmvcuPc1Ln7eijU79MD5C1rx6nDJHpiheorSZlvEtav0WtFHALnl/7P++fh69J2MiUsuJfkzCbIfV6trFaZyJPJe0jiMPo1Ijhzb6c/vONSeHH6ZY1nAuuQMuSDVmyA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=THRGJuuH1SKLqlYToOEMCASmgDqLPN0zK7bQWcKleIc=; b=cuyOP+EW6AYsr8+onha+voanR9OrQLgUV06bWt+/6UeDvOoQbC2t7RHgmxM6m/HPqXhjCgXR2+yVS4+UeIl95+f2+xi/NUwRgyYk8qS8aFWy2bOweAvmIZcfu7Xm3Nb0NTkDf8jlCCFu0I+E7wN45cRQe3PQmU/MkRWC5PzOPiY= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; Received: from BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) by DM6PR21MB1418.namprd21.prod.outlook.com (2603:10b6:5:25c::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6340.11; Sat, 22 Apr 2023 02:19:35 +0000 Received: from BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30]) by BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30%5]) with mapi id 15.20.6340.014; Sat, 22 Apr 2023 02:19:35 +0000 From: Dexuan Cui To: ak@linux.intel.com, arnd@arndb.de, bp@alien8.de, brijesh.singh@amd.com, dan.j.williams@intel.com, dave.hansen@linux.intel.com, haiyangz@microsoft.com, hpa@zytor.com, jane.chu@oracle.com, kirill.shutemov@linux.intel.com, kys@microsoft.com, linux-arch@vger.kernel.org, linux-hyperv@vger.kernel.org, luto@kernel.org, mingo@redhat.com, peterz@infradead.org, rostedt@goodmis.org, sathyanarayanan.kuppuswamy@linux.intel.com, seanjc@google.com, tglx@linutronix.de, tony.luck@intel.com, wei.liu@kernel.org, x86@kernel.org, mikelley@microsoft.com Cc: linux-kernel@vger.kernel.org, Tianyu.Lan@microsoft.com, Dexuan Cui Subject: [PATCH v5 4/6] x86/hyperv: Support hypercalls for TDX guests Date: Fri, 21 Apr 2023 19:17:33 -0700 Message-Id: <20230422021735.27698-5-decui@microsoft.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20230422021735.27698-1-decui@microsoft.com> References: <20230422021735.27698-1-decui@microsoft.com> X-ClientProxiedBy: CY5PR03CA0035.namprd03.prod.outlook.com (2603:10b6:930:8::20) To BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL0PR2101MB1092:EE_|DM6PR21MB1418:EE_ X-MS-Office365-Filtering-Correlation-Id: de9b5a75-f46a-42c4-4069-08db42d8043f X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: tB7Z95NxlZHyJE8JrSAAzwASEwL/wlc5QJJLcDA3DpCdYad+H5llY9wgUogDF/ZU5wtIUvveRWjuI/nNHS/L9ifBjf+HzCd8W0t+OGfo2v459urkBSXA5r1kdTXHkwiag6v0aXVE/P+cpI2rxSsAGvStZ7OwhwWyT2jVqtW5oZSOz5flXzCMVzo/F1CIqK6NKrATBj9uR/FsEcMs64+s9GimvDZt6u+Vg539dIq/PzBcoscLWrvDCtqpkUW4WOG9I6CsI7emGF6oBbSGs2DlO9iSqx9+tBDwQS0yH98AEfjO5LsMjia3YkRZRLBXvXyoEkjK3FeHF7KSciskCcHL0KxQZ0FGOvsb5T1ZYjOZRq8HQ9Xm0N3kdIi6jqpP317DlzXSRRX7IkYWM5JItSs5WbfY62ES31Auw7B52WMh7kHZZ1PZ/oA7dH8NlHR10bIHZecubHfHktjHX367ZSrvITr/hSSxug4enuNC5pSE9czTnO3rwdS/Ed5KOZw6TRMqt2MdhCWQdsauXESEVO2DIPP4rR+25Xtc1WAfQ9YPSEXpiOBP9vAH80tzSBd9MeanONBWUnrEUoHuRR6wFee0azP8j6e2DOm9ULz8QklPTWK7Tg0o9FuRIRsNiJMarzo8/ayqlotaq8WCsXjCsmOfEQ== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR2101MB1092.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(136003)(396003)(39860400002)(376002)(346002)(366004)(451199021)(8676002)(8936002)(38100700002)(4326008)(66946007)(66476007)(41300700001)(82960400001)(921005)(82950400001)(66556008)(316002)(786003)(6636002)(2906002)(5660300002)(10290500003)(7416002)(186003)(107886003)(86362001)(1076003)(6512007)(6506007)(36756003)(83380400001)(2616005)(52116002)(478600001)(6486002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?hG4odNWdZFrqWEcp3FGlWIA84+zUpTJ8eKCOWhaVKUFgkjPyfWaqpvaGacjU?= =?us-ascii?Q?gVWbfvXc19Y76pqeAasTPrPh5BWqqdLyWrjH5A0j2NhGiAXa32Ma7s2g93Rb?= =?us-ascii?Q?8+Xymn/vXsJkY+l0xTzr/1i/r1jZRz3ITfpJWtXdVEMko8hm9qkxAENNTfrn?= =?us-ascii?Q?cctjw2YrgLXeZlWfAS/NOOrZV1UY0h956YSqKKhXTSubVILa0mETUNOdoTIU?= =?us-ascii?Q?AIW7pNcGZyiZ+xxu1FN2hrBK40J36lpkjyH4DGXmXZVPcfpkHElfRP5S7Uk2?= =?us-ascii?Q?Ac6mPlX93/Bfi1KTQQtb7V6aof3RwM61yo5oV2wXg8Rf41tGEr94dnPIQEJZ?= =?us-ascii?Q?LsSuJSv5PXaIESUc8qlur0e0D/Af8NTNv05umuAe3eXzoWNKTzrbCbIPIlEm?= =?us-ascii?Q?ysS9zJHL0ra2Dyt7p7SoIpridz4qtn8ZsDD9B3eEDcLL2viUOhnOocsxlTWx?= =?us-ascii?Q?KWdjtJEfO/WBolW7hsNBnvrUpJNf/62f5z4EQkvoAJgMOELXgty9o4olX5mK?= =?us-ascii?Q?qO6b37j2lVDXvoArCQJXfr0wU2yyDSG9MdnRqLILgvb3t6iSn0aIf92U/xB6?= =?us-ascii?Q?DoNAjd5zKV7A1ePqN1Hdk3BVNPvr/eKvtcNC7NW0JBLNj5M6AMhoT0Ij4O92?= =?us-ascii?Q?WydZYwqCfl2dtSDNQt6Pn5W+ZsQRQdB+2ofsiZlDl819JfMPDtzKwhazUh4+?= =?us-ascii?Q?hWo9fHqEGoCyCgF227rKoMdZ2/vZiOwHG9kQDXWtPubYlpW+hlhR6RYwF06i?= =?us-ascii?Q?GYwxRhr8/KssBFkndIx9jWevV8GIxjMcyk1W7BTFnYMmoVIwk9kZ51d1jYOM?= =?us-ascii?Q?WD/WpN+NN8tfxdJE0rN5pDWNZUsON47mJjfEyqFfr23/FMxZ4Q/AgPYVhOH9?= =?us-ascii?Q?TpL57SAszilCD3TDq0svJpPnc7gqTQa2Z6DDC4qEe2uq5d+u4B5Z2skx+DP2?= =?us-ascii?Q?UUKHC76rnl8yHUmSCR/1dwrBsbjVlsCa/cOL1yBBNSQLtnuJUvMXho3o0Sg1?= =?us-ascii?Q?FntCtCsIDDsSHRSh8HiD4PEfp5AMw0wDYdMKLQMg3ku4zrX5dJpc+iO8z9cK?= =?us-ascii?Q?pMKFfWbm0FHPrZjYpCrxPSU5vJ7UxYrxr+FRaqCeAfqa94Lo0x9V+Za/Dt4N?= =?us-ascii?Q?n5rzPewP2ojj6aY60LfLEazlPBjM94O5/DfEITLz18Um7v1iSd7jA/lIGLjR?= =?us-ascii?Q?+1ksB8bdHLUHhEs5dbm4LXrm1z8muk6Zi8avzZK2o1O48sBJwarUL9mQeJ+i?= =?us-ascii?Q?+/9ptF5eau09L0VSbNZS5Mnq1i7C2MgCndccTsmxDvtas9grL+IaZz/fM82R?= =?us-ascii?Q?zyzDpB4XQsqu5timPgSqOJE/GWuwpNbOnDahWTHtj4iWFj1PoEGJlJoQxs5M?= =?us-ascii?Q?KN45hE9pRpafOjLOQu6JJW00rUZgnCNtZi+DyJ0dl2Ih7V/04uvqF9VkWflP?= =?us-ascii?Q?TF+WLJ+C5o6iN15iArN57qrY/0x3QtBgrNYUiusiCCaz6fyP+HuRy/LG43D/?= =?us-ascii?Q?PSjlOquRg1tlS5DO7/iVU1BscZfE7QmJzX7lxYBIZ3AIlv2VVbtLdz3XHPmO?= =?us-ascii?Q?EP1rRLW8yoxz/t7jcK+6v3L6qfbOe1jTFrdhrbM10K3kMypCDZfxhjO95Q5v?= =?us-ascii?Q?jpd5ED4HpMa/A4RxtDuy6CabqXOsJYuU4dhVarYs3vcu?= X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: de9b5a75-f46a-42c4-4069-08db42d8043f X-MS-Exchange-CrossTenant-AuthSource: BL0PR2101MB1092.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2023 02:19:35.6247 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 7wmKzN4ols8dc1jnXyX2beOV5WozST6WyhJ4MukaJDIyaA9kMJfHk0bDIwsD9sGB4OPax2HAu8iO969rjDQ2JA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR21MB1418 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" A TDX guest uses the GHCI call rather than hv_hypercall_pg. In hv_do_hypercall(), Hyper-V requires that the input/output addresses must have the cc_mask. Reviewed-by: Kuppuswamy Sathyanarayanan Reviewed-by: Michael Kelley Signed-off-by: Dexuan Cui --- arch/x86/hyperv/hv_init.c | 8 ++++++++ arch/x86/hyperv/ivm.c | 14 ++++++++++++++ arch/x86/include/asm/mshyperv.h | 17 +++++++++++++++++ drivers/hv/hv_common.c | 24 ++++++++++++++++++++++++ 4 files changed, 63 insertions(+) Changes in v2: Implemented hv_tdx_hypercall() in C rather than in assembly code. Renamed the parameter names of hv_tdx_hypercall(). Used cc_mkdec() directly in hv_do_hypercall(). Changes in v3: Decrypted/encrypted hyperv_pcpu_input_arg in hv_common_cpu_init() and hv_common_cpu_die(). Changes in v4: __tdx_hypercall(&args, TDX_HCALL_HAS_OUTPUT) -> __tdx_hypercall_ret() hv_common_cpu_die(): explicitly ignore the error set_memory_encrypted() [= Michael Kelley] Added Sathyanarayanan's Reviewed-by. Changes in v5: Added Michael's Reviewed-by. diff --git a/arch/x86/hyperv/hv_init.c b/arch/x86/hyperv/hv_init.c index a5f9474f08e1..f175e0de821c 100644 --- a/arch/x86/hyperv/hv_init.c +++ b/arch/x86/hyperv/hv_init.c @@ -432,6 +432,10 @@ void __init hyperv_init(void) /* Hyper-V requires to write guest os id via ghcb in SNP IVM. */ hv_ghcb_msr_write(HV_X64_MSR_GUEST_OS_ID, guest_id); =20 + /* A TDX guest uses the GHCI call rather than hv_hypercall_pg. */ + if (hv_isolation_type_tdx()) + goto skip_hypercall_pg_init; + hv_hypercall_pg =3D __vmalloc_node_range(PAGE_SIZE, 1, VMALLOC_START, VMALLOC_END, GFP_KERNEL, PAGE_KERNEL_ROX, VM_FLUSH_RESET_PERMS, NUMA_NO_NODE, @@ -471,6 +475,7 @@ void __init hyperv_init(void) wrmsrl(HV_X64_MSR_HYPERCALL, hypercall_msr.as_uint64); } =20 +skip_hypercall_pg_init: /* * hyperv_init() is called before LAPIC is initialized: see * apic_intr_mode_init() -> x86_platform.apic_post_init() and @@ -594,6 +599,9 @@ bool hv_is_hyperv_initialized(void) if (x86_hyper_type !=3D X86_HYPER_MS_HYPERV) return false; =20 + /* A TDX guest uses the GHCI call rather than hv_hypercall_pg. */ + if (hv_isolation_type_tdx()) + return true; /* * Verify that earlier initialization succeeded by checking * that the hypercall page is setup diff --git a/arch/x86/hyperv/ivm.c b/arch/x86/hyperv/ivm.c index 3658ade4f412..23304c9ddd34 100644 --- a/arch/x86/hyperv/ivm.c +++ b/arch/x86/hyperv/ivm.c @@ -415,3 +415,17 @@ bool hv_isolation_type_tdx(void) { return static_branch_unlikely(&isolation_type_tdx); } + +u64 hv_tdx_hypercall(u64 control, u64 param1, u64 param2) +{ + struct tdx_hypercall_args args =3D { }; + + args.r10 =3D control; + args.rdx =3D param1; + args.r8 =3D param2; + + (void)__tdx_hypercall_ret(&args); + + return args.r11; +} +EXPORT_SYMBOL_GPL(hv_tdx_hypercall); diff --git a/arch/x86/include/asm/mshyperv.h b/arch/x86/include/asm/mshyper= v.h index de7ceae9e65e..71077326f57b 100644 --- a/arch/x86/include/asm/mshyperv.h +++ b/arch/x86/include/asm/mshyperv.h @@ -10,6 +10,7 @@ #include #include #include +#include =20 /* * Hyper-V always provides a single IO-APIC at this MMIO address. @@ -45,6 +46,12 @@ int hv_call_deposit_pages(int node, u64 partition_id, u3= 2 num_pages); int hv_call_add_logical_proc(int node, u32 lp_index, u32 acpi_id); int hv_call_create_vp(int node, u64 partition_id, u32 vp_index, u32 flags); =20 +u64 hv_tdx_hypercall(u64 control, u64 param1, u64 param2); + +/* + * If the hypercall involves no input or output parameters, the hypervisor + * ignores the corresponding GPA pointer. + */ static inline u64 hv_do_hypercall(u64 control, void *input, void *output) { u64 input_address =3D input ? virt_to_phys(input) : 0; @@ -52,6 +59,10 @@ static inline u64 hv_do_hypercall(u64 control, void *inp= ut, void *output) u64 hv_status; =20 #ifdef CONFIG_X86_64 + if (hv_isolation_type_tdx()) + return hv_tdx_hypercall(control, + cc_mkdec(input_address), + cc_mkdec(output_address)); if (!hv_hypercall_pg) return U64_MAX; =20 @@ -95,6 +106,9 @@ static inline u64 _hv_do_fast_hypercall8(u64 control, u6= 4 input1) u64 hv_status; =20 #ifdef CONFIG_X86_64 + if (hv_isolation_type_tdx()) + return hv_tdx_hypercall(control, input1, 0); + { __asm__ __volatile__(CALL_NOSPEC : "=3Da" (hv_status), ASM_CALL_CONSTRAINT, @@ -140,6 +154,9 @@ static inline u64 _hv_do_fast_hypercall16(u64 control, = u64 input1, u64 input2) u64 hv_status; =20 #ifdef CONFIG_X86_64 + if (hv_isolation_type_tdx()) + return hv_tdx_hypercall(control, input1, input2); + { __asm__ __volatile__("mov %4, %%r8\n" CALL_NOSPEC diff --git a/drivers/hv/hv_common.c b/drivers/hv/hv_common.c index c55db7ea6580..10e85682e83e 100644 --- a/drivers/hv/hv_common.c +++ b/drivers/hv/hv_common.c @@ -21,6 +21,7 @@ #include #include #include +#include #include #include =20 @@ -128,6 +129,7 @@ int hv_common_cpu_init(unsigned int cpu) u64 msr_vp_index; gfp_t flags; int pgcount =3D hv_root_partition ? 2 : 1; + int ret; =20 /* hv_cpu_init() can be called with IRQs disabled from hv_resume() */ flags =3D irqs_disabled() ? GFP_ATOMIC : GFP_KERNEL; @@ -137,6 +139,17 @@ int hv_common_cpu_init(unsigned int cpu) if (!(*inputarg)) return -ENOMEM; =20 + if (hv_isolation_type_tdx()) { + ret =3D set_memory_decrypted((unsigned long)*inputarg, pgcount); + if (ret) { + /* It may be unsafe to free *inputarg */ + *inputarg =3D NULL; + return ret; + } + + memset(*inputarg, 0x00, pgcount * HV_HYP_PAGE_SIZE); + } + if (hv_root_partition) { outputarg =3D (void **)this_cpu_ptr(hyperv_pcpu_output_arg); *outputarg =3D (char *)(*inputarg) + HV_HYP_PAGE_SIZE; @@ -157,6 +170,8 @@ int hv_common_cpu_die(unsigned int cpu) unsigned long flags; void **inputarg, **outputarg; void *mem; + int pgcount =3D hv_root_partition ? 2 : 1; + int ret; =20 local_irq_save(flags); =20 @@ -171,6 +186,15 @@ int hv_common_cpu_die(unsigned int cpu) =20 local_irq_restore(flags); =20 + if (hv_isolation_type_tdx()) { + ret =3D set_memory_encrypted((unsigned long)mem, pgcount); + if (ret) + pr_warn("Hyper-V: Failed to encrypt input arg on cpu%d: %d\n", + cpu, ret); + /* It's unsafe to free 'mem'. */ + return 0; + } + kfree(mem); =20 return 0; --=20 2.25.1 From nobody Wed Dec 17 00:38:35 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id DCDD8C77B7C for ; Sat, 22 Apr 2023 02:20:13 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229661AbjDVCUM (ORCPT ); Fri, 21 Apr 2023 22:20:12 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51692 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229655AbjDVCUA (ORCPT ); Fri, 21 Apr 2023 22:20:00 -0400 Received: from DM5PR00CU002.outbound.protection.outlook.com (mail-centralusazon11021019.outbound.protection.outlook.com [52.101.62.19]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 4567B30C4; Fri, 21 Apr 2023 19:19:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=atsniD8imyEhsbG8mHBvDFGONOPPE9yAael21Ro0UWZsst0XrrOhNTSn5frQHlkpJpl7HgRG5uO7Qk7A3tFBNNalpaEy5PQEJEnWxuMCB+1kErZ+Tn5YVVS+/5gugt9S4QFRYIfUght7B1gY89hZ+qCENrUSTxTwPOYH5uNxPK9J8k/7tWQN0c56VirgabHvtFHgZ1d4X3myXs/VozMqdJs4YaTWkc1A4deZlRKyl6b3SzGojam+Uzv70twJVbRhFBuycXjSVfe5bnD9cP0+gFAr5wQDfS3OQ873X5LX7mEPqVpxt9dID8lg6+Fdo///J82W8ayo4PhN1Sg4nDS5xg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=+e964X0b0oioUkfFE8R97H946t3mCN5VooqM+mJVlWY=; b=Upi4IjUXuTryWF2C7r345mObMwKtSyr5Uef+viwJVZk36MKlTODQRlfwF33iH1tOQLuvusNLVOktIzrySNuouzb5uroczi9zTPaaJD2u4AovO12w2AfXNe/0CJwF+6flukrwcTHW+LovsBa9z2hmSBRknzv0r/iE1qY/dEXQcGojPKB06x361xV/OqfZIk3IMS0nW5Qtj5ugWL77v0oWksARDzeGOErt+NPps5XDtbL8gD/VCy3bN7cDhtj7YhfPQUJdW+ICiF8CkLe1R5wDoiYFTGUX5tHwZd90WUgn1LJ0TrmzPTb/mN53EzI4JirmrZOoHVhURxErjOXacl18fA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+e964X0b0oioUkfFE8R97H946t3mCN5VooqM+mJVlWY=; b=Z0OE8gTAxTbKY/sEZPcY3KXhb2L3mgHUFoyTMzFazqO2K51XSAQkhurEi/N+RHgSOb3wELQ8guptU8cKp7FPdojZPe37llln8fkI35hicKazWlZCZ1TfOGp2XG1+GuLbdNmVA7YZADVXF7DA1/6RWB0SB2tjacBmLLXDs+RT1rM= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; Received: from BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) by DM6PR21MB1418.namprd21.prod.outlook.com (2603:10b6:5:25c::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6340.11; Sat, 22 Apr 2023 02:19:37 +0000 Received: from BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30]) by BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30%5]) with mapi id 15.20.6340.014; Sat, 22 Apr 2023 02:19:37 +0000 From: Dexuan Cui To: ak@linux.intel.com, arnd@arndb.de, bp@alien8.de, brijesh.singh@amd.com, dan.j.williams@intel.com, dave.hansen@linux.intel.com, haiyangz@microsoft.com, hpa@zytor.com, jane.chu@oracle.com, kirill.shutemov@linux.intel.com, kys@microsoft.com, linux-arch@vger.kernel.org, linux-hyperv@vger.kernel.org, luto@kernel.org, mingo@redhat.com, peterz@infradead.org, rostedt@goodmis.org, sathyanarayanan.kuppuswamy@linux.intel.com, seanjc@google.com, tglx@linutronix.de, tony.luck@intel.com, wei.liu@kernel.org, x86@kernel.org, mikelley@microsoft.com Cc: linux-kernel@vger.kernel.org, Tianyu.Lan@microsoft.com, Dexuan Cui Subject: [PATCH v5 5/6] Drivers: hv: vmbus: Support TDX guests Date: Fri, 21 Apr 2023 19:17:34 -0700 Message-Id: <20230422021735.27698-6-decui@microsoft.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20230422021735.27698-1-decui@microsoft.com> References: <20230422021735.27698-1-decui@microsoft.com> X-ClientProxiedBy: CY5PR03CA0035.namprd03.prod.outlook.com (2603:10b6:930:8::20) To BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL0PR2101MB1092:EE_|DM6PR21MB1418:EE_ X-MS-Office365-Filtering-Correlation-Id: e2e9efd4-d082-4580-7a43-08db42d80583 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: sigEH+Ea1coOlBRFVgOLAV0dCjK/lISBjKsQ3W1QVwN0Z7KchoaRDY8WxNRWg6eQQ+ysEsK/SbGpAxQyEAQ4r29Da/W1RupI45WnifFu2mQ3Tic9suJi7vA21I3UYXWpR31I5CGwcu7S95mWvxNENqENzwf7xB4E28jQNEvqaGgzrSF1pIkW2nZkcwhUPRRvLfvNF79WzrSXkGudRRm8RZ/svQzUJ6TxKwDbaMgr3tGcx2E5KriEfnDu6uFnzee1DCLP6F6969WWFlUpRXTUSQUp+BH8jO0rgdBWLEJP+lSFyZZEW5+VHczTblouenEmHi5x1MXj/RGTzJf7NfFkDrq4FlqpJZkskiQKjlVwZtzcdPjppuOaJIQo/qXMoJTz/RPjjpxxBNuzRa3uXl8vxQZAeHHtXLSMwOEfb6F+9P1RPLetHE4fCJ3xDkvVmONJLjbON9LS2B+CpyxiSZVcgEdPPQIpmoILY9lcjtA7CzEPTGxFY2pHySAIXjMJz774Mzyja0oMpYVyydkaX7WwVRFrsewHrtXqB4K+bRf28EDI0EGnpmhtS9ER26dIYjwm/t39+LwcdfWGLd+hNLbFDjmLqVdkee1uzmm/EseDU9+n8Ug0R9NvgWYY0LpgOArLEssQGS7RrNTjil+8MBoyPw== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR2101MB1092.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(136003)(396003)(39860400002)(376002)(346002)(366004)(451199021)(8676002)(8936002)(38100700002)(4326008)(66946007)(66476007)(41300700001)(82960400001)(921005)(82950400001)(66556008)(316002)(786003)(6636002)(2906002)(5660300002)(10290500003)(7416002)(186003)(107886003)(86362001)(1076003)(6512007)(6506007)(36756003)(83380400001)(2616005)(6666004)(52116002)(478600001)(6486002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?6y8RTw12ZZEnWM3iMlIibYRVFwfCSudk2v495fhq1ubgqHsL6FkLNYjJefAp?= =?us-ascii?Q?FHX1CIEG4qM1UruCKTbIpYOmqbtnF8Y5PTO3pywK13Mul1HkCS+7kes1E30O?= =?us-ascii?Q?GH/SczMYGTnc3OfXRRE2l1r5U1bH5+cvfbs7KKP7DVXbLtwwMKIjZtt8jR+1?= =?us-ascii?Q?uFrjy5f+ZXBoQm0XaOQT51JRq04vq6jCh1PSS+gc1s/6HZm+OMNVwTFy4E8b?= =?us-ascii?Q?FQK0xO2y3MjFuqbUWzruZXVy+rzFm75mbMLSnnvZyE8J9QyWbwtq3a8BOvVk?= =?us-ascii?Q?jyG04f/EBiSZagr1EkRIphMsHOXZt7R3u7y3alqTlwWKdHcr6ELzIya9v9l0?= =?us-ascii?Q?UOfLvWlPH35ygqHIiu9RU3d1PASJuVDql0Xei1hLi7e6khuI7XNGRbTi5LRw?= =?us-ascii?Q?L0Uov85yI9zOWyoiCJ5utMEEIXcJPhKvHMtkUyI6IX57aJzWkPwLnCVa/TWI?= =?us-ascii?Q?UkRRtFt9yXmh4UYss4xgxsI52CBJnuOJn1BmZOnKHa76mocskhk8DdHFyAop?= =?us-ascii?Q?CnwWybfA6l5Je1yTPZPEe4n0/0myIMVMm23zsFogBfff2UiFg5ZKAxJ2F7N4?= =?us-ascii?Q?oyBDjlgQa5mOqjxxHlcW3E5F1oPewlglDzY2EvYKfjViCNgKV+poKA01B2ET?= =?us-ascii?Q?6tzuQspXfsgxW4nUSxwB9G5kQQ1I6b0wQz8EImjNQTDK1536XwOD833VWlrA?= =?us-ascii?Q?3o/zbMzjUNzcy4GgaOjYHiQPRCvFjcIORGLDOYU0Up02Bp25wGngADBufV7Y?= =?us-ascii?Q?ddLSS+6/J0bshLDkFyAxN+F9MiJygsoHq+VWcYHCfxO1bKialiMxrJ/iJC8u?= =?us-ascii?Q?ieOiXfe7AKoU1Dscr280boRpuX+pgjVpRHTOxCF6p8D5RnArkA45S/16MzMK?= =?us-ascii?Q?XQeff5f5wBLTtlYMLUltPvJcjwXxJlm1BM3S84hU6x9gQFebkMRnDOlQp6+r?= =?us-ascii?Q?P0N8haWbnziOf3+XNwEPt19KKY+yodudD55LofdTEn4Xz8ccklT2rIp9fGrB?= =?us-ascii?Q?LjFvNGtDB3St8nMZpMRFvh/LdM5GnlGCxONuzJwe1P+q9x2khr3BOS9EbY8C?= =?us-ascii?Q?rTozjyvFJE3CXs5hW4/anESMubBUJMRB7DOjsgPRIlVzBAf41FgOL9YDGglI?= =?us-ascii?Q?S5Jpw9Qv0qGGfrOAeFxbPKzeu9GraWseq7TDe7zsv1xqhbgnHRlKakZzd5pR?= =?us-ascii?Q?Kxcb5mzI+gLeDJL1WmVbsjuf3RF4MgXZFj/xCHWyxddVTDhLnG2SO9JOQEBM?= =?us-ascii?Q?VsTlvMSJuB1/wAtKgRLHnrdjdwMwsOn1KtUREtpS/wiK78mvUt8FmboY5U2Z?= =?us-ascii?Q?Q1eL8+Fyp2kmEGiRVFT4ApfKoiNPT0FG0TFoEzg4oWcKUqpzM1hHXBESJsS9?= =?us-ascii?Q?/b4UsS8hDpS7Qx2/NuFyaQIzLaU3JG7JbPN1f6+3tg59NLiimu7wX6TukZuW?= =?us-ascii?Q?P16FHDyicnnlOf5UFztiFGd3Iyv2zSHkuC2R69XBlLYj3MxpnXN1HU/s8qhO?= =?us-ascii?Q?NMVdnzVaDNq2tHd7qLZZuOVUxOc74R3mEpM/Ewv0NLKzUc9+PRINBREhwAFc?= =?us-ascii?Q?MWj2Z1ZAAAiAveCUhR5/KIGxC6GHaSd1QuLUdlsWZgpXhO+zP/D4FjO9RHU2?= =?us-ascii?Q?heJdd0dGJFK+RSJS61FkOEZvFDENsLWdrCNta8LJy5wF?= X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: e2e9efd4-d082-4580-7a43-08db42d80583 X-MS-Exchange-CrossTenant-AuthSource: BL0PR2101MB1092.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2023 02:19:37.7544 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: Q5zAcc5Yn8e4R9nJSljbnKWVBJs2/69KtzWpX1tpoaKWWpo0vYSmWl4CJlWR/vqR7TNldbHktiLyvWf/1aNLaA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR21MB1418 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Add Hyper-V specific code so that a TDX guest can run on Hyper-V: No need to use hv_vp_assist_page. Don't use the unsafe Hyper-V TSC page. Don't try to use HV_REGISTER_CRASH_CTL. Don't trust Hyper-V's TLB-flushing hypercalls. Don't use lazy EOI. Share SynIC Event/Message pages and VMBus Monitor pages with the host. Use pgprot_decrypted(PAGE_KERNEL)in hv_ringbuffer_init(). Signed-off-by: Dexuan Cui --- arch/x86/hyperv/hv_apic.c | 6 ++-- arch/x86/hyperv/hv_init.c | 19 +++++++++--- arch/x86/kernel/cpu/mshyperv.c | 21 ++++++++++++- drivers/hv/hv.c | 54 ++++++++++++++++++++++++++++++++-- 4 files changed, 90 insertions(+), 10 deletions(-) Changes in v2: Used a new function hv_set_memory_enc_dec_needed() in __set_memory_enc_pgtable(). Added the missing set_memory_encrypted() in hv_synic_free(). Changes in v3: Use pgprot_decrypted(PAGE_KERNEL)in hv_ringbuffer_init(). (Do not use PAGE_KERNEL_NOENC, which doesn't exist for ARM64). Used cc_mkdec() in hv_synic_enable_regs(). ms_hyperv_init_platform(): Explicitly do not use HV_X64_REMOTE_TLB_FLUSH_RECOMMENDED. Explicitly do not use HV_X64_APIC_ACCESS_RECOMMENDED. Enabled __send_ipi_mask() and __send_ipi_one() for TDX guests. Changes in v4: A minor rebase to Michael's v7 DDA patchset. I'm very happy that I can drop my v3 change to arch/x86/mm/pat/set_memory.c due to Michael's work. Changes in v5: Added memset() to clear synic_message_page and synic_event_page() after set_memory_decrypted(). Rebased the patch since "post_msg_page" has been removed in hyperv-next. Improved the error handling in hv_synic_alloc()/free() [Michael Kelley] diff --git a/arch/x86/hyperv/hv_apic.c b/arch/x86/hyperv/hv_apic.c index fb8b2c088681..16919c7b3196 100644 --- a/arch/x86/hyperv/hv_apic.c +++ b/arch/x86/hyperv/hv_apic.c @@ -173,7 +173,8 @@ static bool __send_ipi_mask(const struct cpumask *mask,= int vector, (exclude_self && weight =3D=3D 1 && cpumask_test_cpu(this_cpu, mask))) return true; =20 - if (!hv_hypercall_pg) + /* A TDX guest doesn't use hv_hypercall_pg. */ + if (!hv_isolation_type_tdx() && !hv_hypercall_pg) return false; =20 if ((vector < HV_IPI_LOW_VECTOR) || (vector > HV_IPI_HIGH_VECTOR)) @@ -227,7 +228,8 @@ static bool __send_ipi_one(int cpu, int vector) =20 trace_hyperv_send_ipi_one(cpu, vector); =20 - if (!hv_hypercall_pg || (vp =3D=3D VP_INVAL)) + /* A TDX guest doesn't use hv_hypercall_pg. */ + if ((!hv_isolation_type_tdx() && !hv_hypercall_pg) || (vp =3D=3D VP_INVAL= )) return false; =20 if ((vector < HV_IPI_LOW_VECTOR) || (vector > HV_IPI_HIGH_VECTOR)) diff --git a/arch/x86/hyperv/hv_init.c b/arch/x86/hyperv/hv_init.c index f175e0de821c..f28357ecad7d 100644 --- a/arch/x86/hyperv/hv_init.c +++ b/arch/x86/hyperv/hv_init.c @@ -79,7 +79,7 @@ static int hyperv_init_ghcb(void) static int hv_cpu_init(unsigned int cpu) { union hv_vp_assist_msr_contents msr =3D { 0 }; - struct hv_vp_assist_page **hvp =3D &hv_vp_assist_page[cpu]; + struct hv_vp_assist_page **hvp; int ret; =20 ret =3D hv_common_cpu_init(cpu); @@ -89,6 +89,7 @@ static int hv_cpu_init(unsigned int cpu) if (!hv_vp_assist_page) return 0; =20 + hvp =3D &hv_vp_assist_page[cpu]; if (hv_root_partition) { /* * For root partition we get the hypervisor provided VP assist @@ -398,11 +399,21 @@ void __init hyperv_init(void) if (hv_common_init()) return; =20 - hv_vp_assist_page =3D kcalloc(num_possible_cpus(), - sizeof(*hv_vp_assist_page), GFP_KERNEL); + /* + * The VP assist page is useless to a TDX guest: the only use we + * would have for it is lazy EOI, which can not be used with TDX. + */ + if (hv_isolation_type_tdx()) + hv_vp_assist_page =3D NULL; + else + hv_vp_assist_page =3D kcalloc(num_possible_cpus(), + sizeof(*hv_vp_assist_page), + GFP_KERNEL); if (!hv_vp_assist_page) { ms_hyperv.hints &=3D ~HV_X64_ENLIGHTENED_VMCS_RECOMMENDED; - goto common_free; + + if (!hv_isolation_type_tdx()) + goto common_free; } =20 if (hv_isolation_type_snp()) { diff --git a/arch/x86/kernel/cpu/mshyperv.c b/arch/x86/kernel/cpu/mshyperv.c index a87fb934cd4b..e9106c9d92f8 100644 --- a/arch/x86/kernel/cpu/mshyperv.c +++ b/arch/x86/kernel/cpu/mshyperv.c @@ -405,8 +405,27 @@ static void __init ms_hyperv_init_platform(void) =20 if (hv_get_isolation_type() =3D=3D HV_ISOLATION_TYPE_SNP) static_branch_enable(&isolation_type_snp); - else if (hv_get_isolation_type() =3D=3D HV_ISOLATION_TYPE_TDX) + else if (hv_get_isolation_type() =3D=3D HV_ISOLATION_TYPE_TDX) { static_branch_enable(&isolation_type_tdx); + + /* + * The GPAs of SynIC Event/Message pages and VMBus + * Moniter pages need to be added by this offset. + */ + ms_hyperv.shared_gpa_boundary =3D cc_mkdec(0); + + /* Don't use the unsafe Hyper-V TSC page */ + ms_hyperv.features &=3D ~HV_MSR_REFERENCE_TSC_AVAILABLE; + + /* HV_REGISTER_CRASH_CTL is unsupported */ + ms_hyperv.misc_features &=3D ~HV_FEATURE_GUEST_CRASH_MSR_AVAILABLE; + + /* Don't trust Hyper-V's TLB-flushing hypercalls */ + ms_hyperv.hints &=3D ~HV_X64_REMOTE_TLB_FLUSH_RECOMMENDED; + + /* A TDX VM must use x2APIC and doesn't use lazy EOI */ + ms_hyperv.hints &=3D ~HV_X64_APIC_ACCESS_RECOMMENDED; + } } =20 if (hv_max_functions_eax >=3D HYPERV_CPUID_NESTED_FEATURES) { diff --git a/drivers/hv/hv.c b/drivers/hv/hv.c index 4e1407d59ba0..fa7dce26ec67 100644 --- a/drivers/hv/hv.c +++ b/drivers/hv/hv.c @@ -18,6 +18,7 @@ #include #include #include +#include #include #include #include "hyperv_vmbus.h" @@ -116,6 +117,7 @@ int hv_synic_alloc(void) { int cpu; struct hv_per_cpu_context *hv_cpu; + int ret =3D -ENOMEM; =20 /* * First, zero all per-cpu memory areas so hv_synic_free() can @@ -159,6 +161,28 @@ int hv_synic_alloc(void) goto err; } } + + /* It's better to leak the page if the decryption fails. */ + if (hv_isolation_type_tdx()) { + ret =3D set_memory_decrypted( + (unsigned long)hv_cpu->synic_message_page, 1); + if (ret) { + pr_err("Failed to decrypt SYNIC msg page\n"); + hv_cpu->synic_message_page =3D NULL; + goto err; + } + + ret =3D set_memory_decrypted( + (unsigned long)hv_cpu->synic_event_page, 1); + if (ret) { + pr_err("Failed to decrypt SYNIC event page\n"); + hv_cpu->synic_event_page =3D NULL; + goto err; + } + + memset(hv_cpu->synic_message_page, 0, PAGE_SIZE); + memset(hv_cpu->synic_event_page, 0, PAGE_SIZE); + } } =20 return 0; @@ -167,18 +191,40 @@ int hv_synic_alloc(void) * Any memory allocations that succeeded will be freed when * the caller cleans up by calling hv_synic_free() */ - return -ENOMEM; + return ret; } =20 =20 void hv_synic_free(void) { int cpu; + int ret; =20 for_each_present_cpu(cpu) { struct hv_per_cpu_context *hv_cpu =3D per_cpu_ptr(hv_context.cpu_context, cpu); =20 + /* It's better to leak the page if the encryption fails. */ + if (hv_isolation_type_tdx()) { + if (hv_cpu->synic_message_page) { + ret =3D set_memory_encrypted((unsigned long) + hv_cpu->synic_message_page, 1); + if (ret) { + pr_err("Failed to encrypt SYNIC msg page\n"); + hv_cpu->synic_message_page =3D NULL; + } + } + + if (hv_cpu->synic_event_page) { + ret =3D set_memory_encrypted((unsigned long) + hv_cpu->synic_event_page, 1); + if (ret) { + pr_err("Failed to encrypt SYNIC event page\n"); + hv_cpu->synic_event_page =3D NULL; + } + } + } + free_page((unsigned long)hv_cpu->synic_event_page); free_page((unsigned long)hv_cpu->synic_message_page); } @@ -215,7 +261,8 @@ void hv_synic_enable_regs(unsigned int cpu) if (!hv_cpu->synic_message_page) pr_err("Fail to map synic message page.\n"); } else { - simp.base_simp_gpa =3D virt_to_phys(hv_cpu->synic_message_page) + simp.base_simp_gpa =3D + cc_mkdec(virt_to_phys(hv_cpu->synic_message_page)) >> HV_HYP_PAGE_SHIFT; } =20 @@ -234,7 +281,8 @@ void hv_synic_enable_regs(unsigned int cpu) if (!hv_cpu->synic_event_page) pr_err("Fail to map synic event page.\n"); } else { - siefp.base_siefp_gpa =3D virt_to_phys(hv_cpu->synic_event_page) + siefp.base_siefp_gpa =3D + cc_mkdec(virt_to_phys(hv_cpu->synic_event_page)) >> HV_HYP_PAGE_SHIFT; } =20 --=20 2.25.1 From nobody Wed Dec 17 00:38:35 2025 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 757B0C7618E for ; Sat, 22 Apr 2023 02:20:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229689AbjDVCUX (ORCPT ); Fri, 21 Apr 2023 22:20:23 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:51694 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229668AbjDVCUJ (ORCPT ); Fri, 21 Apr 2023 22:20:09 -0400 Received: from DM5PR00CU002.outbound.protection.outlook.com (mail-centralusazon11021019.outbound.protection.outlook.com [52.101.62.19]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 633D7359A; Fri, 21 Apr 2023 19:19:46 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Bl0znskAVDvviTe0eOb9iEorssbHkVbAafu/JQFNLfPyxjCrsHJTTuz2Xijsw0HbP9ouh/rEVUhTKy/OXhpXcudHb/MPRrPYDNGPzBwm3DFjYhAjEm0kEwpjfcvEUgGvPugDJdfWIRXKcsFLjQ17GktSCllq3J+/lTRz5HbUb/x038Yrl2LL4zO4LskV5QpdeCaIevNGHSdbfbPG4oc0iXI86tM+g5QMLzWiQbTuYgWol933gzte2KEf+ofILbz+F0FvXY67bJlselofE/F9qfrf8wCc4gCDGf5ixXnX8jGsnAG6WMdnGZSp3kZGdxfLnPyikyT5BpbK00KUrKfi7w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=h4uuvpZIn5e0Rfx2QAmnmA9Qsc2/gmvKc2dBDzzOTkc=; b=OVgrNd8GE5iZqSDkDQRxJFoy2SJTmdFbHBb6XDN/e/L2s0CS72cWNNPBoCBsCnyUjYgG9YALEMQAYobUeU32xeGaPxJ/a6IfQEbI8UElNoD5ZQvYlx7Q5NKZ+bb9KFAktYoUpR772GaQzBUa+2C8r6fMKMbDraEqoOyjrlrHJfP1LkOT0B4+B5Kav6SX7OR4Y1UEX9mWZLGN6AT7vfUftiuxNfbgtscGOaMo08bc1EThih6dwq60LwtMhU+Rg681hWRhIG2hGnuzeF9Lll/xmGQbybUF+WieTxep7kfZD8OJI/HTDKn51zyWJs0tewjFt83h9M8L/rALLUUbUa5N8w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=microsoft.com; dmarc=pass action=none header.from=microsoft.com; dkim=pass header.d=microsoft.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=h4uuvpZIn5e0Rfx2QAmnmA9Qsc2/gmvKc2dBDzzOTkc=; b=JWy+xl2h87LiJey+4uyFOw7mcvfsIpimWEzxD+zenEDP1+4RXdjWzI/WrWyYZKMZLw1UVOxkPNWBl5onx6+7LI7uqj/X2hIiohhAVJrTCDLid03U1qZcC5pH9zGo5VBxF6WiDfgfo4ZJ1PQ6GMpRSC1WZbCVwHyOkmD3nU7twTU= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=microsoft.com; Received: from BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) by DM6PR21MB1418.namprd21.prod.outlook.com (2603:10b6:5:25c::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6340.11; Sat, 22 Apr 2023 02:19:40 +0000 Received: from BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30]) by BL0PR2101MB1092.namprd21.prod.outlook.com ([fe80::932e:24fe:fc1b:5d30%5]) with mapi id 15.20.6340.014; Sat, 22 Apr 2023 02:19:39 +0000 From: Dexuan Cui To: ak@linux.intel.com, arnd@arndb.de, bp@alien8.de, brijesh.singh@amd.com, dan.j.williams@intel.com, dave.hansen@linux.intel.com, haiyangz@microsoft.com, hpa@zytor.com, jane.chu@oracle.com, kirill.shutemov@linux.intel.com, kys@microsoft.com, linux-arch@vger.kernel.org, linux-hyperv@vger.kernel.org, luto@kernel.org, mingo@redhat.com, peterz@infradead.org, rostedt@goodmis.org, sathyanarayanan.kuppuswamy@linux.intel.com, seanjc@google.com, tglx@linutronix.de, tony.luck@intel.com, wei.liu@kernel.org, x86@kernel.org, mikelley@microsoft.com Cc: linux-kernel@vger.kernel.org, Tianyu.Lan@microsoft.com, Dexuan Cui Subject: [PATCH v5 6/6] x86/hyperv: Fix serial console interrupts for TDX guests Date: Fri, 21 Apr 2023 19:17:35 -0700 Message-Id: <20230422021735.27698-7-decui@microsoft.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20230422021735.27698-1-decui@microsoft.com> References: <20230422021735.27698-1-decui@microsoft.com> X-ClientProxiedBy: CY5PR03CA0035.namprd03.prod.outlook.com (2603:10b6:930:8::20) To BL0PR2101MB1092.namprd21.prod.outlook.com (2603:10b6:207:30::23) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: BL0PR2101MB1092:EE_|DM6PR21MB1418:EE_ X-MS-Office365-Filtering-Correlation-Id: 70738538-3c4e-4692-b301-08db42d806c7 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: KFoHNzf1rHRlHg37onlSbgFwkqcPOopxgM+Ee3TJDvzdBhUQw8FDCic4EWrXmNWBW1o3MhdQOG1QNWv1nxUiRzILy0tRcAw4cgn/PYvvSRvNdcPIgna80yi8UUkCt38+og+6tiZVet1n6Vh5YlVxTbEHtJFWnm2BNzGAp9t+gUlGl9ihZMaNX7ASTB2R5uXerE5y3rwP0JmioRNehX8os3vNvReTd1wdVvjec7pW2k7wnKv5NJWrNECViZ0N5U8tO4sd9D0UxzYNcjA5W9eYrazUYuoatQt/4UlyczLCdCf4Xg86HyaViv/xzmdQYzuoB1ajuVepsef3uA3I+GRHNv/rkADSEj1JFdtkttUW3jQlnlhEdDGTvm5MWIN8/9GuQSEpLE3wRLYNp1tis3/7pfbY1x+mkfb9Buz7QkFPdpZNOFhHX1GLNJDDsdmXS9hVY52XjVY0A1C8X3l2VuO3k3GFRu4qIV1Yfpavch9Ry0sh5zRSX3GQBHGsA759F+yaH5T/njBCbW+IOnH9XQln4eqolZ5jvD+ZRub9B4XerFLpE40BkWJicFFYqWHULaFyvsxShep7wcUk/iQTVk8yfgugynOVYanIDM7zjsgtgbt/p09B4vExN/B7kGTIr6apkJtpIy/1+Om3YAmvNc8JXQ== X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:BL0PR2101MB1092.namprd21.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230028)(4636009)(136003)(396003)(39860400002)(376002)(346002)(366004)(451199021)(8676002)(8936002)(38100700002)(4326008)(66946007)(66476007)(41300700001)(82960400001)(921005)(82950400001)(66556008)(316002)(786003)(6636002)(2906002)(5660300002)(10290500003)(7416002)(186003)(107886003)(86362001)(1076003)(6512007)(6506007)(36756003)(83380400001)(2616005)(6666004)(52116002)(478600001)(6486002);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?ObH5N+OsBbwl8nPcslKPbBwPKwEoQF3Ntgtbl/DZ/qZwgtRrSNQSeQPXh+Ey?= =?us-ascii?Q?UbWzv52JIqEcV1N9alGoiBHrKByClnvsZPWQ39B331XT8yKQX7SJmCGudzeA?= =?us-ascii?Q?A49uiLv+YBu2X3hovW8vitf2/qVGfIUTT9NzwMeK4wX/1GXrTw5klTywowHT?= =?us-ascii?Q?8IEdDW+JHw/9R2SNtOCukNlJG8bugxTTib9A4ljw1mlRqJok3FGl/A6UnQsa?= =?us-ascii?Q?K1TbW5m8BA9LhUfhYr/8jjbnlMooZRIbb+dEjydXd7KZ4E/W1Is547DyjV2o?= =?us-ascii?Q?kUBfh3QIHIXLfzVMlLrXE9JXdHbQqFuKKdmUX6PHB+dR+7rv+apzZN5aATyD?= =?us-ascii?Q?a16kueqB4UZDhHQ1PK/s9+Ys3YcZkQiMSGE7xWdDXBqXr3i1iefSUBLWIJlJ?= =?us-ascii?Q?seMZftc9kI+rBEoe1+sHXJViGahC8/CtFa7SwobgOyOSnI03vF5TMki+h8vV?= =?us-ascii?Q?MMafPoQ8dY64/p0bqwTV7/nAoMio+uSjdDPnV/r150b7WV0Yd84o/s8FOWOo?= =?us-ascii?Q?Ag3OMDgFrKEydPg8enm+ED8W1ez2UWxpdmDq5t/X6XnRUwLyRx1zkaOY57Vt?= =?us-ascii?Q?pxbMe8cPnViWm4St5Hx4XtfyliF80zONQGVrfbHDfDImgoSA86SdN5uDIHX5?= =?us-ascii?Q?yyH5DRHMkoeBntfnSJx0Rtn3dQed9MUA+p80h3zS9ZnZGNLByV7Ay/4Lmf+B?= =?us-ascii?Q?ePKKQ4yzd0kxxaDM3CUz5V5DQ6dr8FG8rBBsS1Np3LiCYX7MtcN/W+embZKj?= =?us-ascii?Q?V2k0IO0JZfY76I/aBlWYYEaBuIrBKZs5Dh8Axqk37Rm/YUp0SfD2kLVtQQq2?= =?us-ascii?Q?mWZYMqeyJT3nNmKAM45v3DxR7YPNWMS6jCSzqM0dMof3p2Lfayv8R4xt4faG?= =?us-ascii?Q?w+A+Jp/Jex/X5W3HRP1htSlf1pgaK8fIhCtKpa+1SLQpbQptayFpLsaXGuAA?= =?us-ascii?Q?xQE3NHrPIg0RE+iaVtlbHDpZthuOLXP0IYa9Sw9miNeBHIflYQOTt2i9Ttz1?= =?us-ascii?Q?2EVQNC6kvMgK3ry1Q8KNoe6wsJ0i9wno9scabzIPKSAZO+ZX6m7KJ1nXW1X5?= =?us-ascii?Q?roz/CJlo9qa+8eJJFHnOoN5kZsPSdsRUKN5vm6RCWT3Pdz8U3hZnOeBF3iN+?= =?us-ascii?Q?ZS+fVtApAxB4H3LgnOLXbT69rEUI/4GEdjy+XLF1Qghtu+Dzuwk+EvpxpdFJ?= =?us-ascii?Q?YWOeLtKGvwsTTjjtS0a6RIUA345ekOS9q2sxK4GdEH8DQ3otdb0Yqguo+sBQ?= =?us-ascii?Q?ucR/jWJMAXf0HYovedbiwjxMLn/ekwmZlUGtMkZhY86XJcKavHHmRZFkZ1Sq?= =?us-ascii?Q?Cj1mdfnz1Tkl9nLZuVmMl/NTg6iHmqw0tX4b1yDgcihwsa/WJgj1Sr0l5CqJ?= =?us-ascii?Q?6byjL+bLwJ+YaWVKZnyiwhux9nxjBY1YWBCkYiWAJdGYFUoO4dpRMeNjtgk3?= =?us-ascii?Q?6JTBLQ48wWsIKqfjnCqAsh8ymvJWdLbDrAU3jta/Ph/KkNJ32fDSvrCziMlY?= =?us-ascii?Q?Sumn6a7DzsD9ERhOKyipYIYVTwDTOV2uwE4RfuatTN1VmvndLH1feWFYBZkl?= =?us-ascii?Q?Iqf/YaZwm2v+umg4ag0zTgRAPZyxDBcwBhBFvCTgjKwI+mPz4Wz75Wgs1ijc?= =?us-ascii?Q?c1346ZskR+mVx9zpx2KKtlOHqn1cKRRuMsxs9OGEpvOD?= X-OriginatorOrg: microsoft.com X-MS-Exchange-CrossTenant-Network-Message-Id: 70738538-3c4e-4692-b301-08db42d806c7 X-MS-Exchange-CrossTenant-AuthSource: BL0PR2101MB1092.namprd21.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2023 02:19:39.8798 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 72f988bf-86f1-41af-91ab-2d7cd011db47 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: DBAA1m8IiHIQe50fKZgZfgWVu6xBBWPjm6sqBDrTkrSitfXRvjDtJ2MePYjUbAl8ml7SPJwzvSRVFjGlkjRgPQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR21MB1418 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" When a TDX guest runs on Hyper-V, the UEFI firmware sets the HW_REDUCED flag, and consequently ttyS0 interrupts can't work. Fix the issue by overriding x86_init.acpi.reduced_hw_early_init(). Reviewed-by: Michael Kelley Signed-off-by: Dexuan Cui --- arch/x86/kernel/cpu/mshyperv.c | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) Changes since v1: None. Changes in v5: Improved the comment [Michael Kelley] Added Michael's Reviewed-by. diff --git a/arch/x86/kernel/cpu/mshyperv.c b/arch/x86/kernel/cpu/mshyperv.c index e9106c9d92f8..942170ea6a5d 100644 --- a/arch/x86/kernel/cpu/mshyperv.c +++ b/arch/x86/kernel/cpu/mshyperv.c @@ -318,6 +318,26 @@ static void __init hv_smp_prepare_cpus(unsigned int ma= x_cpus) } #endif =20 +/* + * When a TDX guest runs on Hyper-V, the firmware sets the HW_REDUCED flag= : see + * acpi_tb_create_local_fadt(). Consequently ttyS0 interrupts can't work b= ecause + * request_irq() -> ... -> irq_to_desc() returns NULL for ttyS0. This happ= ens + * because mp_config_acpi_legacy_irqs() sees a nr_legacy_irqs() of 0, so it + * doesn't initialize the array 'mp_irqs[]', and later setup_IO_APIC_irqs(= ) -> + * find_irq_entry() fails to find the legacy irqs from the array, and hence + * doesn't create the necessary irq description info. + * + * Clone arch/x86/kernel/acpi/boot.c: acpi_generic_reduced_hw_init() here, + * except don't change 'legacy_pic'. It keeps its default value + * 'default_legacy_pic'. mp_config_acpi_legacy_irqs() sees a non-zero + * nr_legacy_irqs(), and eventually serial console interrupts works proper= ly. + */ +static void __init reduced_hw_init(void) +{ + x86_init.timers.timer_init =3D x86_init_noop; + x86_init.irqs.pre_vector_init =3D x86_init_noop; +} + static void __init ms_hyperv_init_platform(void) { int hv_max_functions_eax; @@ -425,6 +445,8 @@ static void __init ms_hyperv_init_platform(void) =20 /* A TDX VM must use x2APIC and doesn't use lazy EOI */ ms_hyperv.hints &=3D ~HV_X64_APIC_ACCESS_RECOMMENDED; + + x86_init.acpi.reduced_hw_early_init =3D reduced_hw_init; } } =20 --=20 2.25.1