Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756960AbaFYNm0 (ORCPT ); Wed, 25 Jun 2014 09:42:26 -0400 Received: from mga09.intel.com ([134.134.136.24]:61571 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756638AbaFYNmY (ORCPT ); Wed, 25 Jun 2014 09:42:24 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.01,546,1400050800"; d="scan'208";a="563219055" Message-ID: <1403703728.20275.33.camel@sauron.fi.intel.com> Subject: Re: [PATCH] sysctl: Add a feature to drop caches selectively From: Artem Bityutskiy Reply-To: dedekind1@gmail.com To: Thomas Knauth Cc: David Rientjes , Maksym Planeta , Alexander Viro , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org Date: Wed, 25 Jun 2014 16:42:08 +0300 In-Reply-To: References: <1403626213-7691-1-git-send-email-mcsim.planeta@gmail.com> <1403677528.7903.103.camel@sauron.fi.intel.com> <1403690184.20275.11.camel@sauron.fi.intel.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.10.4 (3.10.4-2.fc20) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2014-06-25 at 15:23 +0200, Thomas Knauth wrote: > On Wed, Jun 25, 2014 at 11:56 AM, Artem Bityutskiy wrote: > > Thanks for the answer, although you forgot to comment on the question > > about possibly extending the new interface to work with file ranges in > > the future. For example, I have a 2 TiB file, and I am only interested > > in dropping caches for the first couple of gigabytes. Would I extend > > your interface, or would I come up with another one? > > Ah, didn't quite understand what was meant with file ranges. Again, we > had not considered this so far. I guess you could make a distinction > between directories and files here. If the path points to a file, you > can have an optional argument indicating the range of bytes you would > like to drop. Something like > > echo "my-file 0-1000,8000-1000" > /proc/sys/vm/sdrop_cache > > If this is desirable, we can add it to the patch. With a binary interface like an ioctl I can see how you could have extra unused fields which you can ignore now and let people start adding extra options like the range in the future. With this kind of interface I am not sure how to do this. Other questions I'd ask would be - how about the access control model? Will only root be able to drop caches? Why can't I drop caches for my own file? I did not put much thinking into this, but it looks like ioctl could be a better interface for the task you are trying to solve... Sorry if I am a bit vague, I am mostly trying to make you guys give this more thoughts, and come up with a deeper analysis. Interfaces are very important to get right, or as right as possible... -- Best Regards, Artem Bityutskiy -- 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/