From nobody Fri May 3 23:02:13 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1dmarc=pass fromdomain=citrix.com); dmarc=pass(p=reject dis=none) header.from=citrix.com ARC-Seal: i=2; a=rsa-sha256; t=1622223615; cv=pass; d=zohomail.com; s=zohoarc; b=Z0Y2izCq8eolYc03hbHUk8zqrCVtFD/Vef4++GlP6r2/33oRopQLCWYC2IPlwkb1aFaF7HX7J0bljVpUtB3JzOuiXIJ1cCE2pliBlVn1zC+j3zEasOGw5A6eYoysPIZVpO6ghYeUCmLCWgRCd4P9JGAJD+XCi3KUT91j4wg/ON4= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1622223615; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=mrKXXGDcgQcBgxROwP6XsmUEHjRwDViGegef7nDL4CI=; b=SagTbkk+LBoEFhzsTlvfOYcXxkvoEfX/XBVWJMgBxN3ugchAeNu0BxPqdmcibvJ2zm0CKzCHdbkdim4lZYus3sQObvMjhwqX7hOYMe04p9dgRwmHSnhUGOfO295lpEs89bbCsEtLCsLoLNMDMRolb/sgEPKHpCsRxNLKESzjERo= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1dmarc=pass fromdomain=citrix.com); dmarc=pass header.from= (p=reject dis=none) header.from= Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 16222236151381004.2170274601224; Fri, 28 May 2021 10:40:15 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.134041.249611 (Exim 4.92) (envelope-from ) id 1lmgSj-0000r7-VX; Fri, 28 May 2021 17:39:57 +0000 Received: by outflank-mailman (output) from mailman id 134041.249611; Fri, 28 May 2021 17:39:57 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lmgSj-0000qy-S7; Fri, 28 May 2021 17:39:57 +0000 Received: by outflank-mailman (input) for mailman id 134041; Fri, 28 May 2021 17:39:56 +0000 Received: from all-amaz-eas1.inumbo.com ([34.197.232.57] helo=us1-amaz-eas2.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lmgSi-0000ZU-SW for xen-devel@lists.xenproject.org; Fri, 28 May 2021 17:39:56 +0000 Received: from esa4.hc3370-68.iphmx.com (unknown [216.71.155.144]) by us1-amaz-eas2.inumbo.com (Halon) with ESMTPS id 33356d93-6df3-44d8-b236-1a4f0aca9dce; Fri, 28 May 2021 17:39:52 +0000 (UTC) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 33356d93-6df3-44d8-b236-1a4f0aca9dce DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=citrix.com; s=securemail; t=1622223592; h=from:to:cc:subject:date:message-id:in-reply-to: references:content-transfer-encoding:mime-version; bh=IxeGPsjlqm7m9YueumxWU8Dv7M+sUj7g2Br9UfeV1pg=; b=D2v2cJuglfoll1Qur8cfLZWgT2zD2UUURlPUtVAerkoFy0db/Le+Eywp 62v3z1eYDcxe6quYaELCBI86frkMLvNzZWyhHkxGK1Vc2pUaeFOseIu7U l9k6ITJV0AQqXmHT0/42NlWpI0glKx637odZVm+oiglTFdNCpp2s5CYQw U=; Authentication-Results: esa4.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com IronPort-SDR: OEJBosAulKLIEV0aQvn9EvzWgualTiQrGPxgzySXbcO7Mp3468v++PvvoGUkdocKOqASmkUdrj ne03uh6TG39czepTOwp3+y8H6WMzSH87badLj3RbM0JIyZ6W0S+F8Wc7aRtAqMF8qbIEgD5cwB 8Uv2yvhSr1UNl5U5LxDbqUk9JBI7KwMYnoISbOy9MNKVwjJygDJu0A0lxHHfpR4KwPKm4K2fRZ 1fHGBbZN4fioBRiFg5AcGmb1JuUVN4HYwjeZHbRupVaACI/0+XZBor9IKsRecHCo8pbighT7hT DGU= X-SBRS: 5.1 X-MesageID: 46432765 X-Ironport-Server: esa4.hc3370-68.iphmx.com X-Remote-IP: 162.221.158.21 X-Policy: $RELAYED IronPort-HdrOrdr: A9a23:oiRMgqDDn9Pj/DTlHeiUsceALOsnbusQ8zAXPh9KJyC9I/b2qy nxppgmPH/P6Ar4WBkb6Le90c67MAzhHP9OkPUs1NKZPTUO11HYVb2KgbGSpgEIXheOjNK1tp 0QAJSWaueAdWSS5PySiGLTfqdCsbv3hZxAx92uskuFJTsaG52IhD0JbDpzfHcGIDWvUvECZe uhD4d81nWdkRt9VLX0OlA1G8z44/HbnpPvZhALQzQ97hOVsD+u4LnmVzCFwxY3SVp0sPQf2F mAtza8yrSosvm9xBOZ/XTU9Y5qlNzozcYGLNCQi/ISNi7nhm+TFcRcsoW5zXUISdyUmRIXeI GmmWZmAy0z0QKRQoiNm2qu5+G6uwxerUMLoDSj8AneSc+QfkNxNyMOv/MBTvN1g3BQ9e2U65 g7qF5xgaAnRi8orB6Nk+QgaCsa4HZc2UBS6tL7r0YvHLf2O4Uh4bD2wituYd899XXBmf4a+a 9VfZnh2Mo= X-IronPort-AV: E=Sophos;i="5.83,230,1616472000"; d="scan'208";a="46432765" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=TyQhfQvVdDRHjgYgZ6P2fgSN5h+BVxSa9/JyFn82AenngFHR2mxmgpY8AhwCYrG18GzQ4zHgfXxJwvvHAe3mLaC4dXzCBaobUqRjmNyx32NSGAwpjp0Ixhw8ihv6uSX6I7YkQzKVi6AqxbZAriDZvSuqIGx1X1za3PsAXShaHZQGtmSsVA5pOteJStiENe+Lp/IyXZRdwFZaVjAE4JMvO2YNaAsxlM6AY8C49vK8Iwu3SMxc/XHsVOQ//1If9Gsjnqfz0k1wBzcPZ5YrTZ7F/XM94oVxdCgNHODk4kLsA8nIZA4Dx2PTY8HmUizHsuGmLdJW90cvyFpPLvrP6ipL7w== 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-SenderADCheck; bh=mrKXXGDcgQcBgxROwP6XsmUEHjRwDViGegef7nDL4CI=; b=LF+thI9QK+mISA6xZ9Ac7EcMrYMCGOaCShxcMqzUDktUuv3NnV50KNcJpfD9FmPzI2ZYJKuaKlCW/MCse+XuuwVHAJOdYHdJ+alhXREmISWoa6BpytlKTrBDpBqRq0vkkRdaz0fJaBAfFd47wi/QMa0AkfcrhUASiaIesjJOGRKfWW9Bk8F5b7QivlupGhhb9s9Pc6bzWkNvhp1QjZ/OTET7/W0+XjIEvM9wmoH9oY1rBCvx7Rsj+Lvn8gmXYmxtUVpfb1sApFnoewmD+Nuj3v2Px+0aMv1pf7z6nGybT0/qTn3ZC5I/sVXXpkQxFJrRG4O/nPZhzjtTO5E1w+n23g== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=citrix.onmicrosoft.com; s=selector2-citrix-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=mrKXXGDcgQcBgxROwP6XsmUEHjRwDViGegef7nDL4CI=; b=rucvN+hcCLRZCJYS9jPBByNYtA6za7XyymGOBkonFy1O+oPiR7ADEwA4q4NR0/mUrcV/SyZ8EQtN79Msp+4SeF5MkQ2pyVS0hVeBE0M3pq/PSYFcINDQkA1Z5cktGPkEuQLXUtBVdh87LwpWK6Kx1UkpP5MyHNmNxInhPrkyi3k= From: Roger Pau Monne To: CC: Roger Pau Monne , Jan Beulich , Andrew Cooper , Wei Liu Subject: [PATCH 1/3] x86/mtrr: remove stale function prototype Date: Fri, 28 May 2021 19:39:33 +0200 Message-ID: <20210528173935.29919-2-roger.pau@citrix.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210528173935.29919-1-roger.pau@citrix.com> References: <20210528173935.29919-1-roger.pau@citrix.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: MR2P264CA0130.FRAP264.PROD.OUTLOOK.COM (2603:10a6:500:30::22) To DS7PR03MB5608.namprd03.prod.outlook.com (2603:10b6:5:2c9::18) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 6b1ace0f-e868-4886-926e-08d921ff97ed X-MS-TrafficTypeDiagnostic: DM4PR03MB5966: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:751; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: lm8mdrUDnkJCAe5TO/og+SxKU2zuS97Yvl0zM44fRBoHk3AIjFl/l0RjBnUWkCsAiblSYqxPT+f/Ref/+02efFVzG1Cw2jzH3fBKQGJADFdjs3egcyAc40bD5HhnizfdfBGPiQUq/xcXzz6jrcZDsHg2EsA1KrMRgdCUPJHrrVkAMFHd92aeyVo5/t/GVbPXR4vh8uhXCXSCZtcW/2NnvqEdx2tZ1etWkHCzL/VnEt1j28oO1o94kYZSXH6h6kjCGHcpVHpAyto8W1nnNtrF0695E5OqCkJG/xXLwseMLAIYg7wgiYa53U+VhGWiW0MBHPZexXwBzW7CfcxbKlc+MP2MPW7G6uGd0DEL/ItFXOJ9LfonQGvJHPQJfEleAg7x84+FJEXQ7Osj7HitN/Ue7JIBeoMmR9O+P6ffC1kQ2fDa2YanKOKb9+I8i8Q8LuzTimjLdnfJ9KTR3MFFoOtFOAoiI9mYk3d/JuhYlz2kN0Q8hGHyzG5uaavEROXBV307LWbVshCNvN9/1GcOy8mDh3Y1jwZzRbXyjAvg/4iHiPRq5WSlzxhKVED2DUnWnecKv4Dr8xTWSYyUSzFmyWcqIz3jW1dfmFhj1vQpgQnGgRU= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DS7PR03MB5608.namprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(396003)(39860400002)(366004)(376002)(136003)(346002)(6666004)(36756003)(2906002)(26005)(2616005)(66476007)(316002)(66556008)(186003)(66946007)(16526019)(54906003)(956004)(4326008)(38100700002)(6486002)(8676002)(4744005)(83380400001)(478600001)(8936002)(6496006)(86362001)(6916009)(1076003)(5660300002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?Vm9aS0l2bkwyQWNUNzZNSU9KZHVaK2hKc0FFZHdBQ0pyaU55UUhBcU1hWTRw?= =?utf-8?B?SlNxU2FpYnNqTzZ2eVh0RVFrNllWeXdMaGltTVVCbEt5TTdic0NzRWdkcFF0?= =?utf-8?B?dW1GeFVXTVMxbHhCQ0lqNzhZUDFtOVNOUmU3VHJ4NFpFZkM3RkVlTm5EQnZV?= =?utf-8?B?c0dDcWd3cUFhL2dONHF5WS85R254VHNZVks2ODBmSUtmNHhWTU5MekNNS3VZ?= =?utf-8?B?WVIwMVdJaUdZeE5TN2gvcHZwenFjdmx0RHZtOFJma1V2L3dkdUlVVDE4ZlhQ?= =?utf-8?B?cnNVamZubHVEWUNFV3Z5M0l3YStBcCsyODFpQ21lSzVyOUg3QVBxdzdTSlFC?= =?utf-8?B?Tk5mRFhveTM3Wnl3MnFIM1lyVFBRSE9BR3Mwa1pvK2xaSHVTYW83SzQwSmZr?= =?utf-8?B?ZjFMTFBlODBva0RLdmpGS245emxwWVA4cjgyY3hCRXBhUGpNQThBRDVxZFNT?= =?utf-8?B?UEhYaXcrY0YvcU9ZTmxrMlNwSjVYeXBCeERnc21lZU5qN0lkMkd0cGtkL1Nw?= =?utf-8?B?RFRwMk5hQU9CdStFNHU3akJMSXpha2xXanlGTGdaSUZBeS9GVXV2M3BCSFM2?= =?utf-8?B?NUszMnBWKys3TndYWGJQSkE5eXBWaDA1QzZkclMvSkd1cDlISnhaRTc4VW5l?= =?utf-8?B?QVR4c1ZES1p2MXNLelY1VlVtN2YraGFybFlWVW9rWHY1elFpMmkxVWhaanVX?= =?utf-8?B?ek9PNTloVUZiQkpKV294T0c4bGR1SHFtK01ObzFMK05ORFRJT1RJeUx3WFBL?= =?utf-8?B?ZENTQUFyOFU0MUJib0pCVEIydzNTRy9hUEtHd2RSOE1qZllEZHJlZ29iMTBn?= =?utf-8?B?Si9YZXpjTmkwclBxOFNEVXM1a0RjR0dSbHhpaUhxTzd4WFdXWk5WYmFDL3Nj?= =?utf-8?B?MENET3prNVFSb09tejJ4dSs0Z1ZRZ3g2SVlMTmhCa0FLeW01REFOSkwvUXdj?= =?utf-8?B?WnBjRk1kUVhxT0Q0M2libm1aaVhmVVVvUE1hT2lCNnN2S3grdW93R25odFVU?= =?utf-8?B?YjBrK0hIRTJRR20ycVJKbGt4aWx3WXk1dzZpckl3a29vdUVUOVNTMWJxVTdH?= =?utf-8?B?S3NPZG1SMUVWcE9ObmxuSzhwMm4xMVVWNys0OW9KczZ1TEpmbGg4RlI1Qk9I?= =?utf-8?B?NnJVaGVqaDdPQjJ5L0svTU5HckZjeWR5UXZ1SG4ydk1aMUwrK2lNSGRZVnk2?= =?utf-8?B?Nlh3TmVOU0J2NlNUcnVBZTdBaFA3Z094WVZMMVMxMktQdkN6YjhnVGNpRVN1?= =?utf-8?B?cEtrVEgzTmEwM21vbG4rUE5Xd3dxTFBsZXhPNzE4ZUJHYUxiUXRYT3FUbDVp?= =?utf-8?B?N0wxQXdsbFlTc1k5UDJBYnhwMis0QWpaeVczTUFXdEdGUWJZbU9BN3JUU3VT?= =?utf-8?B?Ky9IWSs5ZmtTaGFPelkzL3JTSW8rOGtGMkcvQUVjZzFRdm1kK1BDZjd4OHp1?= =?utf-8?B?bURFWk1CMjJTZmllT2ZYMGRsMERUekkyUjdxc1d6OUdzRmxuRllaVmxnUjlP?= =?utf-8?B?YmJpL2Y5dTRiUzR4MUJlNnd4Z1g0dVhRUU1JVFlTWjRIMW1QdkVXMm9rZ2o0?= =?utf-8?B?YzRuMDdXYkNjenJ1RTVIYkpzK2Z4SEpHdjY4b2NTTGNjdHVxNEZTVHhlTTI3?= =?utf-8?B?SFY0VU1VZ1RXTDdJdHRFY2QzaVdQcENnUVNxRTkyRlFSU0Q5Z0tsZGxMb1VP?= =?utf-8?B?NDVFT2x4YlJ5Sm5QMFNGUEpIUXRiRXBMSWhNL2hzc3haM29pS0VNeFFJN1cr?= =?utf-8?Q?RA81GRAAp46lEhyN1XlKvxch52rMQSuVfsVhhrE?= X-MS-Exchange-CrossTenant-Network-Message-Id: 6b1ace0f-e868-4886-926e-08d921ff97ed X-MS-Exchange-CrossTenant-AuthSource: DS7PR03MB5608.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 May 2021 17:39:50.0087 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 335836de-42ef-43a2-b145-348c2ee9ca5b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: hlClrQDZnDfCPAW21vi6mC4zstsM+uDfYwiz/SAd7wb7PZ9m0dV6PY60owNRyJKcP088x6JaKo96YfiwipJW9g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR03MB5966 X-OriginatorOrg: citrix.com X-ZohoMail-DKIM: pass (identity @citrix.com) Fixes: 1c84d04673 ('VMX: remove the problematic set_uc_mode logic') Signed-off-by: Roger Pau Monn=C3=A9 Acked-by: Jan Beulich --- xen/include/asm-x86/mtrr.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/xen/include/asm-x86/mtrr.h b/xen/include/asm-x86/mtrr.h index 4be704cb6a..24e5de5c22 100644 --- a/xen/include/asm-x86/mtrr.h +++ b/xen/include/asm-x86/mtrr.h @@ -78,8 +78,6 @@ extern u32 get_pat_flags(struct vcpu *v, u32 gl1e_flags, = paddr_t gpaddr, extern int epte_get_entry_emt(struct domain *, unsigned long gfn, mfn_t mf= n, unsigned int order, uint8_t *ipat, bool_t direct_mmio); -extern void ept_change_entry_emt_with_range( - struct domain *d, unsigned long start_gfn, unsigned long end_gfn); extern unsigned char pat_type_2_pte_flags(unsigned char pat_type); extern int hold_mtrr_updates_on_aps; extern void mtrr_aps_sync_begin(void); --=20 2.31.1 From nobody Fri May 3 23:02:13 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1dmarc=pass fromdomain=citrix.com); dmarc=pass(p=reject dis=none) header.from=citrix.com ARC-Seal: i=2; a=rsa-sha256; t=1622223615; cv=pass; d=zohomail.com; s=zohoarc; b=MXxmQb1E35CvuSnnyjMgOabXuY5ewNingbzlL0IlDuwhreOY7Sk+kR5dMbtNtCdKcdydI5qjf2kwyjV7BRVV9DWEf3loM91+Z3XvQ9ZipZbKMtVzeMhAoqWSvIYCaMouxK/EES99WEQPSpWszcEKYUFhzED/2llnwdqRMtNRGxI= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1622223615; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=GJvcxRwg22ywwmHcBxf934ac3uryD0pmSSRdtpvFX8U=; b=DAlIuOdIoXR8KJqlPM1mWrlF0lc5UJneQOvJEUYi1r+12laH89Pr+wxTqmtFZ5PIDc4be4hlM6+AbvneHAAzJkn1f1FKUl6SL8m7Z5OFlR3FT75bzpsHDefmRvaovQw/Rl3njoe9ob75aO0GBD7jCCR+TK+nd5CyIWI5cpkZD7E= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1dmarc=pass fromdomain=citrix.com); dmarc=pass header.from= (p=reject dis=none) header.from= Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1622223615373393.4715102290345; Fri, 28 May 2021 10:40:15 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.134042.249621 (Exim 4.92) (envelope-from ) id 1lmgSp-0001Oq-8M; Fri, 28 May 2021 17:40:03 +0000 Received: by outflank-mailman (output) from mailman id 134042.249621; Fri, 28 May 2021 17:40:03 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lmgSp-0001OS-4X; Fri, 28 May 2021 17:40:03 +0000 Received: by outflank-mailman (input) for mailman id 134042; Fri, 28 May 2021 17:40:01 +0000 Received: from all-amaz-eas1.inumbo.com ([34.197.232.57] helo=us1-amaz-eas2.inumbo.com) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lmgSn-0000ZU-Sk for xen-devel@lists.xenproject.org; Fri, 28 May 2021 17:40:01 +0000 Received: from esa5.hc3370-68.iphmx.com (unknown [216.71.155.168]) by us1-amaz-eas2.inumbo.com (Halon) with ESMTPS id b1022678-1dab-4265-9269-6e686d82b118; Fri, 28 May 2021 17:39:58 +0000 (UTC) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: b1022678-1dab-4265-9269-6e686d82b118 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=citrix.com; s=securemail; t=1622223598; h=from:to:cc:subject:date:message-id:in-reply-to: references:content-transfer-encoding:mime-version; bh=bvVMuM/HMP8ABLm9/U9j8KYIq8D0Cdsi9pnwcgfazSA=; b=LyTRiMgARwxMWJf0iOd0g6fgO7fNYbrWhEyQP2s+C4Lsm5vv1lY/MbUj U+Y18oCC0BjbkvU6LMB4Jm2FdiEeJJYFc0wGbh6ZarJf/fXB4mfbe8c7Z pmyc/qetIPim+qZRxmRkoIQbt5uvoyfEOvCiuVOMutw0XkAAa6dRsJKQN w=; Authentication-Results: esa5.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com IronPort-SDR: rBocOdJhVqCmH6Cs3RDUktMijpi94EDlr3TPpoipgHSXRpfQt8xXzV4nr2u4VPFAwQYamS0j3z WGqlE0yh3MUwfLWCFLZug/A/90JJ5NU6D/81PbvM45GbvK82Cjh3BtOtUnBMPl9WtNVOxrHgG7 nIAMR9+qnfYqY5VWIDhqUCvBJFxFlIMAItwRphJ1r2QgCCgKu0hRGILEo1a+EwTN2l7HY0NhVt 7p+gM7HXOEJy75EklgygIqAJnYUFaW/ib7RLgUV5fTrRYPbWqVNCSaKYY0q7RjAnZzUlG+Jtvq p24= X-SBRS: 5.1 X-MesageID: 44631976 X-Ironport-Server: esa5.hc3370-68.iphmx.com X-Remote-IP: 162.221.158.21 X-Policy: $RELAYED IronPort-HdrOrdr: A9a23:C2n/V6B4SRnrqn7lHeiTsceALOsnbusQ8zAXPh9KJyC9I/b2qy nxppgmPH/P6Ar4WBkb6Le90c67MAzhHP9OkPUs1NKZPTUO11HYVb2KgbGSpgEIXheOjNK1tp 0QAJSWaueAdWSS5PySiGLTfqdCsbv3hZxAx92uskuFJTsaG52IhD0JbDpzfHcGIDWvUvECZe uhD4d81nSdkTN9VLXzOpFrNNKz5uHjpdbDW1orFhQn4A6BgXeD87jhCSWV2R8YTndm3aoi2X KtqX262oyT99WAjjPM3W7a6Jpb3PH7zMFYOcCKgs8Jbh3xlweTYph7UbHqhkF3nAiW0idvrD DwmWZmAywqgEmhOF1d4CGdmzUI6QxerkMLkjSj8CLeSaWTfkNJNyJD7bgpPycxpXBQ5O2Vfc pwriqkXqFsfGT9dRLGlpL1viFR5z6JSEUZ4JguZgRkIPAjgZ9q3MAiFRBuYdg99ByT0vFtLA A4NrCj2B8RSyLAU0zk X-IronPort-AV: E=Sophos;i="5.83,230,1616472000"; d="scan'208";a="44631976" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=VL4xxXwrS9wor3KCFhWxFqH85w/w/lqVzjp0ykSTdTuosQ2UxDusrYYy5o5ajI0mzb5xOF2T+z5g70TuAslv2bB6jxe5JcWc7q2ysf9f1BaSLXmE39SlNh+YP7nJh7xpC7hqAAdrrZsxQJql5lh8bpJ8MEvu2CT+cCWGOMkar8v7lxJZXxQTPnR9Jy35gnQGsSZCOIF/m6dx7ivSFZYCwfolPm7Qp/uTcRu5WA20MHwYo1Y9cNmuSUqX5Zsev9J5rP+F3bjwLbuhhqXoo5jVpc9wfkjnlL8Slif+rgFVnU/ERurtAA9/jBl7CMeZlQCeuCQjUniO4SbL/e2z+07liA== 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-SenderADCheck; bh=GJvcxRwg22ywwmHcBxf934ac3uryD0pmSSRdtpvFX8U=; b=PVu8KguDHiNNM7R3JAZFYL3XKaxTMwDG+R6RQljmh/PG0YB5gNFOIFrdkHRnrK9ZUx9rxkFckBdoV2Q6B0ydPbUoDc/DRq3cCds8TF2UjH5gOhU3VLJKTn9wN1+3lFchdr/RmIsEahld4I79lw3pEy9JWOcpKP89FXrpqjoG04jBOGd5FyEMcBKdt3aTh0QP6mNwwIURyT92Az+2KwWkH2p1lM8evfgIf8DeFFD8G6kuiW3k8viy7lnJ9BCKvO5p6t1aUiQw0Vb/XIQf5WwHh0hrYZnPzRalWH01zWVSBHgaLF51SjPRY8tYEommPD65CVQIEk7EayHtDNTvEYOXvQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=citrix.onmicrosoft.com; s=selector2-citrix-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=GJvcxRwg22ywwmHcBxf934ac3uryD0pmSSRdtpvFX8U=; b=f5WRMDZSbDPh7hqA2tz1KmBVpBEO3Kf/o/d9srbj8eeX3loE7hg53UwXrkxGr0xAhu+4d8+kawQRrSXiNWFazcelrE5vcJ7tYEa2IYkf9XO9cIq6reQ97rCqFXT2cmH+ViGyoAanFDbjj+5swEmF23kedbRWZOP506L6jreizxk= From: Roger Pau Monne To: CC: Roger Pau Monne , Jan Beulich , Andrew Cooper , Wei Liu , Jun Nakajima , Kevin Tian , George Dunlap Subject: [PATCH 2/3] x86/mtrr: move epte_get_entry_emt to p2m-ept.c Date: Fri, 28 May 2021 19:39:34 +0200 Message-ID: <20210528173935.29919-3-roger.pau@citrix.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210528173935.29919-1-roger.pau@citrix.com> References: <20210528173935.29919-1-roger.pau@citrix.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: MR2P264CA0129.FRAP264.PROD.OUTLOOK.COM (2603:10a6:500:30::21) To DS7PR03MB5608.namprd03.prod.outlook.com (2603:10b6:5:2c9::18) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 3c37482d-435b-4952-7e5d-08d921ff9acb X-MS-TrafficTypeDiagnostic: DM4PR03MB5966: X-LD-Processed: 335836de-42ef-43a2-b145-348c2ee9ca5b,ExtAddr X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:5797; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: h3hnrmg3OiMYZGcJHHwtV98Ty13QItmCXyTTQku2UDTDqmsN+owKX7p8flQvmxoJ8xs3r4TTyFTDz/9BHACM6+7FI3iNsTxZqcDjwX6t6/dLBzFcBOm16XGb4ZC0xxRsVb1B0xzBrtfMe9XXxaz1b0AzNkTbGheMo3eyj7GQYoSCbX3F8HIRLK/s/oUEj4ilC0eX6sgF1RM00bdXHJBFivRAohWv6ywY2JGqUVk/qJR2JPBveIy4E/BoZmwjFKkXUewDNGzJbVbaEP45xwIXl1sXEBJCcfbRHkrCrGoxppfRgRkYBRwCt41LG+lZIF1JE0Kkf/v3OmjYlvCeTUu005WZlx7MC+IWkqDjds1ELir39znwkB1CzoKlL3b7tWtYfO3XYUHL+swJDwqoG4byQtaeMT8yCx6tF3mpkrC8mE2dy8nz9s80C7+Yy5KlfclIRCAeqBqevs6rUnejKZw6A/Raomf05OhWEfNg6BH9iHl1W8GOcYTymZhNDV6QzxaZ/WcPMSS3Yt2pPZb05voaCeemNHkPmSxnPsgrJffbW5iROx/yPF8QAgxlBhaa/7BxqfQOWzmNn5BZMkIHb4qo1boKejh5VE0rlM6vgN0SCs8= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DS7PR03MB5608.namprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(396003)(39860400002)(366004)(376002)(136003)(346002)(6666004)(107886003)(36756003)(2906002)(26005)(2616005)(66476007)(316002)(66556008)(186003)(66946007)(16526019)(54906003)(956004)(4326008)(38100700002)(6486002)(30864003)(8676002)(83380400001)(478600001)(8936002)(6496006)(86362001)(6916009)(1076003)(5660300002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?VyttYlNnQzBIZ0hYSTZmVk1BWjNJUURKNGF0TzlQNGxzQ3pRY2VtTG5hTVhH?= =?utf-8?B?M1pkZHBkdjg1cjdtRkdVZmJuaFhVTXllaVAwUkduMXdoK2VQLzBVN000QVRE?= =?utf-8?B?ME9xQXFJemFlVEdyUjQrTUJqa3NRWTlpVU1MUXdsM1lCekwxZ0E5a0U0VVBo?= =?utf-8?B?bDc1aVl6V0haM0E0QnpzaUNITWIwemhoTnlwOXh2SXFPZ2ljUUpkV2UvT0Y4?= =?utf-8?B?ME5Da2Y2N3FOQlFIeDJlMUQ3ckRXT05vWktJcEphUUl3SitwYWtuYTFmc2Q4?= =?utf-8?B?QjhnU2x5NVJmOTBSMjdqVE42OXl1czRrK3VEeWY1aEZuTDQ2M0pBR0tkSFJn?= =?utf-8?B?T2dJRDJqd0ZMUDB0Y2hHUnE1Qm45SjRITlZNNTdpUnhyYXpaUmxNSEtGamlR?= =?utf-8?B?T1Y2Sm1MZExmWDVtVGdUYXBGM3lndzBPNTMxQklSU3JpVjJxNklkQU92WDJH?= =?utf-8?B?YXZtQUVOSElLaW5VY09aS2pIQndNQ3FSeHVLcm9Fb2hhMnFIRXBNeDFVT3hj?= =?utf-8?B?bllqR2pNdjRRekpZazB0bU9kYW5HcEFZeTJLOGt4eXphK3puN0JkTFRZZ1l4?= =?utf-8?B?d2UwSkhBN1FhSG5ZMzVGL0dmT01NTnFSN0R2MWpYY1BXbmhkUk91Y2tuM3VW?= =?utf-8?B?Qi9EbkZQZW84Q2hSczVmS1NoYmFCNGwvQUVOb1NIMmhycTdXWlRROXNJNFdx?= =?utf-8?B?ZFUvUEJDRWdPd1k4ZDcrMHQrOG5mcDR2L1BJcHJzSWN2MnVhakk1MC80SjRH?= =?utf-8?B?VWhNUzBHUDkzZDAvNU5SZDZ3Y3ZncGtjSktnOVB2Q0xmdTdzL1BLNTVIV0th?= =?utf-8?B?alZYOWw3bWlHS0M5OUU4Q3hSSHlWckFOclFsTndrYWFFS1A0aEU3NGN5YUZh?= =?utf-8?B?VnJHRGJkcjJGV09WNUVadlUyOUE4dktvdTRhMTc5ZWJoM1dyM2JHNGJFWncw?= =?utf-8?B?QnUwMThXQXRkTTZYUlZCWGhmZ0ZFMUhrM29TMVp4NVpWdUQxSDBkNHp2V0h5?= =?utf-8?B?STM1ZFNqSTc0NXJjUzdCUG1Sdmp6WjlYbEZydHdSMWNLSVV0Yk1wZlY4ZmNL?= =?utf-8?B?YVpQWXJmM2xFajc3ZStlK1NGaHJISXJLZk5qYjRYbXBBaVBkQmVxUEZhV2Z1?= =?utf-8?B?L1ZvdGpPeUNpeWxFRmxGbGQvNG9iN1VUSEFrL2I1N01VeXNGTW9HbGJRVmpT?= =?utf-8?B?TUIxUnFGZDdMUi9pWVhWSEVCZys1QW9IL1FlWTE0TEk2OGkyTGhRMkNyWmxB?= =?utf-8?B?R3R1QlZQK3lxQVJCcEJkUG56dENnL1JoL2NaZnhZamlQbVEvbThOd3BmOTY2?= =?utf-8?B?MGpBQ0g1Z3BTSHNsWDJDcTYzcy9idWVTeVpaZWhPNnpndjMzaGZibjZmRkdW?= =?utf-8?B?ZTZSenJTS290YzR1bzU2YjRuaDlOazAzbVdDTjlnYVB1N3c2Sm05WWM1cUxz?= =?utf-8?B?blIzNHBOM2FyQW8xM0JsYTY5ZWs0dEt5UDJ2UUV4emptblRTMHFkUG9FNTk2?= =?utf-8?B?UXMvZmo3UkY1Vmo2ZG12Uk1tZGpXa3NKQWlVOVQ4djBiU0hyTW4vYUxmYmh2?= =?utf-8?B?cWdRWS96OE0ya3Y3YXEyelhXOGlNRW5uSVYrbnJ2MG0ycVRQQmNZYXN6UkU1?= =?utf-8?B?UEV2ajBacENHNGY1bG1Pd1g2UWtVeFNMV2JCbHY5Sm9SSWVhNmtsNTZKS0tp?= =?utf-8?B?U1lteHprcHQ4c0NzWDdSRlhSYmlCZEM1VlU5blZrb1lrOGw3YVVYMkVHTEox?= =?utf-8?Q?eFa9dZX5alb8nhd9tJPuUSoeWWLWMOStaBk/ZQP?= X-MS-Exchange-CrossTenant-Network-Message-Id: 3c37482d-435b-4952-7e5d-08d921ff9acb X-MS-Exchange-CrossTenant-AuthSource: DS7PR03MB5608.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 May 2021 17:39:54.8444 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 335836de-42ef-43a2-b145-348c2ee9ca5b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: iiGorM+JTj0lcezfzXSFfpp9FQvYBQOtEOMU32kLtWGxw0Z/FqNT1CP99ZxVDAWJN6Pcn/q5LqhdAnwEwc+IgQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR03MB5966 X-OriginatorOrg: citrix.com X-ZohoMail-DKIM: pass (identity @citrix.com) This is an EPT specific function, so it shouldn't live in the generic mtrr file. Such movement is also needed for future work that will require passing a p2m_type_t parameter to epte_get_entry_emt, and making that type visible to the mtrr users is cumbersome and unneeded. Moving epte_get_entry_emt out of mtrr.c requires making the helper to get the MTRR type of an address from the mtrr state public. While there rename the function to start with the mtrr prefix, like other mtrr related functions. While there fix some of the types of the function parameters. No functional change intended. Signed-off-by: Roger Pau Monn=C3=A9 Reviewed-by: Jan Beulich Reviewed-by: Kevin Tian --- xen/arch/x86/hvm/mtrr.c | 107 +--------------------------- xen/arch/x86/hvm/vmx/vmx.c | 4 +- xen/arch/x86/mm/p2m-ept.c | 114 ++++++++++++++++++++++++++++-- xen/include/asm-x86/hvm/vmx/vmx.h | 2 + xen/include/asm-x86/mtrr.h | 5 +- 5 files changed, 118 insertions(+), 114 deletions(-) diff --git a/xen/arch/x86/hvm/mtrr.c b/xen/arch/x86/hvm/mtrr.c index 82ded1635c..4a9f3177ed 100644 --- a/xen/arch/x86/hvm/mtrr.c +++ b/xen/arch/x86/hvm/mtrr.c @@ -194,8 +194,7 @@ void hvm_vcpu_cacheattr_destroy(struct vcpu *v) * May return a negative value when order > 0, indicating to the caller * that the respective mapping needs splitting. */ -static int get_mtrr_type(const struct mtrr_state *m, - paddr_t pa, unsigned int order) +int mtrr_get_type(const struct mtrr_state *m, paddr_t pa, unsigned int ord= er) { uint8_t overlap_mtrr =3D 0; uint8_t overlap_mtrr_pos =3D 0; @@ -323,7 +322,7 @@ static uint8_t effective_mm_type(struct mtrr_state *m, * just use it */=20 if ( gmtrr_mtype =3D=3D NO_HARDCODE_MEM_TYPE ) - mtrr_mtype =3D get_mtrr_type(m, gpa, 0); + mtrr_mtype =3D mtrr_get_type(m, gpa, 0); else mtrr_mtype =3D gmtrr_mtype; =20 @@ -350,7 +349,7 @@ uint32_t get_pat_flags(struct vcpu *v, guest_eff_mm_type =3D effective_mm_type(g, pat, gpaddr,=20 gl1e_flags, gmtrr_mtype); /* 2. Get the memory type of host physical address, with MTRR */ - shadow_mtrr_type =3D get_mtrr_type(&mtrr_state, spaddr, 0); + shadow_mtrr_type =3D mtrr_get_type(&mtrr_state, spaddr, 0); =20 /* 3. Find the memory type in PAT, with host MTRR memory type * and guest effective memory type. @@ -789,106 +788,6 @@ void memory_type_changed(struct domain *d) } } =20 -int epte_get_entry_emt(struct domain *d, unsigned long gfn, mfn_t mfn, - unsigned int order, uint8_t *ipat, bool_t direct_mm= io) -{ - int gmtrr_mtype, hmtrr_mtype; - struct vcpu *v =3D current; - unsigned long i; - - *ipat =3D 0; - - if ( v->domain !=3D d ) - v =3D d->vcpu ? d->vcpu[0] : NULL; - - /* Mask, not add, for order so it works with INVALID_MFN on unmapping = */ - if ( rangeset_overlaps_range(mmio_ro_ranges, mfn_x(mfn), - mfn_x(mfn) | ((1UL << order) - 1)) ) - { - if ( !order || rangeset_contains_range(mmio_ro_ranges, mfn_x(mfn), - mfn_x(mfn) | ((1UL << order= ) - 1)) ) - { - *ipat =3D 1; - return MTRR_TYPE_UNCACHABLE; - } - /* Force invalid memory type so resolve_misconfig() will split it = */ - return -1; - } - - if ( !mfn_valid(mfn) ) - { - *ipat =3D 1; - return MTRR_TYPE_UNCACHABLE; - } - - if ( !direct_mmio && !is_iommu_enabled(d) && !cache_flush_permitted(d)= ) - { - *ipat =3D 1; - return MTRR_TYPE_WRBACK; - } - - for ( i =3D 0; i < (1ul << order); i++ ) - { - if ( is_special_page(mfn_to_page(mfn_add(mfn, i))) ) - { - if ( order ) - return -1; - *ipat =3D 1; - return MTRR_TYPE_WRBACK; - } - } - - if ( direct_mmio ) - return MTRR_TYPE_UNCACHABLE; - - gmtrr_mtype =3D hvm_get_mem_pinned_cacheattr(d, _gfn(gfn), order); - if ( gmtrr_mtype >=3D 0 ) - { - *ipat =3D 1; - return gmtrr_mtype !=3D PAT_TYPE_UC_MINUS ? gmtrr_mtype - : MTRR_TYPE_UNCACHABLE; - } - if ( gmtrr_mtype =3D=3D -EADDRNOTAVAIL ) - return -1; - - gmtrr_mtype =3D v ? get_mtrr_type(&v->arch.hvm.mtrr, gfn << PAGE_SHIFT= , order) - : MTRR_TYPE_WRBACK; - hmtrr_mtype =3D get_mtrr_type(&mtrr_state, mfn_x(mfn) << PAGE_SHIFT, o= rder); - if ( gmtrr_mtype < 0 || hmtrr_mtype < 0 ) - return -1; - - /* If both types match we're fine. */ - if ( likely(gmtrr_mtype =3D=3D hmtrr_mtype) ) - return hmtrr_mtype; - - /* If either type is UC, we have to go with that one. */ - if ( gmtrr_mtype =3D=3D MTRR_TYPE_UNCACHABLE || - hmtrr_mtype =3D=3D MTRR_TYPE_UNCACHABLE ) - return MTRR_TYPE_UNCACHABLE; - - /* If either type is WB, we have to go with the other one. */ - if ( gmtrr_mtype =3D=3D MTRR_TYPE_WRBACK ) - return hmtrr_mtype; - if ( hmtrr_mtype =3D=3D MTRR_TYPE_WRBACK ) - return gmtrr_mtype; - - /* - * At this point we have disagreeing WC, WT, or WP types. The only - * combination that can be cleanly resolved is WT:WP. The ones involvi= ng - * WC need to be converted to UC, both due to the memory ordering - * differences and because WC disallows reads to be cached (WT and WP - * permit this), while WT and WP require writes to go straight to memo= ry - * (WC can buffer them). - */ - if ( (gmtrr_mtype =3D=3D MTRR_TYPE_WRTHROUGH && - hmtrr_mtype =3D=3D MTRR_TYPE_WRPROT) || - (gmtrr_mtype =3D=3D MTRR_TYPE_WRPROT && - hmtrr_mtype =3D=3D MTRR_TYPE_WRTHROUGH) ) - return MTRR_TYPE_WRPROT; - - return MTRR_TYPE_UNCACHABLE; -} - /* * Local variables: * mode: C diff --git a/xen/arch/x86/hvm/vmx/vmx.c b/xen/arch/x86/hvm/vmx/vmx.c index 7e3e67fdc3..0d4b47681b 100644 --- a/xen/arch/x86/hvm/vmx/vmx.c +++ b/xen/arch/x86/hvm/vmx/vmx.c @@ -417,12 +417,12 @@ static int vmx_domain_initialise(struct domain *d) static void domain_creation_finished(struct domain *d) { gfn_t gfn =3D gaddr_to_gfn(APIC_DEFAULT_PHYS_BASE); - uint8_t ipat; + bool ipat; =20 if ( !has_vlapic(d) || mfn_eq(apic_access_mfn, INVALID_MFN) ) return; =20 - ASSERT(epte_get_entry_emt(d, gfn_x(gfn), apic_access_mfn, 0, &ipat, + ASSERT(epte_get_entry_emt(d, gfn, apic_access_mfn, 0, &ipat, true) =3D=3D MTRR_TYPE_WRBACK); ASSERT(ipat); =20 diff --git a/xen/arch/x86/mm/p2m-ept.c b/xen/arch/x86/mm/p2m-ept.c index a3beaf91e2..f1d1d07e92 100644 --- a/xen/arch/x86/mm/p2m-ept.c +++ b/xen/arch/x86/mm/p2m-ept.c @@ -20,6 +20,7 @@ #include #include #include +#include #include #include #include @@ -485,6 +486,108 @@ static int ept_invalidate_emt_range(struct p2m_domain= *p2m, return rc; } =20 +int epte_get_entry_emt(struct domain *d, gfn_t gfn, mfn_t mfn, + unsigned int order, bool *ipat, bool direct_mmio) +{ + int gmtrr_mtype, hmtrr_mtype; + struct vcpu *v =3D current; + unsigned long i; + + *ipat =3D false; + + if ( v->domain !=3D d ) + v =3D d->vcpu ? d->vcpu[0] : NULL; + + /* Mask, not add, for order so it works with INVALID_MFN on unmapping = */ + if ( rangeset_overlaps_range(mmio_ro_ranges, mfn_x(mfn), + mfn_x(mfn) | ((1UL << order) - 1)) ) + { + if ( !order || rangeset_contains_range(mmio_ro_ranges, mfn_x(mfn), + mfn_x(mfn) | ((1UL << order= ) - 1)) ) + { + *ipat =3D true; + return MTRR_TYPE_UNCACHABLE; + } + /* Force invalid memory type so resolve_misconfig() will split it = */ + return -1; + } + + if ( !mfn_valid(mfn) ) + { + *ipat =3D true; + return MTRR_TYPE_UNCACHABLE; + } + + if ( !direct_mmio && !is_iommu_enabled(d) && !cache_flush_permitted(d)= ) + { + *ipat =3D true; + return MTRR_TYPE_WRBACK; + } + + for ( i =3D 0; i < (1ul << order); i++ ) + { + if ( is_special_page(mfn_to_page(mfn_add(mfn, i))) ) + { + if ( order ) + return -1; + *ipat =3D true; + return MTRR_TYPE_WRBACK; + } + } + + if ( direct_mmio ) + return MTRR_TYPE_UNCACHABLE; + + gmtrr_mtype =3D hvm_get_mem_pinned_cacheattr(d, gfn, order); + if ( gmtrr_mtype >=3D 0 ) + { + *ipat =3D true; + return gmtrr_mtype !=3D PAT_TYPE_UC_MINUS ? gmtrr_mtype + : MTRR_TYPE_UNCACHABLE; + } + if ( gmtrr_mtype =3D=3D -EADDRNOTAVAIL ) + return -1; + + gmtrr_mtype =3D v ? mtrr_get_type(&v->arch.hvm.mtrr, + gfn_x(gfn) << PAGE_SHIFT, order) + : MTRR_TYPE_WRBACK; + hmtrr_mtype =3D mtrr_get_type(&mtrr_state, mfn_x(mfn) << PAGE_SHIFT, + order); + if ( gmtrr_mtype < 0 || hmtrr_mtype < 0 ) + return -1; + + /* If both types match we're fine. */ + if ( likely(gmtrr_mtype =3D=3D hmtrr_mtype) ) + return hmtrr_mtype; + + /* If either type is UC, we have to go with that one. */ + if ( gmtrr_mtype =3D=3D MTRR_TYPE_UNCACHABLE || + hmtrr_mtype =3D=3D MTRR_TYPE_UNCACHABLE ) + return MTRR_TYPE_UNCACHABLE; + + /* If either type is WB, we have to go with the other one. */ + if ( gmtrr_mtype =3D=3D MTRR_TYPE_WRBACK ) + return hmtrr_mtype; + if ( hmtrr_mtype =3D=3D MTRR_TYPE_WRBACK ) + return gmtrr_mtype; + + /* + * At this point we have disagreeing WC, WT, or WP types. The only + * combination that can be cleanly resolved is WT:WP. The ones involvi= ng + * WC need to be converted to UC, both due to the memory ordering + * differences and because WC disallows reads to be cached (WT and WP + * permit this), while WT and WP require writes to go straight to memo= ry + * (WC can buffer them). + */ + if ( (gmtrr_mtype =3D=3D MTRR_TYPE_WRTHROUGH && + hmtrr_mtype =3D=3D MTRR_TYPE_WRPROT) || + (gmtrr_mtype =3D=3D MTRR_TYPE_WRPROT && + hmtrr_mtype =3D=3D MTRR_TYPE_WRTHROUGH) ) + return MTRR_TYPE_WRPROT; + + return MTRR_TYPE_UNCACHABLE; +} + /* * Resolve deliberately mis-configured (EMT field set to an invalid value) * entries in the page table hierarchy for the given GFN: @@ -519,7 +622,7 @@ static int resolve_misconfig(struct p2m_domain *p2m, un= signed long gfn) =20 if ( level =3D=3D 0 || is_epte_superpage(&e) ) { - uint8_t ipat =3D 0; + bool ipat; =20 if ( e.emt !=3D MTRR_NUM_TYPES ) break; @@ -535,7 +638,7 @@ static int resolve_misconfig(struct p2m_domain *p2m, un= signed long gfn) e.emt =3D 0; if ( !is_epte_valid(&e) || !is_epte_present(&e) ) continue; - e.emt =3D epte_get_entry_emt(p2m->domain, gfn + i, + e.emt =3D epte_get_entry_emt(p2m->domain, _gfn(gfn + i= ), _mfn(e.mfn), 0, &ipat, e.sa_p2mt =3D=3D p2m_mmio_d= irect); e.ipat =3D ipat; @@ -553,7 +656,8 @@ static int resolve_misconfig(struct p2m_domain *p2m, un= signed long gfn) } else { - int emt =3D epte_get_entry_emt(p2m->domain, gfn, _mfn(e.mf= n), + int emt =3D epte_get_entry_emt(p2m->domain, _gfn(gfn), + _mfn(e.mfn), level * EPT_TABLE_ORDER, &ipa= t, e.sa_p2mt =3D=3D p2m_mmio_dir= ect); bool_t recalc =3D e.recalc; @@ -788,8 +892,8 @@ ept_set_entry(struct p2m_domain *p2m, gfn_t gfn_, mfn_t= mfn, =20 if ( mfn_valid(mfn) || p2m_allows_invalid_mfn(p2mt) ) { - uint8_t ipat =3D 0; - int emt =3D epte_get_entry_emt(p2m->domain, gfn, mfn, + bool ipat; + int emt =3D epte_get_entry_emt(p2m->domain, _gfn(gfn), mfn, i * EPT_TABLE_ORDER, &ipat, p2mt =3D=3D p2m_mmio_direct); =20 diff --git a/xen/include/asm-x86/hvm/vmx/vmx.h b/xen/include/asm-x86/hvm/vm= x/vmx.h index 534e9fc221..f668ee1f09 100644 --- a/xen/include/asm-x86/hvm/vmx/vmx.h +++ b/xen/include/asm-x86/hvm/vmx/vmx.h @@ -599,6 +599,8 @@ void ept_p2m_uninit(struct p2m_domain *p2m); =20 void ept_walk_table(struct domain *d, unsigned long gfn); bool_t ept_handle_misconfig(uint64_t gpa); +int epte_get_entry_emt(struct domain *d, gfn_t gfn, mfn_t mfn, + unsigned int order, bool *ipat, bool direct_mmio); void setup_ept_dump(void); void p2m_init_altp2m_ept(struct domain *d, unsigned int i); /* Locate an alternate p2m by its EPTP */ diff --git a/xen/include/asm-x86/mtrr.h b/xen/include/asm-x86/mtrr.h index 24e5de5c22..e0fd1005ce 100644 --- a/xen/include/asm-x86/mtrr.h +++ b/xen/include/asm-x86/mtrr.h @@ -72,12 +72,11 @@ extern int mtrr_add_page(unsigned long base, unsigned l= ong size, unsigned int type, char increment); extern int mtrr_del(int reg, unsigned long base, unsigned long size); extern int mtrr_del_page(int reg, unsigned long base, unsigned long size); +extern int mtrr_get_type(const struct mtrr_state *m, paddr_t pa, + unsigned int order); extern void mtrr_centaur_report_mcr(int mcr, u32 lo, u32 hi); extern u32 get_pat_flags(struct vcpu *v, u32 gl1e_flags, paddr_t gpaddr, paddr_t spaddr, uint8_t gmtrr_mtype); -extern int epte_get_entry_emt(struct domain *, unsigned long gfn, mfn_t mf= n, - unsigned int order, uint8_t *ipat, - bool_t direct_mmio); extern unsigned char pat_type_2_pte_flags(unsigned char pat_type); extern int hold_mtrr_updates_on_aps; extern void mtrr_aps_sync_begin(void); --=20 2.31.1 From nobody Fri May 3 23:02:13 2024 Delivered-To: importer@patchew.org Received-SPF: pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) client-ip=192.237.175.120; envelope-from=xen-devel-bounces@lists.xenproject.org; helo=lists.xenproject.org; Authentication-Results: mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1dmarc=pass fromdomain=citrix.com); dmarc=pass(p=reject dis=none) header.from=citrix.com ARC-Seal: i=2; a=rsa-sha256; t=1622223618; cv=pass; d=zohomail.com; s=zohoarc; b=fFN3juE4Ya5tJ9kWnnYEfehGGabq+pLmwWf1H5iYlWGgT1y2IKasH4uAHZX697uJ9z96SOHut9a1ZxNip4EacZgmHBHt7m9p6FfeWHbltnjxZRj25LuL8y+64coCcsD/ILhjGd7e9TInuziFG8iQtFq3OAgju4Y1I1AitvH5syU= ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=zohomail.com; s=zohoarc; t=1622223618; h=Content-Type:Content-Transfer-Encoding:Cc:Date:From:In-Reply-To:List-Subscribe:List-Post:List-Id:List-Help:List-Unsubscribe:MIME-Version:Message-ID:References:Sender:Subject:To; bh=AtL+Hp/29nuKWkROvVXgdUUpYBKLh+7LK1UvsLfBAnw=; b=N8nOXxLilzKwY+6yQV01druzpj6sSWKfZF6j7N6NeqHKIyiCq/ALBh1M5EeNES0yrK6fQZztigSY+QgDPpJcKZzkrpkv1lHGccpSglEITcKmW3BUbREZr97LxKIlg+eHu6y8hz8e3T+DrkwYSXhZgZ4VMLyFn+jz/J/nsQbsK9M= ARC-Authentication-Results: i=2; mx.zohomail.com; dkim=pass; spf=pass (zohomail.com: domain of lists.xenproject.org designates 192.237.175.120 as permitted sender) smtp.mailfrom=xen-devel-bounces@lists.xenproject.org; arc=pass (i=1dmarc=pass fromdomain=citrix.com); dmarc=pass header.from= (p=reject dis=none) header.from= Return-Path: Received: from lists.xenproject.org (lists.xenproject.org [192.237.175.120]) by mx.zohomail.com with SMTPS id 1622223618653411.7728641716302; Fri, 28 May 2021 10:40:18 -0700 (PDT) Received: from list by lists.xenproject.org with outflank-mailman.134043.249633 (Exim 4.92) (envelope-from ) id 1lmgSs-00027j-Of; Fri, 28 May 2021 17:40:06 +0000 Received: by outflank-mailman (output) from mailman id 134043.249633; Fri, 28 May 2021 17:40:06 +0000 Received: from localhost ([127.0.0.1] helo=lists.xenproject.org) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lmgSs-00027V-KV; Fri, 28 May 2021 17:40:06 +0000 Received: by outflank-mailman (input) for mailman id 134043; Fri, 28 May 2021 17:40:05 +0000 Received: from us1-rack-iad1.inumbo.com ([172.99.69.81]) by lists.xenproject.org with esmtp (Exim 4.92) (envelope-from ) id 1lmgSr-0001si-9c for xen-devel@lists.xenproject.org; Fri, 28 May 2021 17:40:05 +0000 Received: from esa1.hc3370-68.iphmx.com (unknown [216.71.145.142]) by us1-rack-iad1.inumbo.com (Halon) with ESMTPS id 76e5c0c5-a728-4b27-b30c-372172b9795a; Fri, 28 May 2021 17:40:03 +0000 (UTC) X-Outflank-Mailman: Message body and most headers restored to incoming version X-BeenThere: xen-devel@lists.xenproject.org List-Id: Xen developer discussion List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Errors-To: xen-devel-bounces@lists.xenproject.org Precedence: list Sender: "Xen-devel" X-Inumbo-ID: 76e5c0c5-a728-4b27-b30c-372172b9795a DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=citrix.com; s=securemail; t=1622223603; h=from:to:cc:subject:date:message-id:in-reply-to: references:content-transfer-encoding:mime-version; bh=9nedISlwrSegLGQxOvVojP0HNEh86KYexV0VnKl/dG8=; b=FUJX7JZdG9gd2p1alHgC9aB55BfKKsmdyp3vd4zuIIYJHeLVSf0iiE71 VRThPvCbf/FGle+eL9s7LA55w0RLAQsCVxV/fsBjwwjrGRIDC8eV9/y0H dmIm4rYKTVTyYQO+O4Z/+6mu1Gp1ZIRVXEX6F5D3xgn5pudNzrKYYvMct k=; Authentication-Results: esa1.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com IronPort-SDR: +A6iCFZLI6rXnxmFGJOfd3k8JDSXNOVdeDCEXUdwGYwtgSr6OVJ63IB2y9kTZzLm4sktvEoB42 AbzRV5Wc9bFpVbi/Q9T+s4DlEqC0mcDTy6jhBpnGQ0T1bM4Q0e2vdatMiTT+rPKxE26UF60mVd 2eaMqOolIxZvLy2rBemlUZi2tyn9PN7LDBWozyj96Ama/jA2FG7MYl+V+02YjAHaVU1fMxqOHP clbf0fQ2bDuPSxRQvsQUZYt+qgwCsWjnwq3VT/PMDZ5kYqWAkgKmrOXlc3eB+YxUW95piKyeSH mIM= X-SBRS: 5.1 X-MesageID: 45249119 X-Ironport-Server: esa1.hc3370-68.iphmx.com X-Remote-IP: 162.221.158.21 X-Policy: $RELAYED IronPort-HdrOrdr: A9a23:fffGramE+8v+XlF6bWGj5YTKDX3pDfMTimdD5ihNYBxZY6Wkfp +V8sjzhCWatN9OYh0dcLC7WJVpQRvnhPlICO4qTMiftWjdyReVxeRZjLcKrAeQYhEWmtQtsJ uINpIOcuEYbmIK/voSgjPIa+rIqePvmMvD6Ja8vhUdOD2CKZsQkjuRYjzrYnGeLzM2Y6bReq Dsgvau8FGbCAUqh4mAdzc4t6+pnayDqHqICiR2RiIP2U2rt3eF+bT6Gx+X0lM1SDVU24ov9m DDjkjQ+rijm+vT8G6f60bjq7Bt3PfxwNpKA8KBzuIPLC/3twqubIN9H5WfoTEOpv214lpCqq iOn/4ZBbU215rtRBDznfO0sDOQlgrGqkWSiWNwuEGT5PAQH1kBepN8beszSGqq16Ii1OsMnZ 6jkVjp76a+oHv77WzADuPzJmZXf3yP0DAfeN4o/ghiuLQlGclsRPQkjTho+bc7bW7HAdMcYa ZT5P+13occTbrMVQGmgoAo+q32YkgO X-IronPort-AV: E=Sophos;i="5.83,230,1616472000"; d="scan'208";a="45249119" ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=eq0HHQa16ty6I6Sw1KK7gCgNcq4GyIBfkIH7izqrshKu4ASDTD2YlZsrIVPQ6vT1JwdiltXscFW4EoD8B/lBcRhZC+fntCxwHMhmVpqazSNPUTvle66bfoGZFf+PBrSJxV2RsyTR0SNRJ3BaXCIYhjWb9/PLXvgcT9ZS6ahqe0a70/FIo+/aE17uAuamKvqLcsGowbBFdv6y9P+e3DD+9ZHoHTOGY0wZhN98/2K8ZMcHRV4v2wOjPGvS6NtbBdzbHbNxLEoD7HdvZU7IfFjwNP3nMIBkXUKkNZdyjrbVwPNU56/EdFhj8J8Q04uCAIlGCyxcS1W4xO+pMFQN2AW5mA== 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-SenderADCheck; bh=AtL+Hp/29nuKWkROvVXgdUUpYBKLh+7LK1UvsLfBAnw=; b=Ivw8QTXbgd70tuzaa37WLHViUjyfKa6m0Xvdj+Jn427ltUk5cE22rSFbv/khbz0jPgmfZoYPeNfqqzG9HdfeWQ8UXa6CwHGBoBWBeFhUVczmQ/4B6Sx1LCuYdLBHq7JqOstLcA7UXZ+I8mYV0/mQbolTzlR3VXwe7FHa28NUTd9jmfOg9S9HVnmPaojkoARXxz/sHPXcrQOso2ncD31RqhlBgzVB09CsBVgvyAdzImR+zr7iU18avUfKKHpb2WEDuYSfj7szvdolubW7O0d2d4SPDp/wXXVyLGz9M3HlswKtaov04c94q36sXuMFcOYfqfv1gDKRE5UZjtb1vtnDkQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=citrix.onmicrosoft.com; s=selector2-citrix-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=AtL+Hp/29nuKWkROvVXgdUUpYBKLh+7LK1UvsLfBAnw=; b=UJFZGiApEH1vy8jgKaaJZ+4rdKdmjxOjojLnKlEZMMTN8In71yspd7/+zwVhtLdfcZ2azEu+czUq0guRWd/JGC8CdmzQhX6y0/3mzhpf0Ap2oCs/WbuxCTGt8OGTxxxZir6bPFORz5sA222biacpCqU0FzmcE38H+i0VMVn2Pas= From: Roger Pau Monne To: CC: Roger Pau Monne , Jun Nakajima , Kevin Tian , Jan Beulich , Andrew Cooper , Wei Liu , George Dunlap Subject: [PATCH 3/3] x86/ept: force WB cache attributes for grant and foreign maps Date: Fri, 28 May 2021 19:39:35 +0200 Message-ID: <20210528173935.29919-4-roger.pau@citrix.com> X-Mailer: git-send-email 2.31.1 In-Reply-To: <20210528173935.29919-1-roger.pau@citrix.com> References: <20210528173935.29919-1-roger.pau@citrix.com> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable X-ClientProxiedBy: MRXP264CA0003.FRAP264.PROD.OUTLOOK.COM (2603:10a6:500:15::15) To DS7PR03MB5608.namprd03.prod.outlook.com (2603:10b6:5:2c9::18) MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: a4af593d-a375-4a60-cea1-08d921ff9e03 X-MS-TrafficTypeDiagnostic: DM4PR03MB5966: X-LD-Processed: 335836de-42ef-43a2-b145-348c2ee9ca5b,ExtAddr X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:7691; X-MS-Exchange-SenderADCheck: 1 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 0W4ypPRZ+w1jF86gzWKgk6EEFkt5587z3m8IXeFrfv8/pObMpkKfSYnhHc18hM41sAR2pryeQIS1CANCWoLHA7YogZrsGSrjTkw4PSmV2SgPKWvIlu7y44VObq+KXhnLWXFZ/dQIZb6OTHFJhXO17amqd4UzEtdv7PK2FLZzql6RAi7CJGZ0DmJBdbIM5/+HUAOFzWhKLnahWQO22Iub2MCAmtumZbKCABkiv5mxA05+D/uBVTodYCH82b6yGbEm7WLncmQVdXdAz3FS0Evq36Uh5zRDTFvPP0s3Biu/DXjeW52svAQGADP+0LJ0exGO1cl6IaEpBVRzNbCTdHqWJfA2+LO/clUGYAjrNGJYOpfJZDPF0cSE+4mHqbPuh/UDEupfMN8Yp9RedM0k+VcrcE/bdL6Fchr1hx3ej/map4c8v1N63xG9jxuntinHbzPCTwbLH9L8uZKrWrcTxwDmc5UOeica35LaA7gVcSBqPB5yZ7Ei+K6pfG9EU4zP5kHr3bHVUyiLgVEt3IFC3j2a3P+EfG0vfKy4dTWPCfaqQv54yeTPBomMk1zBHapTE58o+w3984i0uVi7qFAnCXePFfRYm7bkHy0DV0tFjibsuK4= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DS7PR03MB5608.namprd03.prod.outlook.com;PTR:;CAT:NONE;SFS:(4636009)(396003)(39860400002)(366004)(376002)(136003)(346002)(6666004)(107886003)(36756003)(2906002)(26005)(2616005)(66476007)(316002)(66556008)(186003)(66946007)(16526019)(54906003)(956004)(4326008)(38100700002)(6486002)(8676002)(83380400001)(478600001)(8936002)(6496006)(86362001)(6916009)(1076003)(5660300002);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?RVMxQnZYM3lmKyt4UmhVN0xBSnA2LzFvc3lUUlRIdnpBdE5mcHBTbmVHbHcw?= =?utf-8?B?VlhUWlRjbUk3OEdoWGhGVDRwTVU5aWZTOGZSWmY2TXhDL1NDVFVRZkNocmpQ?= =?utf-8?B?Qkl3eUlybm1La0lQdTBsOXVNaXNnSDBxemNRY25jUHhGVTNuU0JSY0pualpM?= =?utf-8?B?cGFJMmQ3MXcrT0pkV1ZPK2Yvbkt6VEd4MU16R2Z3THIxVllXdXBSV2ZkWlVl?= =?utf-8?B?a09aQ2UrL3pEeEpaNUVZUDZZaGNxVHh1U1V1aW1sWC96RmN6akJQZmthOWor?= =?utf-8?B?R05lTUFzQnRMUlRKdW1ydzFFOWpqZnIzNzYybnh5REt2SktyblpJaFZwQlQ1?= =?utf-8?B?MHE5NFZMNzJTQm1YVTB4SHJ6YzBEQ3M4VXNPbitRRGp3TlNnNTlTVndkQzdk?= =?utf-8?B?STU4bHhqQlpCWWpENE4vMzBteXI0RUFqU2xuRTNNdVU3UzRSSmtMWHg1L3NJ?= =?utf-8?B?UkVQTjh5cVAzZDgvMU4yNXUwcEk4Q1BaUGFCd2VocWVjT0h5RFRBRHAyd1VZ?= =?utf-8?B?REFvc2R0L09ERm5uNUh4R2tQNDFQd3RVa1VSMmUxTkhhTkQvQm5KYmMrK2Iv?= =?utf-8?B?bTlQQ0xRT3JmQ0ZBTjNFTENWUmxBSDdCOGRqTEhNNUUrQnNDcTc2VTJaWXpI?= =?utf-8?B?R2d0THExL2x3R0hnbUlKc0w1em5ReEs5VHJyZjRXSUh3L2NYTE5LWW5TV1VN?= =?utf-8?B?bWNUWkIzUEp1ZkFxV2FzUFFOQTZjUWZBcHhMQjJ6QWVkT05CbHN1NzVOTlg5?= =?utf-8?B?SEpseVJpaGsvdXl6dzlMbGFVOURQTnQzTVBGZi9obzBKWjVYU09EbEk2bkF0?= =?utf-8?B?VjJCSXhWVlgxNlZNS0Mvc2xQclJjd2Y1RXlPSnZLMFJNeFQ2U2JpbExzd0kz?= =?utf-8?B?THExeDRHR0xoTzMrbjNldFZtM2lhNnRjSVQ0K1FweTg1THRzMVhmYXgyakVr?= =?utf-8?B?Q0ZWTXFId0JkaTBRQlN2My96UlZlVkRtNFlobjliVGNLaUoxeVlBOEpFdXZ4?= =?utf-8?B?dGVMV1Vmcm1lbTYxajlSU0dxM0pYR2tDMmxiMXlQajZmVTRkMTM5dE4wK3hN?= =?utf-8?B?OUYxb0xEK2c0M05iOGEvcTUyYXlyajNueTl5cGs2ZEVCQVlSaVF0d2t4Tkli?= =?utf-8?B?cnU4TVJQKy8zMUhzSHhCWmVYYTBZOEhsU1FacXJNSC81VUVjelhqZkkwM1VU?= =?utf-8?B?UjlDNXFHZWJMaEZkNUZqQWQ0MTZtL0x5aFBFRFhteVI4ZHB0cVIweEc2N3NR?= =?utf-8?B?V2RhYzNKTXpySWxmbkpXWnlIc1RhRkE5Wm04YWlwSTlsaHRBdC80ZDlYSUNt?= =?utf-8?B?OTEvRmxiMW9sZ1g1am5Gdkdnb2w0VnAwSTNUUUJiUno3ZFd2dzA1Wmx0Nmg4?= =?utf-8?B?eGJPeDFDb014R3gyNHFvVWZZamZhdDBTalpvS3lRdThwRzlhNU13blNiS3FU?= =?utf-8?B?bk92YVYwMXJ6NytQK2lzSmFRZ2pOZW14aWU0YUZCNTBYRHRDRlR0RzBCTjZV?= =?utf-8?B?SnNNc0cvV3N0SVhVUEhGMGVCYW5GWGpNTisrbFpjWWY0RTBxenpreEkzb2d6?= =?utf-8?B?SjNaZ0ZKWW5KRGlYQkxZZTVpMXdSWHlGREI0OTViRzh1WTFuSTNRVjdVRDV5?= =?utf-8?B?ZFNMK2puOTB5elc3RzNKSWN1VUFmUnhHYlQ1MXFuOWJacG1LQ3ZYaFFTZHBD?= =?utf-8?B?RUpjZEF3cGRxMmpMeEFXWnlpME5tRW5VU21UNk1yb2Nqc3I5V01pS2pjU0xQ?= =?utf-8?Q?Rf9k7Hxs1otxM2PBqjylduMC5ldK23bpPoYvn4e?= X-MS-Exchange-CrossTenant-Network-Message-Id: a4af593d-a375-4a60-cea1-08d921ff9e03 X-MS-Exchange-CrossTenant-AuthSource: DS7PR03MB5608.namprd03.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 28 May 2021 17:40:00.3203 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 335836de-42ef-43a2-b145-348c2ee9ca5b X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: TdcYqbP2mgA9Fh5hUhfZm4Tx+YmL8FNLGFav7GYHvq1HX0EN9wCvr99EbgxiEdgDEz3eAMm6Zh/ETzQEM0XNrA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR03MB5966 X-OriginatorOrg: citrix.com X-ZohoMail-DKIM: pass (identity @citrix.com) Force WB type for grants and foreign pages. Those are usually mapped over unpopulated physical ranges in the p2m, and those ranges would usually be UC in the MTRR state, which is unlikely to be the correct cache attribute. It's also cumbersome (or even impossible) for the guest to be setting the MTRR type for all those mappings as WB, as MTRR ranges are finite. Note that on AMD we cannot force a cache attribute because of the lack of ignore PAT equivalent, so the behavior here slightly diverges between AMD and Intel (or EPT vs NPT/shadow). Signed-off-by: Roger Pau Monn=C3=A9 Reviewed-by: Jan Beulich Reviewed-by: Kevin Tian --- xen/arch/x86/hvm/vmx/vmx.c | 2 +- xen/arch/x86/mm/p2m-ept.c | 35 ++++++++++++++++++++++++++----- xen/include/asm-x86/hvm/vmx/vmx.h | 2 +- 3 files changed, 32 insertions(+), 7 deletions(-) diff --git a/xen/arch/x86/hvm/vmx/vmx.c b/xen/arch/x86/hvm/vmx/vmx.c index 0d4b47681b..e09b7e3af9 100644 --- a/xen/arch/x86/hvm/vmx/vmx.c +++ b/xen/arch/x86/hvm/vmx/vmx.c @@ -423,7 +423,7 @@ static void domain_creation_finished(struct domain *d) return; =20 ASSERT(epte_get_entry_emt(d, gfn, apic_access_mfn, 0, &ipat, - true) =3D=3D MTRR_TYPE_WRBACK); + p2m_mmio_direct) =3D=3D MTRR_TYPE_WRBACK); ASSERT(ipat); =20 if ( set_mmio_p2m_entry(d, gfn, apic_access_mfn, PAGE_ORDER_4K) ) diff --git a/xen/arch/x86/mm/p2m-ept.c b/xen/arch/x86/mm/p2m-ept.c index f1d1d07e92..59c0325473 100644 --- a/xen/arch/x86/mm/p2m-ept.c +++ b/xen/arch/x86/mm/p2m-ept.c @@ -487,11 +487,12 @@ static int ept_invalidate_emt_range(struct p2m_domain= *p2m, } =20 int epte_get_entry_emt(struct domain *d, gfn_t gfn, mfn_t mfn, - unsigned int order, bool *ipat, bool direct_mmio) + unsigned int order, bool *ipat, p2m_type_t type) { int gmtrr_mtype, hmtrr_mtype; struct vcpu *v =3D current; unsigned long i; + bool direct_mmio =3D type =3D=3D p2m_mmio_direct; =20 *ipat =3D false; =20 @@ -535,9 +536,33 @@ int epte_get_entry_emt(struct domain *d, gfn_t gfn, mf= n_t mfn, } } =20 - if ( direct_mmio ) + switch ( type ) + { + case p2m_mmio_direct: return MTRR_TYPE_UNCACHABLE; =20 + case p2m_grant_map_ro: + case p2m_grant_map_rw: + case p2m_map_foreign: + /* + * Force WB type for grants and foreign pages. Those are usually m= apped + * over unpopulated physical ranges in the p2m, and those would us= ually + * be UC in the MTRR state, which is unlikely to be the correct ca= che + * attribute. It's also cumbersome (or even impossible) for the gu= est + * to be setting the MTRR type for all those mappings as WB, as MT= RR + * ranges are finite. + * + * Note that on AMD we cannot force a cache attribute because of t= he + * lack of ignore PAT equivalent, so the behavior here slightly + * diverges. See p2m_type_to_flags for the AMD attributes. + */ + *ipat =3D true; + return MTRR_TYPE_WRBACK; + + default: + break; + } + gmtrr_mtype =3D hvm_get_mem_pinned_cacheattr(d, gfn, order); if ( gmtrr_mtype >=3D 0 ) { @@ -640,7 +665,7 @@ static int resolve_misconfig(struct p2m_domain *p2m, un= signed long gfn) continue; e.emt =3D epte_get_entry_emt(p2m->domain, _gfn(gfn + i= ), _mfn(e.mfn), 0, &ipat, - e.sa_p2mt =3D=3D p2m_mmio_d= irect); + e.sa_p2mt); e.ipat =3D ipat; =20 nt =3D p2m_recalc_type(e.recalc, e.sa_p2mt, p2m, gfn += i); @@ -659,7 +684,7 @@ static int resolve_misconfig(struct p2m_domain *p2m, un= signed long gfn) int emt =3D epte_get_entry_emt(p2m->domain, _gfn(gfn), _mfn(e.mfn), level * EPT_TABLE_ORDER, &ipa= t, - e.sa_p2mt =3D=3D p2m_mmio_dir= ect); + e.sa_p2mt); bool_t recalc =3D e.recalc; =20 if ( recalc && p2m_is_changeable(e.sa_p2mt) ) @@ -895,7 +920,7 @@ ept_set_entry(struct p2m_domain *p2m, gfn_t gfn_, mfn_t= mfn, bool ipat; int emt =3D epte_get_entry_emt(p2m->domain, _gfn(gfn), mfn, i * EPT_TABLE_ORDER, &ipat, - p2mt =3D=3D p2m_mmio_direct); + p2mt); =20 if ( emt >=3D 0 ) new_entry.emt =3D emt; diff --git a/xen/include/asm-x86/hvm/vmx/vmx.h b/xen/include/asm-x86/hvm/vm= x/vmx.h index f668ee1f09..0deb507490 100644 --- a/xen/include/asm-x86/hvm/vmx/vmx.h +++ b/xen/include/asm-x86/hvm/vmx/vmx.h @@ -600,7 +600,7 @@ void ept_p2m_uninit(struct p2m_domain *p2m); void ept_walk_table(struct domain *d, unsigned long gfn); bool_t ept_handle_misconfig(uint64_t gpa); int epte_get_entry_emt(struct domain *d, gfn_t gfn, mfn_t mfn, - unsigned int order, bool *ipat, bool direct_mmio); + unsigned int order, bool *ipat, p2m_type_t type); void setup_ept_dump(void); void p2m_init_altp2m_ept(struct domain *d, unsigned int i); /* Locate an alternate p2m by its EPTP */ --=20 2.31.1