From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx158.postini.com [74.125.245.158]) by kanga.kvack.org (Postfix) with SMTP id EB8FB6B004A for ; Fri, 6 Apr 2012 19:35:46 -0400 (EDT) Received: by iajr24 with SMTP id r24so4990361iaj.14 for ; Fri, 06 Apr 2012 16:35:46 -0700 (PDT) Date: Fri, 6 Apr 2012 16:35:29 -0700 (PDT) From: Hugh Dickins Subject: Re: [PATCH v2] hugetlb: fix race condition in hugetlb_fault() In-Reply-To: <20120406162618.3307a9bd.akpm@linux-foundation.org> Message-ID: References: <201203302018.q2UKIFH5020745@farm-0012.internal.tilera.com> <201203311339.q2VDdJMD006254@farm-0012.internal.tilera.com> <4F7887A5.3060700@tilera.com> <20120406152305.59408e35.akpm@linux-foundation.org> <20120406162618.3307a9bd.akpm@linux-foundation.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: owner-linux-mm@kvack.org List-ID: To: Andrew Morton Cc: Chris Metcalf , Hillf Danton , linux-kernel@vger.kernel.org, linux-mm@kvack.org, Michal Hocko , KAMEZAWA Hiroyuki On Fri, 6 Apr 2012, Andrew Morton wrote: > On Fri, 6 Apr 2012 16:10:13 -0700 (PDT) > Hugh Dickins wrote: > > > > The resulting patch is okay; but let's reassure Chris that his > > original patch was better, before he conceded to make the get_page > > and put_page unconditional, and added unnecessary detail of the race. > > > > Yes, the v1 patch was better. No reason was given for changing it? I think Chris was aiming to be a model citizen, and followed review suggestions that he would actually have done better to resist. Hugh -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org