From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757642Ab1KRPHq (ORCPT ); Fri, 18 Nov 2011 10:07:46 -0500 Received: from cantor2.suse.de ([195.135.220.15]:59636 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754933Ab1KRPHp (ORCPT ); Fri, 18 Nov 2011 10:07:45 -0500 Date: Fri, 18 Nov 2011 16:07:42 +0100 From: Michal Hocko To: Hillf Danton Cc: Andrea Arcangeli , Andrew Morton , Johannes Weiner , linux-mm@kvack.org, LKML Subject: Re: [PATCH] hugetlb: detect race if fail to COW Message-ID: <20111118150742.GA23223@tiehlicka.suse.cz> References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri 18-11-11 22:04:37, Hillf Danton wrote: > In the error path that we fail to allocate new huge page, before try again, we > have to check race since page_table_lock is re-acquired. I do not think we can race here because we are serialized by hugetlb_instantiation_mutex AFAIU. Without this lock, however, we could fall into avoidcopy and shortcut despite the fact that other thread has already did the job. The mutex usage is not obvious in hugetlb_cow so maybe we want to be explicit about it (either a comment or do the recheck). > > If racing, our job is done. > > Signed-off-by: Hillf Danton > --- > > --- a/mm/hugetlb.c Fri Nov 18 21:38:30 2011 > +++ b/mm/hugetlb.c Fri Nov 18 21:48:15 2011 > @@ -2407,7 +2407,14 @@ retry_avoidcopy: > BUG_ON(page_count(old_page) != 1); > BUG_ON(huge_pte_none(pte)); > spin_lock(&mm->page_table_lock); > - goto retry_avoidcopy; > + ptep = huge_pte_offset(mm, address & huge_page_mask(h)); > + if (likely(pte_same(huge_ptep_get(ptep), pte))) > + goto retry_avoidcopy; > + /* > + * race occurs while re-acquiring page_table_lock, and > + * our job is done. > + */ > + return 0; > } > WARN_ON_ONCE(1); > } > > -- > 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 -- Michal Hocko SUSE Labs SUSE LINUX s.r.o. Lihovarska 1060/12 190 00 Praha 9 Czech Republic