Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757553Ab2EaMLi (ORCPT ); Thu, 31 May 2012 08:11:38 -0400 Received: from mx1.redhat.com ([209.132.183.28]:1026 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755897Ab2EaMLh (ORCPT ); Thu, 31 May 2012 08:11:37 -0400 Message-ID: <1338466281.19369.44.camel@cr0> Subject: Re: [RFC Patch] fs: implement per-file drop caches From: Cong Wang To: KOSAKI Motohiro Cc: =?ISO-8859-1?Q?P=E1draig?= Brady , linux-kernel@vger.kernel.org, Andrew Morton , Cong Wang , Alexander Viro , Matthew Wilcox , linux-fsdevel@vger.kernel.org, linux-mm@kvack.org Date: Thu, 31 May 2012 20:11:21 +0800 In-Reply-To: <4FC70FFE.50809@gmail.com> References: <1338385120-14519-1-git-send-email-amwang@redhat.com> <4FC6393B.7090105@draigBrady.com> <1338445233.19369.21.camel@cr0> <4FC70FFE.50809@gmail.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8bit Mime-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2107 Lines: 48 On Thu, 2012-05-31 at 02:30 -0400, KOSAKI Motohiro wrote: > (5/31/12 2:20 AM), Cong Wang wrote: > > On Wed, 2012-05-30 at 16:14 +0100, Pádraig Brady wrote: > >> On 05/30/2012 02:38 PM, Cong Wang wrote: > >>> This is a draft patch of implementing per-file drop caches. > >>> > >>> It introduces a new fcntl command F_DROP_CACHES to drop > >>> file caches of a specific file. The reason is that currently > >>> we only have a system-wide drop caches interface, it could > >>> cause system-wide performance down if we drop all page caches > >>> when we actually want to drop the caches of some huge file. > >> > >> This is useful functionality. > >> Though isn't it already provided with POSIX_FADV_DONTNEED? > > > > Thanks for teaching this! > > > > However, from the source code of madvise_dontneed() it looks like it is > > using a totally different way to drop page caches, that is to invalidate > > the page mapping, and trigger a re-mapping of the file pages after a > > page fault. So, yeah, this could probably drop the page caches too (I am > > not so sure, haven't checked the code in details), but with my patch, it > > flushes the page caches directly, what's more, it can also prune > > dcache/icache of the file. > > madvise should work. I don't think we need duplicate interface. Moreomover > madvise(2) is cleaner than fcntl(2). > I think madvise(DONTNEED) attacks the problem in a different approach, it munmaps the file mapping and by the way drops the page caches, my approach is to drop the page caches directly similar to what sysctl drop_caches. What about private file mapping? Could madvise(DONTNEED) drop the page caches too even when the other process is doing the same private file mapping? At least my patch could do this. I am not sure if fcntl() is a good interface either, this is why the patch is marked as RFC. :-D Thanks! -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/