From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx140.postini.com [74.125.245.140]) by kanga.kvack.org (Postfix) with SMTP id 5029C6B004D for ; Wed, 9 Jan 2013 06:38:25 -0500 (EST) Received: by mail-ob0-f182.google.com with SMTP id 16so2060072obc.27 for ; Wed, 09 Jan 2013 03:38:24 -0800 (PST) MIME-Version: 1.0 In-Reply-To: References: <20130108163141.GA27555@shutemov.name> <20130108173747.GF9163@redhat.com> <20130108180346.GH9163@redhat.com> Date: Wed, 9 Jan 2013 19:38:24 +0800 Message-ID: Subject: Re: oops in copy_page_rep() From: Hillf Danton Content-Type: text/plain; charset=UTF-8 Sender: owner-linux-mm@kvack.org List-ID: To: Linus Torvalds Cc: Andrea Arcangeli , "Kirill A. Shutemov" , Hugh Dickins , Dave Jones , Linux Kernel , Andrew Morton , Mel Gorman , Linux-MM , Rik van Riel On Wed, Jan 9, 2013 at 2:21 AM, Linus Torvalds wrote: > On Tue, Jan 8, 2013 at 10:03 AM, Andrea Arcangeli wrote: >> >> It looks very fine to me, but I suggest to move it above the >> pmd_numa() check because of the newly introduced >> migrate_misplaced_transhuge_page method relying on pmd_same too. > > Hmm. If we need it there, then we need to fix the *later* case of > pmd_numa() too: > > if (pmd_numa(*pmd)) > return do_pmd_numa_page(mm, vma, address, pmd); > > Also, and more fundamentally, since do_pmd_numa_page() doesn't take > the orig_pmd thing as an argument (and re-check it under the > page-table lock), testing pmd_trans_splitting() on it is pointless, > since it can change later. > A splitting pmd has to be huge first, and we do handle huge pmd already in the up dozen of lines. That said, we can igore splitting check in this case, Sir. Hillf > So no, moving the check up does *not* make sense, at least not without > other changes. Because if I read things right, pmd_trans_splitting() > really has to be done with the page-table lock protection (where "with > page-table lock protection" does *not* mean that it has to be done > under the page table lock, but if it is done outside, then the pmd > entry has to be re-verified after getting the lock - which both > do_huge_pmd_wp_page() and huge_pmd_set_accessed() correctly do). > > Comments? > > Linus -- 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