From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pl1-f197.google.com (mail-pl1-f197.google.com [209.85.214.197]) by kanga.kvack.org (Postfix) with ESMTP id 3C50F8E0002 for ; Wed, 16 Jan 2019 20:26:51 -0500 (EST) Received: by mail-pl1-f197.google.com with SMTP id y2so5028215plr.8 for ; Wed, 16 Jan 2019 17:26:51 -0800 (PST) Received: from ipmail07.adl2.internode.on.net (ipmail07.adl2.internode.on.net. [150.101.137.131]) by mx.google.com with ESMTP id m20si75388pgk.323.2019.01.16.17.26.48 for ; Wed, 16 Jan 2019 17:26:49 -0800 (PST) Date: Thu, 17 Jan 2019 12:26:45 +1100 From: Dave Chinner Subject: Re: [PATCH] mm/mincore: allow for making sys_mincore() privileged Message-ID: <20190117012645.GU4205@dastard> References: <20190110122442.GA21216@nautica> <20190111020340.GM27534@dastard> <20190111040434.GN27534@dastard> <20190111073606.GP27534@dastard> <20190115234510.GA6173@dastard> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Linus Torvalds Cc: Dominique Martinet , Jiri Kosina , Matthew Wilcox , Jann Horn , Andrew Morton , Greg KH , Peter Zijlstra , Michal Hocko , Linux-MM , kernel list , Linux API On Wed, Jan 16, 2019 at 04:54:49PM +1200, Linus Torvalds wrote: > On Wed, Jan 16, 2019 at 11:45 AM Dave Chinner wrote: > > > > I'm assuming that you can invalidate the page cache reliably by a > > means that does not repeated require probing to detect invalidation > > has occurred. I've mentioned one method in this discussion > > already... > > Yes. And it was made clear to you that it was a bug in xfs dio and > what the right thing to do was. > > And you ignored that, and claimed it was a feature. Linus, either you aren't listening or you're being intentionally provocative. So, for the *third* time this thread: we can probably remove this code but first we need to be sure it doesn't cause unexpected regressions before we commit such a change. We are not cowboys who test userspace behavioural changes on users without review or discussion. Indeed, I wrote a patch to remove the invalidation /several days ago/ and put it into my test trees, and it's been there since. Just because you don't see immediate changes doesn't mean it isn't happening. > Either you care or you don't. If you don't care (and so far everything > you said seems to imply you don't), Linus, this is just a personal attack and IMO a violation of the CoC. It's straight out wrong, insulting, totally unprofessional and completely uncalled for. This is most definitely not a useful technical response to the issues I raised. i.e you cut out all the context of my response about whether "no probing necessary" page cache invalidation attacks are something we need to care about in the future. We don't need you to shout about existing "no probing necessary" page cache invalidation attacks that are already being addressed, we need to determine if it's going to be a recurring problem in future because that directly affects the mitigation strategies we can implement. -Dave. -- Dave Chinner david@fromorbit.com