From: David Howells <dhowells@redhat.com>
To: Andrew Morton <akpm@linux-foundation.org>
Cc: dhowells@redhat.com, torvalds@linux-foundation.org,
npiggin@suse.de, gerg@snapgear.com, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 3/3] NOMMU: Make the initial mmap allocation excess behaviour Kconfig configurable
Date: Wed, 06 May 2009 12:42:41 +0100 [thread overview]
Message-ID: <23252.1241610161@redhat.com> (raw)
In-Reply-To: <20090505151520.070c84c1.akpm@linux-foundation.org>
Andrew Morton <akpm@linux-foundation.org> wrote:
> Nasty problem.
Yes. That's part of the fun of the NOMMU world. It has many of the same
problems as the MMU world - just more exaggerated.
> > -int sysctl_nr_trim_pages = 1; /* page trimming behaviour */
> > +int sysctl_nr_trim_pages = CONFIG_NOMMU_INITIAL_TRIM_EXCESS;
> > int heap_stack_gap = 0;
> >
>
> But there's a risk of -ENOMEM regression on other system here?
There shouldn't be (assuming you mean with this patch), the default is the
same as the original value.
> It's unlikely to be a huge problem for real-world embedded developers,
> as long as they know about this change. And because you set the
> Kconfig default to "no change" then I guess they'll be none the wiser.
>
> I think that patches 2 and 3 (and #1 unless I reorder and redo things)
> are 2.6.30 material. Agree?
Assuming you mean go in before 2.6.30 is cut, then yes. If you want, I can
reorder the patches to put #1 last.
David
prev parent reply other threads:[~2009-05-06 11:44 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-05 21:26 [PATCH 1/3] Use roundown_pow_of_two() in zone_batchsize() David Howells
2009-05-05 21:26 ` [PATCH 2/3] NOMMU: Clamp zone_batchsize() to 0 under NOMMU conditions David Howells
2009-05-05 21:26 ` [PATCH 3/3] NOMMU: Make the initial mmap allocation excess behaviour Kconfig configurable David Howells
2009-05-05 22:15 ` Andrew Morton
2009-05-06 0:09 ` Greg Ungerer
2009-05-06 11:42 ` David Howells [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=23252.1241610161@redhat.com \
--to=dhowells@redhat.com \
--cc=akpm@linux-foundation.org \
--cc=gerg@snapgear.com \
--cc=linux-kernel@vger.kernel.org \
--cc=npiggin@suse.de \
--cc=torvalds@linux-foundation.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.