[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <1403677528.7903.103.camel@sauron.fi.intel.com>
Date: Wed, 25 Jun 2014 09:25:28 +0300
From: Artem Bityutskiy <dedekind1@...il.com>
To: David Rientjes <rientjes@...gle.com>
Cc: Maksym Planeta <mcsim.planeta@...il.com>,
Alexander Viro <viro@...iv.linux.org.uk>,
Thomas Knauth <thomas.knauth@....de>,
linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] sysctl: Add a feature to drop caches selectively
On Tue, 2014-06-24 at 14:59 -0700, David Rientjes wrote:
> On Tue, 24 Jun 2014, Maksym Planeta wrote:
>
> > To clean the page cache one can use /proc/sys/vm/drop_caches. But this
> > drops the whole page cache. In contrast to that sdrop_caches enables
> > ability to drop the page cache selectively by path string.
> >
> > Suggested-by: Thomas Knauth <thomas.knauth@....de>
> > Signed-off-by: Maksym Planeta <mcsim.planeta@...il.com>
>
> Could you include some information in the commit message about why this is
> useful? Specifically, why you want to drop pagecache only from a specific
> path.
>
> The name of the sysctl is also quite non-descriptive.
Plus some explanations WRT why proc-based interface and what would be
the alternatives, what if tomorrow we want to extend the functionality
and drop caches only for certain file range, is this only for regular
files or also for directories, why posix_fadvice(DONTNEED) is not
sufficient.
--
Best Regards,
Artem Bityutskiy
--
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