From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id BFED3C2D0BF for ; Fri, 6 Dec 2019 01:48:36 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 8E6412464E for ; Fri, 6 Dec 2019 01:48:36 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 8E6412464E Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=owner-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix) id 333246B134A; Thu, 5 Dec 2019 20:48:36 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 2E2A56B134B; Thu, 5 Dec 2019 20:48:36 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 220CB6B134C; Thu, 5 Dec 2019 20:48:36 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0059.hostedemail.com [216.40.44.59]) by kanga.kvack.org (Postfix) with ESMTP id 0C7166B134A for ; Thu, 5 Dec 2019 20:48:36 -0500 (EST) Received: from smtpin27.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay03.hostedemail.com (Postfix) with SMTP id B313F8017B12 for ; Fri, 6 Dec 2019 01:48:35 +0000 (UTC) X-FDA: 76233032190.27.sleep62_852a1311a0d16 X-HE-Tag: sleep62_852a1311a0d16 X-Filterd-Recvd-Size: 4358 Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by imf19.hostedemail.com (Postfix) with ESMTP for ; Fri, 6 Dec 2019 01:48:34 +0000 (UTC) X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 05 Dec 2019 17:48:33 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.69,282,1571727600"; d="scan'208";a="205970739" Received: from richard.sh.intel.com (HELO localhost) ([10.239.159.54]) by orsmga008.jf.intel.com with ESMTP; 05 Dec 2019 17:48:31 -0800 Date: Fri, 6 Dec 2019 09:48:25 +0800 From: Wei Yang To: David Hildenbrand Cc: Wei Yang , Wei Yang , n-horiguchi@ah.jp.nec.com, akpm@linux-foundation.org, linux-mm@kvack.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/2] mm/memory-failure.c: not necessary to recalculate hpage Message-ID: <20191206014825.GA3846@richard> Reply-To: Wei Yang References: <20191118082003.26240-1-richardw.yang@linux.intel.com> <20191118082003.26240-2-richardw.yang@linux.intel.com> <20191202222827.isaelnqmuyn7zrns@master> <37eedde2-05ab-e42e-7bcd-09090b090366@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <37eedde2-05ab-e42e-7bcd-09090b090366@redhat.com> User-Agent: Mutt/1.9.4 (2018-02-28) X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: On Thu, Dec 05, 2019 at 04:06:20PM +0100, David Hildenbrand wrote: >On 02.12.19 23:28, Wei Yang wrote: >> On Wed, Nov 20, 2019 at 04:07:38PM +0100, David Hildenbrand wrote: >>> On 18.11.19 09:20, Wei Yang wrote: >>>> hpage is not changed. >>>> >>>> Signed-off-by: Wei Yang >>>> --- >>>> mm/memory-failure.c | 1 - >>>> 1 file changed, 1 deletion(-) >>>> >>>> diff --git a/mm/memory-failure.c b/mm/memory-failure.c >>>> index 392ac277b17d..9784f4339ae7 100644 >>>> --- a/mm/memory-failure.c >>>> +++ b/mm/memory-failure.c >>>> @@ -1319,7 +1319,6 @@ int memory_failure(unsigned long pfn, int flags) >>>> } >>>> unlock_page(p); >>>> VM_BUG_ON_PAGE(!page_count(p), p); >>>> - hpage = compound_head(p); >>>> } >>>> /* >>>> >>> >>> I am *absolutely* no transparent huge page expert (sorry :) ), but won't the >>> split_huge_page(p) eventually split the compound page, such that >>> compound_head(p) will return something else after that call? >>> >> >> Hi, David >> >> Took sometime to look into the code and re-think about it. Found maybe we can >> simplify this in another way. >> >> First, code touches here means split_huge_page() succeeds and "p" is now a PTE >> page. So compound_head(p) == p. > >While this would also be my intuition, I can't state that this is >guaranteed to be the case (IOW, I did not check the code/documentation) :) > If my understanding is correct, split_huge_page() succeeds the THP would be tear down to normal page. >> >> Then let's look at who will use hpage in the following function. There are two >> uses in current upstream: >> >> * page_flags calculation >> * hwpoison_user_mappings() >> >> The first one would be removed in next patch since PageHuge is handled at the >> beginning. >> >> And in the second place, comment says if split succeeds, hpage points to page >> "p". >> >> After all, we don't need to re-calculate hpage after split, and just replace >> hpage in hwpoison_user_mappings() with p is enough. > >That assumption would only be true in case all compound pages at this >point are transparent huge pages, no? AFAIK that is not necessarily >true. Or am I missing something? > Function hwpoison_user_mappings() just handle user space mapping. If my understanding is correct, we just have three type of pages would be used in user space mapping: * normal page * THP * hugetlb Since THP would be split or already returned and hugetlb is handled in another branch, this means for other pages hwpoison_user_mappings() would just return true. > >-- >Thanks, > >David / dhildenb -- Wei Yang Help you, Help me