linux-kernel - Re: [PATCH v3 00/20] vmw_balloon: compaction, shrinker, 64-bit, etc.

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: <0AC59738-06A0-43DC-8622-D4177FDDC1F3@vmware.com>
Date:   Tue, 30 Oct 2018 16:52:55 +0000
From:   Nadav Amit <namit@...are.com>
To:     "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
CC:     Arnd Bergmann <arnd@...db.de>,
        Xavier Deguillard <xdeguillard@...are.com>,
        LKML <linux-kernel@...r.kernel.org>,
        "Michael S. Tsirkin" <mst@...hat.com>,
        Jason Wang <jasowang@...hat.com>,
        "linux-mm@...ck.org" <linux-mm@...ck.org>,
        "virtualization@...ts.linux-foundation.org" 
        <virtualization@...ts.linux-foundation.org>
Subject: Re: [PATCH v3 00/20] vmw_balloon: compaction, shrinker, 64-bit, etc.

From: gregkh@...uxfoundation.org
Sent: October 30, 2018 at 4:51:19 PM GMT
> To: Nadav Amit <namit@...are.com>
> Cc: Arnd Bergmann <arnd@...db.de>, Xavier Deguillard <xdeguillard@...are.com>, LKML <linux-kernel@...r.kernel.org>, Michael S. Tsirkin <mst@...hat.com>, Jason Wang <jasowang@...hat.com>, linux-mm@...ck.org <linux-mm@...ck.org>, virtualization@...ts.linux-foundation.org <virtualization@...ts.linux-foundation.org>
> Subject: Re: [PATCH v3 00/20] vmw_balloon: compaction, shrinker, 64-bit, etc.
> 
> 
> On Tue, Oct 30, 2018 at 04:32:22PM +0000, Nadav Amit wrote:
>> From: Nadav Amit
>> Sent: September 26, 2018 at 7:13:16 PM GMT
>>> To: Arnd Bergmann <arnd@...db.de>, gregkh@...uxfoundation.org
>>> Cc: Xavier Deguillard <xdeguillard@...are.com>, linux-kernel@...r.kernel.org>, Nadav Amit <namit@...are.com>, Michael S. Tsirkin <mst@...hat.com>, Jason Wang <jasowang@...hat.com>, linux-mm@...ck.org>, virtualization@...ts.linux-foundation.org
>>> Subject: [PATCH v3 00/20] vmw_balloon: compaction, shrinker, 64-bit, etc.
>>> 
>>> 
>>> This patch-set adds the following enhancements to the VMware balloon
>>> driver:
>>> 
>>> 1. Balloon compaction support.
>>> 2. Report the number of inflated/deflated ballooned pages through vmstat.
>>> 3. Memory shrinker to avoid balloon over-inflation (and OOM).
>>> 4. Support VMs with memory limit that is greater than 16TB.
>>> 5. Faster and more aggressive inflation.
>>> 
>>> To support compaction we wish to use the existing infrastructure.
>>> However, we need to make slight adaptions for it. We add a new list
>>> interface to balloon-compaction, which is more generic and efficient,
>>> since it does not require as many IRQ save/restore operations. We leave
>>> the old interface that is used by the virtio balloon.
>>> 
>>> Big parts of this patch-set are cleanup and documentation. Patches 1-13
>>> simplify the balloon code, document its behavior and allow the balloon
>>> code to run concurrently. The support for concurrency is required for
>>> compaction and the shrinker interface.
>>> 
>>> For documentation we use the kernel-doc format. We are aware that the
>>> balloon interface is not public, but following the kernel-doc format may
>>> be useful one day.
>>> 
>>> v2->v3: * Moving the balloon magic-number out of uapi (Greg)
>>> 
>>> v1->v2:	* Fix build error when THP is off (kbuild)
>>> 	* Fix build error on i386 (kbuild)
>> 
>> Greg,
>> 
>> I realize you didn’t apply patches 17-20. Any reason for that?
> 
> I have no idea, that was a few thousand patches reviewed ago...
> 
> Did I not say anything about this when I applied them?
> 
> greg k-h

You regarded the magic-number in v2, which I fixed for v3.

Should I resend?