linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Alexander Duyck <alexander.duyck@gmail.com>
To: "Bodeddula, Balasubramaniam" <bodeddub@amazon.com>
Cc: "aarcange@redhat.com" <aarcange@redhat.com>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
	"alexander.h.duyck@linux.intel.com" 
	<alexander.h.duyck@linux.intel.com>,
	"dan.j.williams@intel.com" <dan.j.williams@intel.com>,
	"dave.hansen@intel.com" <dave.hansen@intel.com>,
	"david@redhat.com" <david@redhat.com>,
	"konrad.wilk@oracle.com" <konrad.wilk@oracle.com>,
	"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
	"lcapitulino@redhat.com" <lcapitulino@redhat.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"linux-mm@kvack.org" <linux-mm@kvack.org>,
	"mgorman@techsingularity.net" <mgorman@techsingularity.net>,
	"mhocko@kernel.org" <mhocko@kernel.org>,
	"mst@redhat.com" <mst@redhat.com>,
	"nitesh@redhat.com" <nitesh@redhat.com>,
	"osalvador@suse.de" <osalvador@suse.de>,
	"pagupta@redhat.com" <pagupta@redhat.com>,
	"pbonzini@redhat.com" <pbonzini@redhat.com>,
	"riel@surriel.com" <riel@surriel.com>,
	"vbabka@suse.cz" <vbabka@suse.cz>,
	"wei.w.wang@intel.com" <wei.w.wang@intel.com>,
	"willy@infradead.org" <willy@infradead.org>,
	"yang.zhang.wz@gmail.com" <yang.zhang.wz@gmail.com>, "Graf (AWS),
	Alexander" <graf@amazon.de>,
	"Herrenschmidt, Benjamin" <benh@amazon.com>
Subject: Re: [PATCH v17 1/9] mm: Adjust shuffle code to allow for future coalescing
Date: Wed, 10 Mar 2021 17:05:12 -0800	[thread overview]
Message-ID: <CAKgT0Ud508jLc4NVm1XxNSjEnq3NoLS=Q1V+o=6JZoJF_r_m0A@mail.gmail.com> (raw)
In-Reply-To: <A221FEC9-71CE-4D0A-9F39-F75C337B5D22@contoso.com>

Hi Bala,

There was a similar effort several months ago that was trying to do
this in conjunction with pre-zeroing of pages. I suspect if you wanted
to you could probably pick up some of their patch set and work with
that. It can be found at:
https://www.spinics.net/lists/linux-mm/msg239735.html

Thanks.

- Alex

On Tue, Mar 9, 2021 at 12:13 AM Bodeddula, Balasubramaniam
<bodeddub@amazon.com> wrote:
>
> Hi Alexander,
>
>
>
> My team was evaluating FPR and observed that these patches don’t report memory for deallocated hugeapages directly and need to cycle through buddy allocator. For example, say we need to allocate a maximum of 12 * 1G hugepages (by setting nr_hugepages), use 8 * 1G hugepages, and then deallocate 4 * 1G hugepages. Unlike regular 4K pages, this 4G worth of memory will not be reported until we set nr_hugepages to 8 (wait sometime(?) for FPR to do its work) and set it back again to 12. While this works fine in theory, in practice,  setting nr_hugepages to 12 could fail too due to fragmentation (this could depend on other processes memory usage behavior).
>
>
>
> If FPR could report this free memory without cycling through buddy allocator, it makes the solution more robust. I am looking for advice on how feasible this approach is and what would be the effort for building this functionality. In general, if there are other thoughts on how we can address this, please do let me know.
>
>
>
> Thanks,
>
> bala

       reply	other threads:[~2021-03-11  1:06 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <A221FEC9-71CE-4D0A-9F39-F75C337B5D22@contoso.com>
2021-03-11  1:05 ` Alexander Duyck [this message]
2020-02-11 22:45 [PATCH v17 0/9] mm / virtio: Provide support for free page reporting Alexander Duyck
2020-02-11 22:46 ` [PATCH v17 1/9] mm: Adjust shuffle code to allow for future coalescing Alexander Duyck

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='CAKgT0Ud508jLc4NVm1XxNSjEnq3NoLS=Q1V+o=6JZoJF_r_m0A@mail.gmail.com' \
    --to=alexander.duyck@gmail.com \
    --cc=aarcange@redhat.com \
    --cc=akpm@linux-foundation.org \
    --cc=alexander.h.duyck@linux.intel.com \
    --cc=benh@amazon.com \
    --cc=bodeddub@amazon.com \
    --cc=dan.j.williams@intel.com \
    --cc=dave.hansen@intel.com \
    --cc=david@redhat.com \
    --cc=graf@amazon.de \
    --cc=konrad.wilk@oracle.com \
    --cc=kvm@vger.kernel.org \
    --cc=lcapitulino@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mgorman@techsingularity.net \
    --cc=mhocko@kernel.org \
    --cc=mst@redhat.com \
    --cc=nitesh@redhat.com \
    --cc=osalvador@suse.de \
    --cc=pagupta@redhat.com \
    --cc=pbonzini@redhat.com \
    --cc=riel@surriel.com \
    --cc=vbabka@suse.cz \
    --cc=wei.w.wang@intel.com \
    --cc=willy@infradead.org \
    --cc=yang.zhang.wz@gmail.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;
as well as URLs for NNTP newsgroup(s).