From: Rik van Riel <riel@redhat.com> To: Johannes Weiner <hannes@cmpxchg.org>, Andrew Morton <akpm@linux-foundation.org> Cc: Mel Gorman <mgorman@suse.de>, Michal Hocko <mhocko@suse.com>, Vladimir Davydov <vdavydov.dev@gmail.com>, linux-mm@kvack.org, cgroups@vger.kernel.org, linux-kernel@vger.kernel.org, kernel-team@fb.com Subject: Re: [PATCH] mm: vmscan: fix IO/refault regression in cache workingset transition Date: Wed, 05 Apr 2017 18:11:04 -0400 [thread overview] Message-ID: <1491430264.16856.43.camel@redhat.com> (raw) In-Reply-To: <20170404220052.27593-1-hannes@cmpxchg.org> [-- Attachment #1: Type: text/plain, Size: 934 bytes --] On Tue, 2017-04-04 at 18:00 -0400, Johannes Weiner wrote: > + > + /* > + * When refaults are being observed, it means a new > workingset > + * is being established. Disable active list protection to > get > + * rid of the stale workingset quickly. > + */ This looks a little aggressive. What is this expected to do when you have multiple workloads sharing the same LRU, and one of the workloads is doing refaults, while the other workload is continuing to use the same working set as before? I have been trying to wrap my mind around that for the past day or so, and figure I should just ask the question :) > + if (file && actual_reclaim && lruvec->refaults != refaults) > { > + inactive_ratio = 0; > + } else { > + gb = (inactive + active) >> (30 - PAGE_SHIFT); > + if (gb) > + inactive_ratio = int_sqrt(10 * gb); > + else > + inactive_ratio = 1; > + } -- All rights reversed [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 473 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Rik van Riel <riel-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org> To: Johannes Weiner <hannes-druUgvl0LCNAfugRpC6u6w@public.gmane.org>, Andrew Morton <akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org> Cc: Mel Gorman <mgorman-l3A5Bk7waGM@public.gmane.org>, Michal Hocko <mhocko-IBi9RG/b67k@public.gmane.org>, Vladimir Davydov <vdavydov.dev-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>, linux-mm-Bw31MaZKKs3YtjvyW6yDsg@public.gmane.org, cgroups-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, kernel-team-b10kYP2dOMg@public.gmane.org Subject: Re: [PATCH] mm: vmscan: fix IO/refault regression in cache workingset transition Date: Wed, 05 Apr 2017 18:11:04 -0400 [thread overview] Message-ID: <1491430264.16856.43.camel@redhat.com> (raw) In-Reply-To: <20170404220052.27593-1-hannes-druUgvl0LCNAfugRpC6u6w@public.gmane.org> [-- Attachment #1: Type: text/plain, Size: 934 bytes --] On Tue, 2017-04-04 at 18:00 -0400, Johannes Weiner wrote: > + > + /* > + * When refaults are being observed, it means a new > workingset > + * is being established. Disable active list protection to > get > + * rid of the stale workingset quickly. > + */ This looks a little aggressive. What is this expected to do when you have multiple workloads sharing the same LRU, and one of the workloads is doing refaults, while the other workload is continuing to use the same working set as before? I have been trying to wrap my mind around that for the past day or so, and figure I should just ask the question :) > + if (file && actual_reclaim && lruvec->refaults != refaults) > { > + inactive_ratio = 0; > + } else { > + gb = (inactive + active) >> (30 - PAGE_SHIFT); > + if (gb) > + inactive_ratio = int_sqrt(10 * gb); > + else > + inactive_ratio = 1; > + } -- All rights reversed [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 473 bytes --]
next prev parent reply other threads:[~2017-04-05 22:11 UTC|newest] Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top 2017-04-04 22:00 [PATCH] mm: vmscan: fix IO/refault regression in cache workingset transition Johannes Weiner 2017-04-04 22:00 ` Johannes Weiner 2017-04-04 22:07 ` Andrew Morton 2017-04-04 22:07 ` Andrew Morton 2017-04-04 22:29 ` Johannes Weiner 2017-04-04 22:29 ` Johannes Weiner 2017-04-04 22:29 ` Johannes Weiner 2017-04-05 22:11 ` Rik van Riel [this message] 2017-04-05 22:11 ` Rik van Riel 2017-04-06 14:49 ` Johannes Weiner 2017-04-06 14:49 ` Johannes Weiner 2017-04-06 14:49 ` Johannes Weiner 2017-04-06 16:51 ` Rik van Riel 2017-04-06 16:51 ` Rik van Riel 2017-04-06 16:51 ` Rik van Riel
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=1491430264.16856.43.camel@redhat.com \ --to=riel@redhat.com \ --cc=akpm@linux-foundation.org \ --cc=cgroups@vger.kernel.org \ --cc=hannes@cmpxchg.org \ --cc=kernel-team@fb.com \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-mm@kvack.org \ --cc=mgorman@suse.de \ --cc=mhocko@suse.com \ --cc=vdavydov.dev@gmail.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: linkBe 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.