public inbox for kvm@vger.kernel.org
 help / color / mirror / Atom feed
From: Claudio Imbrenda <imbrenda@linux.ibm.com>
To: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-s390@vger.kernel.org, frankja@linux.ibm.com,
	nsg@linux.ibm.com, nrb@linux.ibm.com, seiden@linux.ibm.com,
	schlameuss@linux.ibm.com, hca@linux.ibm.com, svens@linux.ibm.com,
	agordeev@linux.ibm.com, gor@linux.ibm.com, david@redhat.com,
	gerald.schaefer@linux.ibm.com
Subject: Re: [PATCH v3 21/23] KVM: s390: Enable 1M pages for gmap
Date: Thu, 6 Nov 2025 17:36:50 +0100	[thread overview]
Message-ID: <20251106173650.31907261@p-imbrenda> (raw)
In-Reply-To: <5dd6e694-8cf9-4b1c-ae83-088b6bd22a17@de.ibm.com>

On Thu, 6 Nov 2025 17:22:33 +0100
Christian Borntraeger <borntraeger@de.ibm.com> wrote:

> Am 06.11.25 um 17:11 schrieb Claudio Imbrenda:
> > While userspace is allowed to have pages of any size, the new gmap
> > would always use 4k pages to back the guest.
> > 
> > Enable 1M pages for gmap.
> > 
> > This allows 1M pages to be used to back a guest when userspace is using
> > 1M pages for the corresponding addresses (e.g. THP or hugetlbfs).
> > 
> > Remove the limitation that disallowed having nested guests and
> > hugepages at the same time.  
> 
> Nice. This might allow us to enable hpage=1 as new default as soon as
> things stabilize.
> 
> We would also be able to use 2GB huge pages for the qemu mapping?

yes, but I don't think that userspace can have 2G THPs right now

> With this patch we then have 1MB pages for the guest mapping.
> 
> As a future improvement, maybe also allow guest mapping 2GB pages.

the core of the new gmap code introduced in this series supports 2G
pages already; to enable 2G pages, a simple patch similar to this one
is all that's needed.

I refrained from enabling 2G pages in this series because this series
is already too big, and also I would not be able to test it easily.

  reply	other threads:[~2025-11-06 16:37 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-11-06 16:10 [PATCH v3 00/23] KVM: s390: gmap rewrite, the real deal Claudio Imbrenda
2025-11-06 16:10 ` [PATCH v3 01/23] KVM: s390: Refactor pgste lock and unlock functions Claudio Imbrenda
2025-11-11 11:19   ` Christoph Schlameuss
2025-11-06 16:10 ` [PATCH v3 02/23] KVM: s390: add P bit in table entry bitfields, move union vaddress Claudio Imbrenda
2025-11-06 16:10 ` [PATCH v3 03/23] s390: Move sske_frame() to a header Claudio Imbrenda
2025-11-06 16:10 ` [PATCH v3 04/23] KVM: s390: Add gmap_helper_set_unused() Claudio Imbrenda
2025-11-12  6:25   ` Christoph Schlameuss
2025-11-06 16:10 ` [PATCH v3 05/23] KVM: s390: Enable KVM_GENERIC_MMU_NOTIFIER Claudio Imbrenda
2025-11-13  9:46   ` Christoph Schlameuss
2025-11-06 16:11 ` [PATCH v3 06/23] KVM: s390: Rename some functions in gaccess.c Claudio Imbrenda
2025-11-13  9:52   ` Christoph Schlameuss
2025-11-06 16:11 ` [PATCH v3 07/23] KVM: s390: KVM-specific bitfields and helper functions Claudio Imbrenda
2025-11-13 10:18   ` Christoph Schlameuss
2025-11-06 16:11 ` [PATCH v3 08/23] KVM: s390: KVM page table management functions: allocation Claudio Imbrenda
2025-11-20  8:50   ` Heiko Carstens
2025-11-06 16:11 ` [PATCH v3 09/23] KVM: s390: KVM page table management functions: clear and replace Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 10/23] KVM: s390: KVM page table management functions: walks Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 11/23] KVM: s390: KVM page table management functions: storage keys Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 12/23] KVM: s390: KVM page table management functions: lifecycle management Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 13/23] KVM: s390: KVM page table management functions: CMMA Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 14/23] KVM: s390: New gmap code Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 15/23] KVM: s390: Add helper functions for fault handling Claudio Imbrenda
2025-11-18 15:10   ` Heiko Carstens
2025-11-19 11:06     ` Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 16/23] KVM: s390: Add some helper functions needed for vSIE Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 17/23] KVM: s390: Stop using CONFIG_PGSTE Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 18/23] KVM: s390: Switch to new gmap Claudio Imbrenda
2025-11-18 15:14   ` Heiko Carstens
2025-11-19 11:12     ` Claudio Imbrenda
2025-11-19 13:00     ` Claudio Imbrenda
2025-11-18 15:18   ` Heiko Carstens
2025-11-19 11:13     ` Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 19/23] KVM: s390: Remove gmap from s390/mm Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 20/23] KVM: S390: Remove PGSTE code from linux/s390 mm Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 21/23] KVM: s390: Enable 1M pages for gmap Claudio Imbrenda
2025-11-06 16:22   ` Christian Borntraeger
2025-11-06 16:36     ` Claudio Imbrenda [this message]
2025-11-06 16:37       ` Christian Borntraeger
2025-11-06 16:11 ` [PATCH v3 22/23] KVM: s390: Storage key manipulation IOCTL Claudio Imbrenda
2025-11-06 16:11 ` [PATCH v3 23/23] KVM: s390: Fix storage key memop IOCTLs Claudio Imbrenda
2025-11-18 15:28 ` [PATCH v3 00/23] KVM: s390: gmap rewrite, the real deal Heiko Carstens
2025-11-19 11:14   ` Claudio Imbrenda

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=20251106173650.31907261@p-imbrenda \
    --to=imbrenda@linux.ibm.com \
    --cc=agordeev@linux.ibm.com \
    --cc=borntraeger@de.ibm.com \
    --cc=david@redhat.com \
    --cc=frankja@linux.ibm.com \
    --cc=gerald.schaefer@linux.ibm.com \
    --cc=gor@linux.ibm.com \
    --cc=hca@linux.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=nrb@linux.ibm.com \
    --cc=nsg@linux.ibm.com \
    --cc=schlameuss@linux.ibm.com \
    --cc=seiden@linux.ibm.com \
    --cc=svens@linux.ibm.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox