From mboxrd@z Thu Jan 1 00:00:00 1970 From: Oleg Nesterov Subject: Re: [PATCH 19/19] mm: filemap: Avoid unnecessary barries and waitqueue lookups in unlock_page fastpath Date: Tue, 13 May 2014 20:22:10 +0200 Message-ID: <20140513182210.GC12123@redhat.com> References: <1399974350-11089-1-git-send-email-mgorman@suse.de> <1399974350-11089-20-git-send-email-mgorman@suse.de> <20140513125313.GR23991@suse.de> <20140513141748.GD2485@laptop.programming.kicks-ass.net> <20140513152719.GF18164@linux.vnet.ibm.com> <20140513154435.GG2485@laptop.programming.kicks-ass.net> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "Paul E. McKenney" , Mel Gorman , Andrew Morton , Johannes Weiner , Vlastimil Babka , Jan Kara , Michal Hocko , Hugh Dickins , Dave Hansen , Linux Kernel , Linux-MM , Linux-FSDevel , Linus Torvalds , David Howells To: Peter Zijlstra Return-path: Received: from mx1.redhat.com ([209.132.183.28]:3970 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751511AbaEMSXK (ORCPT ); Tue, 13 May 2014 14:23:10 -0400 Content-Disposition: inline In-Reply-To: <20140513154435.GG2485@laptop.programming.kicks-ass.net> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On 05/13, Peter Zijlstra wrote: > > Ah, yes, so I'll defer to Oleg and Linus to explain that one. As per the > name: smp_mb__before_spinlock() should of course imply a full barrier. Oh yes, I agree, the name is confusing. At least the comment tries to explain what it does. Oleg.