From mboxrd@z Thu Jan 1 00:00:00 1970 From: Naoya Horiguchi Subject: Re: [PATCH v2 2/4] mm: introduce fincore() Date: Fri, 4 Jul 2014 12:31:07 -0400 Message-ID: <20140704163107.GA17877@nhori> References: <1404424335-30128-1-git-send-email-n-horiguchi@ah.jp.nec.com> <1404424335-30128-3-git-send-email-n-horiguchi@ah.jp.nec.com> <20140704101230.GA24688@infradead.org> <5816450.BPnLjGgtl5@obelix> Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: quoted-printable Return-path: Content-Disposition: inline In-Reply-To: <5816450.BPnLjGgtl5@obelix> Sender: owner-linux-mm@kvack.org To: =?iso-8859-1?Q?C=E9dric?= Villemain Cc: Christoph Hellwig , Andrew Morton , Konstantin Khlebnikov , Wu Fengguang , Arnaldo Carvalho de Melo , Borislav Petkov , "Kirill A. Shutemov" , Johannes Weiner , Rusty Russell , David Miller , Andres Freund , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Dave Hansen , Michael Kerrisk , Linux API , Naoya Horiguchi List-Id: linux-api@vger.kernel.org On Fri, Jul 04, 2014 at 05:15:59PM +0200, C=E9dric Villemain wrote: > Le vendredi 4 juillet 2014 03:12:30 Christoph Hellwig a =E9crit : > > On Thu, Jul 03, 2014 at 05:52:13PM -0400, Naoya Horiguchi wrote: > > > This patch provides a new system call fincore(2), which provides > > > mincore()- like information, i.e. page residency of a given file. > > > But unlike mincore(), fincore() has a mode flag which allows us to > > > extract detailed information about page cache like pfn and page > > > flag. This kind of information is very helpful, for example when > > > applications want to know the file cache status to control the IO > > > on their own way. > >=20 > > It's still a nasty multiplexer for multiple different reporting > > formats in a single system call. How about your really just do a > > fincore that mirrors mincore instead of piggybacking exports of > > various internal flags (tags and page flags onto it. We can do it in mincore-compatible way with FINCORE_BMAP mode. If you choose it, you don't care about any details about other modes. I don't make no default mode, but if we have a good reason, I'm OK to set FINCORE_BMAP as default mode. > The fincore =E0 la mincore got some arguments against it too. It seems = this=20 > implementations try (I've not tested nor have a close look yet) to=20 > answer both concerns : have details and also possible to have=20 > aggregation function not too expansive. Correct, that's the motivation of this non-trivial interface. This could finally obsoletes messy /proc/kpage{flags,count} and/or /proc/pid/pagemap kind of things, and we will not have to collect information over all these interfaces (so that's less expensive.) Thanks, Naoya Horiguchi -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org