From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Weiner Subject: Re: [PATCH 12/16] mm: shmem: Avoid atomic operation during shmem_getpage_gfp Date: Fri, 18 Apr 2014 14:13:00 -0400 Message-ID: <20140418181300.GG29210@cmpxchg.org> References: <1397832643-14275-1-git-send-email-mgorman@suse.de> <1397832643-14275-13-git-send-email-mgorman@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Linux-MM , Linux-FSDevel To: Mel Gorman Return-path: Content-Disposition: inline In-Reply-To: <1397832643-14275-13-git-send-email-mgorman@suse.de> Sender: owner-linux-mm@kvack.org List-Id: linux-fsdevel.vger.kernel.org On Fri, Apr 18, 2014 at 03:50:39PM +0100, Mel Gorman wrote: > shmem_getpage_gfp uses an atomic operation to set the SwapBacked field > before it's even added to the LRU or visible. This is unnecessary as what > could it possible race against? Use an unlocked variant. > > Signed-off-by: Mel Gorman Acked-by: Johannes Weiner -- 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