All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sean Christopherson <seanjc@google.com>
To: Claudio Imbrenda <imbrenda@linux.ibm.com>
Cc: "Carlos López" <clopez@suse.de>,
	kvm@vger.kernel.org, linux-mm@kvack.org, pbonzini@redhat.com,
	linux-s390@vger.kernel.org
Subject: Re: [RFC PATCH 0/4] KVM: x86/mmu: don't zap secondary MMU on permission upgrade
Date: Mon, 2 Nov 2026 07:31:29 -0800	[thread overview]
Message-ID: <auis0SaMkRB6odsK@google.com> (raw)
In-Reply-To: <20261030133939.630e29af@p-imbrenda>

On Fri, Oct 30, 2026, Claudio Imbrenda wrote:
> On Fri, 30 Oct 2026 12:11:07 +0100
> Carlos López <clopez@suse.de> wrote:
> 
> [...]
> 
> > > regarding this series:
> > > 
> > > on x86 you don't care what the new permissions are, because you're
> > > updating when re-faulting (if I understood correctly).  
> > 
> > Correct.
> > 
> > > for s390 it might be beneficial to know what the new permissions are;
> > > of course we could look up the vma and find out, but since you are
> > > already checking the permissions, maybe you can find a way to also
> > > convey the new permission bits?
> > > 
> > > that would make our life on s390 easier :)  
> > 
> > Right, getting the new bits from vma->flags should be trivially done
> > from the MMU_NOTIFY_PROTECTION_VMA callsites. There'd be no users in
> > this series though.  The rest of architectures still need the notifier to
> > tell them if the change is an upgrade, since they don't have an easy way
> > to grab the old RWX bits. Passing the old RWX bits from the MMU notifier
> > does not seem trivial to me either.

But why is core MM even doing mmu_notifier_invalidate_range_{start,end}() in the
first place?  I.e. why add MMU_NOTIFIER_RANGE_PERM_UPGRADE?  Just skip the
"invalidation".


  reply	other threads:[~2026-11-02 15:31 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-10-29 14:33 [RFC PATCH 0/4] KVM: x86/mmu: don't zap secondary MMU on permission upgrade Carlos López
2026-10-29 14:33 ` [RFC PATCH 1/4] mm/mprotect: add MM_CP_PERM_UPGRADE Carlos López
2026-10-30 14:44   ` sashiko-bot
2026-10-29 14:33 ` [RFC PATCH 2/4] mm/mmu_notifier: add MMU_NOTIFIER_RANGE_PERM_UPGRADE Carlos López
2026-10-30 14:44   ` sashiko-bot
2026-11-04 19:33   ` kernel test robot
2026-11-04 21:34   ` kernel test robot
2026-10-29 14:33 ` [RFC PATCH 3/4] KVM: plumb MMU_NOTIFIER_RANGE_PERM_UPGRADE into kvm_gfn_range Carlos López
2026-10-30 14:44   ` sashiko-bot
2026-10-29 14:33 ` [RFC PATCH 4/4] KVM: x86/mmu: don't zap the secondary MMU on permission upgrade Carlos López
2026-10-30 14:44   ` sashiko-bot
2026-10-29 16:52 ` [RFC PATCH 0/4] KVM: x86/mmu: don't zap " Claudio Imbrenda
2026-10-30 11:11   ` Carlos López
2026-10-30 12:39     ` Claudio Imbrenda
2026-11-02 15:31       ` Sean Christopherson [this message]
2026-11-03 11:09         ` Carlos López
2026-11-03 15:03           ` Sean Christopherson
2026-11-04  0:38             ` Carlos López
2026-11-04 15:38               ` Sean Christopherson

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=auis0SaMkRB6odsK@google.com \
    --to=seanjc@google.com \
    --cc=clopez@suse.de \
    --cc=imbrenda@linux.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    /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.