linux-kernel - Re: [PATCH v3 2/2] s390/mm: re-enable the shared zeropage for !PV and !skeys KVM guests

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <Zh0oUwOqQaUxqqOt@li-008a6a4c-3549-11b2-a85c-c5cc2836eea2.ibm.com>
Date: Mon, 15 Apr 2024 15:14:59 +0200
From: Alexander Gordeev <agordeev@...ux.ibm.com>
To: Christian Borntraeger <borntraeger@...ux.ibm.com>
Cc: David Hildenbrand <david@...hat.com>, linux-kernel@...r.kernel.org,
        linux-mm@...ck.org, Janosch Frank <frankja@...ux.ibm.com>,
        Claudio Imbrenda <imbrenda@...ux.ibm.com>,
        Heiko Carstens <hca@...ux.ibm.com>, Vasily Gorbik <gor@...ux.ibm.com>,
        Andrew Morton <akpm@...ux-foundation.org>,
        Peter Xu <peterx@...hat.com>, Sven Schnelle <svens@...ux.ibm.com>,
        Gerald Schaefer <gerald.schaefer@...ux.ibm.com>,
        Andrea Arcangeli <aarcange@...hat.com>, kvm@...r.kernel.org,
        linux-s390@...r.kernel.org
Subject: Re: [PATCH v3 2/2] s390/mm: re-enable the shared zeropage for !PV
 and !skeys KVM guests

On Mon, Apr 15, 2024 at 01:49:15PM +0200, Christian Borntraeger wrote:

Hi Christian,

> > On 11.04.24 18:37, Alexander Gordeev wrote:
> > > On Thu, Apr 11, 2024 at 06:14:41PM +0200, David Hildenbrand wrote:
> > > David, Christian,
> > > > Tested-by: Christian Borntraeger <borntraeger@...ux.ibm.com>
> > > Please, correct me if I am wrong, but (to my understanding) the
> > > Tested-by for v2 does not apply for this version of the patch?
> > I thought I'd removed it -- you're absolutely, this should be dropped. Hopefully Christian has time to retest.
> 
> So I can confirm that this patch does continue fix the qemu memory consumption for a guest doing managedsave/start.

I will re-add your Tested-by.

> A quick check of other aspects seems to be ok. We will have more coverage on the base functionality as soon as it hits next(via Andrew) as our daily CI will pick this up for lots of KVM tests.

As per the cover letter I will pull it via s390 tree:
"Based on s390/features. Andrew agreed that both patches can go via the
s390x tree."

Thanks!