[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.00.0811101410170.2108@chino.kir.corp.google.com>
Date: Mon, 10 Nov 2008 14:15:29 -0800 (PST)
From: David Rientjes <rientjes@...gle.com>
To: Andrea Righi <righi.andrea@...il.com>
cc: Andrew Morton <akpm@...ux-foundation.org>,
kamezawa.hiroyu@...fujitsu.com, balbir@...ux.vnet.ibm.com,
mrubin@...gle.com, menage@...gle.com, dave@...ux.vnet.ibm.com,
chlunde@...g.uio.no, dpshah@...gle.com, eric.rannaud@...il.com,
fernando@....ntt.co.jp, agk@...rceware.org, m.innocenti@...eca.it,
s-uchida@...jp.nec.com, ryov@...inux.co.jp, matt@...ehost.com,
dradford@...ehost.com, kosaki.motohiro@...fujitsu.com,
linux-mm@...ck.org, linux-kernel@...r.kernel.org,
containers@...ts.osdl.org
Subject: Re: [PATCH -mm] mm: fine-grained dirty_ratio_pcm and dirty_background_ratio_pcm
(v2)
On Mon, 10 Nov 2008, Andrea Righi wrote:
> The KB limit is a static value, the other depends on the dirtyable
> memory. If we want to preserve the same behaviour we should do the
> following:
>
> - when dirty_ratio changes to x:
> dirty_amount_in_bytes = x * dirtyable_memory / 100.
>
> - when dirty_amount_in_bytes changes to x:
> dirty_ratio = x / dirtyable_memory * 100
>
I think the idea is for a dynamic dirty_ratio based on a static value
dirty_amount_in_bytes:
dirtyable_memory = determine_dirtyable_memory() * PAGE_SIZE;
dirty_ratio = dirty_amount_in_bytes / dirtyable_memory;
> But anytime the dirtyable memory changes (as well as the total memory in
> the system) we should update both values accordingly to preserve the
> coherency between them.
>
Only dirty_ratio is actually updated if dirty_amount_in_bytes is static.
This allows you to control how many pages are NR_FILE_DIRTY or
NR_UNSTABLE_NFS and gives you the granularity that you want with
dirty_ratio_pcm, but on a byte scale instead of percent.
It's also a clean interface:
echo 200M > /proc/sys/vm/dirty_ratio_bytes
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists