From: Johannes Weiner <hannes@cmpxchg.org>
To: paul.szabo@sydney.edu.au
Cc: linux-mm@kvack.org, 695182@bugs.debian.org
Subject: Re: [RFC] Comments and questions
Date: Fri, 8 Feb 2013 01:50:11 -0500 [thread overview]
Message-ID: <20130208065011.GA7511@cmpxchg.org> (raw)
In-Reply-To: <201301210343.r0L3h0rP030204@como.maths.usyd.edu.au>
On Mon, Jan 21, 2013 at 02:43:00PM +1100, paul.szabo@sydney.edu.au wrote:
> The setting of highmem_is_dirtyable seems used only to calculate limits
> and threshholds, not used in any decisions: seems odd.
That's okay. We prefer placing page cache in highmem either way, this
flag is just about whether the amount of allowable dirty cache scales
with the amount of available lowmem or whether it scales with all of
memory, including highmem. Per default we scale with the amount of
lowmem. Because even though the user data can live in highmem, the
management of this data requires lowmem, so lowmem ends up being the
bottle neck. The flag is for people that know how to set up their
systems such as not to exhaust lowmem and avoid OOM kills.
> [Subtraction of min_free_kbytes reported previously.]
This is fixed upstream with these two commits:
commit ab8fabd46f811d5153d8a0cd2fac9a0d41fb593d
Author: Johannes Weiner <jweiner@redhat.com>
Date: Tue Jan 10 15:07:42 2012 -0800
mm: exclude reserved pages from dirtyable memory
commit c8b74c2f6604923de91f8aa6539f8bb934736754
Author: Sonny Rao <sonnyrao@chromium.org>
Date: Thu Dec 20 15:05:07 2012 -0800
mm: fix calculation of dirtyable memory
Since this was a conceptual fix and not based on actual bug reports,
it was never included in -stable kernels. Unless I am mistaken, we
still do not have an actual bug report that demands for these patches.
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
prev parent reply other threads:[~2013-02-08 6:50 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-01-21 3:43 [RFC] Comments and questions paul.szabo
2013-02-08 6:50 ` Johannes Weiner [this message]
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=20130208065011.GA7511@cmpxchg.org \
--to=hannes@cmpxchg.org \
--cc=695182@bugs.debian.org \
--cc=linux-mm@kvack.org \
--cc=paul.szabo@sydney.edu.au \
/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).