All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sean Christopherson <seanjc@google.com>
To: Fabio Coatti <fabio.coatti@gmail.com>
Cc: Bagas Sanjaya <bagasdotme@gmail.com>,
	stable@vger.kernel.org,  regressions@lists.linux.dev,
	kvm@vger.kernel.org,  Junaid Shahid <junaids@google.com>,
	Paolo Bonzini <pbonzini@redhat.com>
Subject: Re: WARNING trace at kvm_nx_huge_page_recovery_worker on 6.3.4
Date: Tue, 30 May 2023 10:37:30 -0700	[thread overview]
Message-ID: <ZHY0WkNlui91Mxoj@google.com> (raw)
In-Reply-To: <CADpTngWiXNh1wAFM_EYGm-Coa8nv61Tu=3TG+Z2dVCojp2K1yg@mail.gmail.com>

On Tue, May 30, 2023, Fabio Coatti wrote:
> Il giorno dom 28 mag 2023 alle ore 14:44 Bagas Sanjaya
> <bagasdotme@gmail.com> ha scritto:
> > #regzbot ^introduced: v6.3.1..v6.3.2
> > #regzbot title: WARNING trace at kvm_nx_huge_page_recovery_worker when opening a new tab in Chrome
> 
> Out of curiosity, I recompiled 6.3.4 after reverting the following
> commit mentioned in 6.3.2 changelog:
> 
> commit 2ec1fe292d6edb3bd112f900692d9ef292b1fa8b
> Author: Sean Christopherson <seanjc@google.com>
> Date:   Wed Apr 26 15:03:23 2023 -0700
> KVM: x86: Preserve TDP MMU roots until they are explicitly invalidated
> commit edbdb43fc96b11b3bfa531be306a1993d9fe89ec upstream.
> 
> And the WARN message no longer appears on my host kernel logs, at
> least so far :)

Hmm, more than likely an NX shadow page is outliving a memslot update.  I'll take
another look at those flows to see if I can spot a race or leak.

> > Fabio, can you also check the mainline (on guest)?
> 
> Not sure to understand, you mean 6.4-rcX? I can do that, sure, but why
> on guest?

Misunderstanding probably?  Please do test with 6.4-rcX on the host.  I expect
the WARN to reproduce there as well, but if it doesn't then we'll have a very
useful datapoint.

  reply	other threads:[~2023-05-30 17:37 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-26  7:43 WARNING trace at kvm_nx_huge_page_recovery_worker on 6.3.4 Fabio Coatti
2023-05-26  8:34 ` Bagas Sanjaya
2023-05-26 17:01 ` Sean Christopherson
2023-05-28  9:22   ` Fabio Coatti
2023-05-28 10:54   ` Fabio Coatti
2023-05-28 12:44 ` Bagas Sanjaya
2023-05-30 10:42   ` Fabio Coatti
2023-05-30 17:37     ` Sean Christopherson [this message]
2023-05-31  1:27       ` Sean Christopherson
2023-05-31  2:04         ` Sean Christopherson
2023-06-01  8:38           ` Fabio Coatti

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZHY0WkNlui91Mxoj@google.com \
    --to=seanjc@google.com \
    --cc=bagasdotme@gmail.com \
    --cc=fabio.coatti@gmail.com \
    --cc=junaids@google.com \
    --cc=kvm@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    --cc=regressions@lists.linux.dev \
    --cc=stable@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.