From: Rik van Riel <riel@redhat.com>
To: Dave Chinner <david@fromorbit.com>, Daniel Walker <danielwa@cisco.com>
Cc: Alexander Viro <viro@zeniv.linux.org.uk>,
Khalid Mughal <khalidm@cisco.com>,
xe-kernel@external.cisco.com, dave.hansen@intel.com,
hannes@cmpxchg.org, Jonathan Corbet <corbet@lwn.net>,
linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-fsdevel@vger.kernel.org, linux-mm@kvack.org,
"Nag Avadhanam (nag)" <nag@cisco.com>
Subject: Re: [PATCH] kernel: fs: drop_caches: add dds drop_caches_count
Date: Tue, 16 Feb 2016 11:12:23 -0500 [thread overview]
Message-ID: <1455639143.15821.21.camel@redhat.com> (raw)
In-Reply-To: <20160216052852.GW19486@dastard>
[-- Attachment #1: Type: text/plain, Size: 1533 bytes --]
On Tue, 2016-02-16 at 16:28 +1100, Dave Chinner wrote:
> On Mon, Feb 15, 2016 at 03:52:31PM -0800, Daniel Walker wrote:
> > On 02/15/2016 03:05 PM, Dave Chinner wrote:
> > >
> > > As for a replacement, looking at what pages you consider
> > > "droppable"
> > > is really only file pages that are not under dirty or under
> > > writeback. i.e. from /proc/meminfo:
> > >
> > > Active(file): 220128 kB
> > > Inactive(file): 60232 kB
> > > Dirty: 0 kB
> > > Writeback: 0 kB
> > >
> > > i.e. reclaimable file cache = Active + inactive - dirty -
> > > writeback.
> .....
>
> > As to his other suggestion of estimating the droppable cache, I
> > have considered it but found it unusable. The problem is the
> > inactive file pages count a whole lot pages more than the
> > droppable pages.
>
> inactive file pages are supposed to be exactly that - inactive. i.e.
> the have not been referenced recently, and are unlikely to be dirty.
> They should be immediately reclaimable.
Inactive file pages can still be mapped by
processes.
The reason we do not unmap file pages when
moving them to the inactive list is that
some workloads fill essentially all of memory
with mmapped file pages.
Given that the inactive list is generally a
considerable fraction of file memory, unmapping
pages that get deactivated could create a lot
of churn and unnecessary page faults for that
kind of workload.
--
All rights reversed
[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
prev parent reply other threads:[~2016-02-16 16:12 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-12 20:14 [PATCH] kernel: fs: drop_caches: add dds drop_caches_count Daniel Walker
2016-02-12 20:14 ` Daniel Walker
2016-02-14 21:18 ` Dave Chinner
2016-02-14 21:18 ` Dave Chinner
2016-02-15 18:19 ` Daniel Walker
2016-02-15 18:19 ` Daniel Walker
2016-02-15 23:05 ` Dave Chinner
2016-02-15 23:05 ` Dave Chinner
2016-02-15 23:52 ` Daniel Walker
2016-02-15 23:52 ` Daniel Walker
2016-02-15 23:52 ` Daniel Walker
2016-02-16 0:45 ` Theodore Ts'o
2016-02-16 0:45 ` Theodore Ts'o
2016-02-16 0:45 ` Theodore Ts'o
2016-02-16 2:58 ` Nag Avadhanam (nag)
2016-02-16 2:58 ` Nag Avadhanam (nag)
2016-02-16 5:38 ` Dave Chinner
2016-02-16 5:38 ` Dave Chinner
2016-02-16 7:14 ` Nag Avadhanam
2016-02-16 7:14 ` Nag Avadhanam
2016-02-16 8:35 ` Dave Chinner
2016-02-16 8:35 ` Dave Chinner
2016-02-16 8:43 ` Vladimir Davydov
2016-02-16 8:43 ` Vladimir Davydov
2016-02-16 18:37 ` Nag Avadhanam
2016-02-16 18:37 ` Nag Avadhanam
2016-02-16 5:28 ` Dave Chinner
2016-02-16 5:28 ` Dave Chinner
2016-02-16 5:28 ` Dave Chinner
2016-02-16 5:57 ` Nag Avadhanam
2016-02-16 5:57 ` Nag Avadhanam
2016-02-16 8:22 ` Dave Chinner
2016-02-16 8:22 ` Dave Chinner
2016-02-16 16:12 ` Rik van Riel [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=1455639143.15821.21.camel@redhat.com \
--to=riel@redhat.com \
--cc=corbet@lwn.net \
--cc=danielwa@cisco.com \
--cc=dave.hansen@intel.com \
--cc=david@fromorbit.com \
--cc=hannes@cmpxchg.org \
--cc=khalidm@cisco.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=nag@cisco.com \
--cc=viro@zeniv.linux.org.uk \
--cc=xe-kernel@external.cisco.com \
/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.