Received-SPF: pass (google.com: domain of linux-kernel+bounces-210778-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Wed, 12 Jun 2024 01:50:35 +0000
In-Reply-To: <Zmj+I2jtZLkFGgpd@yzhao56-desk.sh.intel.com>
Precedence: bulk
Mime-Version: 1.0
References: <20240605224527.2907272-1-seanjc@google.com> <171803644297.3355626.18364027225741935056.b4-ty@google.com>
 <Zmj+I2jtZLkFGgpd@yzhao56-desk.sh.intel.com>
Message-ID: <Zmj-6wyzIFHuQw-4@google.com>
Subject: Re: [PATCH 0/2] nVMX: Ensure host's PAT stays sane
From: Sean Christopherson <seanjc@google.com>
To: Yan Zhao <yan.y.zhao@intel.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>, linux-kernel@vger.kernel.org, 
	Xiangfei Ma <xiangfeix.ma@intel.com>
Content-Type: text/plain; charset="us-ascii"

On Wed, Jun 12, 2024, Yan Zhao wrote:
> 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/ :)

Doh, I did squash that, I just forgot to update the "thanks" email with the
correct hashes.

[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/184ee0d5
[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/ee1d79c3