Received-SPF: pass (google.com: domain of linux-kernel+bounces-210775-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Wed, 12 Jun 2024 09:47:15 +0800
From: Yan Zhao <yan.y.zhao@intel.com>
To: Sean Christopherson <seanjc@google.com>
CC: Paolo Bonzini <pbonzini@redhat.com>, <linux-kernel@vger.kernel.org>,
	Xiangfei Ma <xiangfeix.ma@intel.com>
Subject: Re: [PATCH 0/2] nVMX: Ensure host's PAT stays sane
Message-ID: <Zmj+I2jtZLkFGgpd@yzhao56-desk.sh.intel.com>
Reply-To: Yan Zhao <yan.y.zhao@intel.com>
References: <20240605224527.2907272-1-seanjc@google.com>
 <171803644297.3355626.18364027225741935056.b4-ty@google.com>
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <171803644297.3355626.18364027225741935056.b4-ty@google.com>
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?nLlX9DQPK46XkwvHcbuLRREbW35BH+Fr1L7jKytO6ThX/iG0323CGoUZ7XKf?=
 =?us-ascii?Q?2LlST6j2n/E0qZIlR/lzX9A7sY6W27jTvqOMfaGzLXDYxSf5ZijZJEEk5czM?=
 =?us-ascii?Q?f3ppsFCQtBymq+2QPMWQzZs0In7BSC7+KBC2C+tKaDG0JuaN0VP82WbskEmv?=
 =?us-ascii?Q?S4gn86taJf4Gw5tNE8rIBs5Aar2QoufQW8FMLmS6fTWvzNtXUsEjlM2Th9Hc?=
 =?us-ascii?Q?ox7iomsn5Q0Uy0geUYgtnwOf/3H8cYjiexu1z5GirlnlOPUSxRt4W1+nXvtI?=
 =?us-ascii?Q?HvAzEgfFCFYmH27XztiMzWRtaT04AME+VzqSW52IJq275Y/jUjnx3IhdIW7m?=
 =?us-ascii?Q?U+DSEbKvu+e+AD6p3zcMzrSisa7JoKnKfR7d61b7Se5PjBqzGGjivgTPj+0J?=
 =?us-ascii?Q?9SgOaBnMoDVQ8JkRC2wQTnTqPLDL7DLZNnE3H1r7mC3PzrW2/JXwUt5YjVf4?=
 =?us-ascii?Q?dppkpCqgPlCxESCuAtC+IfM2B9gCq1a+cl1JYQdmhOOue+T/xKOw6HPA27wS?=
 =?us-ascii?Q?GyQ9LOVXThvJBe74HxDQkPjKlJEj7IIfPQ1P6I/j+NzUrPSbti2kH+3yA2Iz?=
 =?us-ascii?Q?xJjq4PuOZCbfFHsQO0sqoFC20lA+Xur6FchO92mjRTTzws6+lle9Kb4IjKFX?=
 =?us-ascii?Q?Q0Baquwv4vUiE0GbNk1SyztpNRvV9ra7/317PntYKPuiNnTfoqT04XViNmOc?=
 =?us-ascii?Q?U/N6w0pUqA2N2ulFiFnbVNIlyfyMxaeTGiu3rfkFcXX29XKaSJFMMcRXeVpP?=
 =?us-ascii?Q?clq/TY2W1AaN5rb9JW2IBfbhbA82/+cf/argymOb0VQmKKB3ZNgH0IZxE0/N?=
 =?us-ascii?Q?MwfLrxXSrPB4ujAetT7m55WD6zXUHGTSGdIhMR+JUeHOsUbzKabGf5WlvL1a?=
 =?us-ascii?Q?W4ubKWBEj4ZZQ8r4+lMFkO+8HTIBBBzeoOXGHNXIA5KtsAmuw9OXVcsepC6d?=
 =?us-ascii?Q?HBUISciO65l62Oa3rhjWnyAgRPEUDHjbv5IZtXhiBQasljwyJthPYSpxUmGn?=
 =?us-ascii?Q?HsGbpIZm0WqDWJ1vV8DhjD0q+97QSFA/YTMPqGtTVnaZY8Va2U2l3R5llYK5?=
 =?us-ascii?Q?LVI8NZ5iZjwjYBJRcXwGaK98y8IVb8e+7y2bTjHa5vpLrrA6/TcZyn8p/den?=
 =?us-ascii?Q?efMLJHOtlTTZ4lnvJ/b1ln1lfP5h+pZBQMQbHyeLYvuEZpwcIVS4ztEhr9o4?=
 =?us-ascii?Q?cdOjx8BqQF0mQYomQPjQ0TvauCFujfb0zC2cKnbot9Gg7LF0pIQLMfalkSUR?=
 =?us-ascii?Q?mPkgk5s3kBdRsCvEMtmuokwCpvCKgdV4qBIjNiNKUBIwj8lL18DvaYvQyrjU?=
 =?us-ascii?Q?vNbvzZ7FD2xgSgqKCSb7ixRNJTh81QAU9TuhlI296MoTHogvax3NxL8gHByh?=
 =?us-ascii?Q?RvV8PsRKesR1XoWiEC1ZULf0UM1r/iT8Ndet3Q9OIPMK5uYjc8R6dTIkuXTF?=
 =?us-ascii?Q?z6rUa2Ju1y9fF8/ooYDNdOqvDmWJg8YudwQTxwovfjOX+p44MqWO+K5SK50K?=
 =?us-ascii?Q?I8YqtIXkIOP+kGc0MzcjKNp5MsrWO/TBvUgCBiMs1N9v2bHIGFffOiPYTLXy?=
 =?us-ascii?Q?QeBc25AQ+CIsYaDeDhFHeRqpmkR1szV0z7qifU1q?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 6074c88a-7fc4-4de5-0c06-08dc8a81b99d
X-MS-Exchange-CrossTenant-AuthSource: DS7PR11MB5966.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 12 Jun 2024 01:48:15.0423
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: +qPwJzEN8ms/n273VYENUIwHemrk1pR4MA10L5WbkhVAXpJ1nfY2oGVFMMgoUOVE3HBPw+fP9U8dp3s8XO231A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR11MB5326
X-OriginatorOrg: intel.com

On Tue, Jun 11, 2024 at 06:18:40PM -0700, Sean Christopherson wrote:
> On Wed, 05 Jun 2024 15:45:25 -0700, Sean Christopherson wrote:
> > Fix a bug(s) where test_load_host_pat clobbers PAT and causes all
> > subsequent tests to effectively run with all memory mapped as UC.  Xiangfei
> > first noticed that this caused rdtsc_vmexit_diff_test to fail, but it can
> > also lead to timeouts if more tests are added, i.e. if more stuff runs after
> > test_load_host_pat.
> > 
> > Sean Christopherson (2):
> >   nVMX: Ensure host's PAT is loaded at the end of all VMX tests
> >   nVMX: Verify KVM actually loads the value in HOST_PAT into the PAT MSR
> > 
> > [...]
> 
> Applied to kvm-x86 next, thanks!
> 
> [1/2] nVMX: Ensure host's PAT is loaded at the end of all VMX tests
>       https://github.com/kvm-x86/kvm-unit-tests/commit/8cbb8d3cbca9
This is also good, though I thought you would fix the theoretical one as
https://lore.kernel.org/all/ZmesywdpFG3qDIkZ@google.com/ :)

> [2/2] nVMX: Verify KVM actually loads the value in HOST_PAT into the PAT MSR
>       https://github.com/kvm-x86/kvm-unit-tests/commit/d31433807f2b
> 
> --
> https://github.com/kvm-x86/kvm-unit-tests/tree/next