From: Florian Weimer <fw@deneb.enyo.de>
To: nmanthey@conp-solutions.com
Cc: libc-alpha@sourceware.org, Siddhesh Poyarekar <siddhesh@sourceware.org>
Subject: Re: [[RFC][PATCH] v1 0/2] malloc/realloc with transparent huge page support
Date: Mon, 04 May 2020 10:27:05 +0200 [thread overview]
Message-ID: <877dxsw1eu.fsf@mid.deneb.enyo.de> (raw)
In-Reply-To: <7bd30e6c-7ae8-4c03-a818-6309351b3df9@email.android.com> (nmanthey's message of "Mon, 04 May 2020 09:27:14 +0200")
* nmanthey:
>> Still, it is easier to change a run-time kernel setting than to
>> upgrade glibc.
>
> That is something I do not have control over. Hence, I would offer
> the ability to running processes. Statically linked binaries could
> come with their own modified glibc; and for such a purpose these
> patches are valuable. There might also be scenarios where not all
> processes should get huge pages, so being able to enable this on a
> per-process level looks reasonable to me.
I'm not sure if it is a good idea to work around kernel problems in
glibc just because it happens to work in your setup, given your local
constraints.
It may be time for per-namespace/cgroups settings for hugepages,
though.
>> I suggest to repeat the experiment with "always". There is a reason
>> why this setting exists. The results presented so far are incomplete.
>>
>> The paper doesn't provide details on the NUMA configuration of the
>> test system, so one has to wonder if there are any surprises there as
>> well.
>
> I agree, and I can look up the numa configuration. Even with less
> performance increase, as a unprivileged user, I would like to enable
> THP.
>
> Do you have a specific ask of what else should be measured?
I'm not a virtual memory management or performance expert. I will ask
around.
The immediate problem I see is that requesting transparent hugepages
in this way forces the kernel to perform more aggressive
defragmentation, which can hurt overall system performance if
processes are shortlived.
next parent reply other threads:[~2020-05-04 8:27 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <7bd30e6c-7ae8-4c03-a818-6309351b3df9@email.android.com>
2020-05-04 8:27 ` Florian Weimer [this message]
2020-05-04 13:31 ` Adhemerval Zanella
2020-09-23 22:12 ` [[PATCH] v2 0/1] " Norbert Manthey
2020-09-23 22:13 ` [[PATCH] v2 1/1] malloc: support transparent huge pages Norbert Manthey
2020-05-03 22:07 [[RFC][PATCH] v1 0/2] malloc/realloc with transparent huge page support Norbert Manthey
2020-05-03 22:15 ` H.J. Lu
2020-05-04 4:36 ` Florian Weimer
2020-05-04 6:58 ` Norbert Manthey
2020-05-04 7:07 ` Florian Weimer
2020-05-04 21:07 ` DJ Delorie
2020-05-04 21:44 ` Norbert Manthey
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=877dxsw1eu.fsf@mid.deneb.enyo.de \
--to=fw@deneb.enyo.de \
--cc=libc-alpha@sourceware.org \
--cc=nmanthey@conp-solutions.com \
--cc=siddhesh@sourceware.org \
/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 read-only IMAP folder(s) and NNTP newsgroup(s).