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=-0.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS autolearn=ham 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 E1A3FC677FC for ; Thu, 11 Oct 2018 17:38:59 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id A52BA2085B for ; Thu, 11 Oct 2018 17:38:59 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A52BA2085B Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728140AbeJLBHL (ORCPT ); Thu, 11 Oct 2018 21:07:11 -0400 Received: from mga17.intel.com ([192.55.52.151]:41981 "EHLO mga17.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726071AbeJLBHL (ORCPT ); Thu, 11 Oct 2018 21:07:11 -0400 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga107.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Oct 2018 10:38:58 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.54,369,1534834800"; d="scan'208";a="99455331" Received: from ahduyck-mobl.amr.corp.intel.com (HELO [10.7.198.157]) ([10.7.198.157]) by orsmga002.jf.intel.com with ESMTP; 11 Oct 2018 10:38:39 -0700 Subject: Re: [PATCH v5 4/4] mm: Defer ZONE_DEVICE page initialization to the point where we init pgmap To: Michal Hocko Cc: Dan Williams , Linux MM , Andrew Morton , Linux Kernel Mailing List , linux-nvdimm , Pasha Tatashin , Dave Hansen , =?UTF-8?B?SsOpcsO0bWUgR2xpc3Nl?= , rppt@linux.vnet.ibm.com, Ingo Molnar , "Kirill A. Shutemov" , yi.z.zhang@linux.intel.com References: <20180925200551.3576.18755.stgit@localhost.localdomain> <20180925202053.3576.66039.stgit@localhost.localdomain> <20181009170051.GA40606@tiger-server> <25092df0-b7b4-d456-8409-9c004cb6e422@linux.intel.com> <20181010095838.GG5873@dhcp22.suse.cz> <20181010172451.GK5873@dhcp22.suse.cz> <98c35e19-13b9-0913-87d9-b3f1ab738b61@linux.intel.com> <20181010185242.GP5873@dhcp22.suse.cz> <20181011085509.GS5873@dhcp22.suse.cz> From: Alexander Duyck Message-ID: <6f32f23c-c21c-9d42-7dda-a1d18613cd3c@linux.intel.com> Date: Thu, 11 Oct 2018 10:38:39 -0700 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.2.1 MIME-Version: 1.0 In-Reply-To: <20181011085509.GS5873@dhcp22.suse.cz> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 10/11/2018 1:55 AM, Michal Hocko wrote: > On Wed 10-10-18 20:52:42, Michal Hocko wrote: > [...] >> My recollection was that we do clear the reserved bit in >> move_pfn_range_to_zone and we indeed do in __init_single_page. But then >> we set the bit back right afterwards. This seems to be the case since >> d0dc12e86b319 which reorganized the code. I have to study this some more >> obviously. > > so my recollection was wrong and d0dc12e86b319 hasn't really changed > much because __init_single_page wouldn't zero out the struct page for > the hotplug contex. A comment in move_pfn_range_to_zone explains that we > want the reserved bit because pfn walkers already do see the pfn range > and the page is not fully associated with the zone until it is onlined. > > I am thinking that we might be overzealous here. With the full state > initialized we shouldn't actually care. pfn_to_online_page should return > NULL regardless of the reserved bit and normal pfn walkers shouldn't > touch pages they do not recognize and a plain page with ref. count 1 > doesn't tell much to anybody. So I _suspect_ that we can simply drop the > reserved bit setting here. So this has me a bit hesitant to want to just drop the bit entirely. If nothing else I think I may wan to make that a patch onto itself so that if we aren't going to set it we just drop it there. That way if it does cause issues we can bisect it to that patch and pinpoint the cause. > Regarding the post initialization required by devm_memremap_pages and > potentially others. Can we update the altmap which is already a way how > to get alternative struct pages by a constructor which we could call > from memmap_init_zone and do the post initialization? This would reduce > the additional loop in the caller while it would still fit the overall > design of the altmap and the core hotplug doesn't have to know anything > about DAX or whatever needs a special treatment. > > Does that make any sense? I think the only thing that is currently using the altmap is the ZONE_DEVICE memory init. Specifically I think it is only really used by the devm_memremap_pages version of things, and then only under certain circumstances. Also the HMM driver doesn't pass an altmap. What we would really need is a non-ZONE_DEVICE users of the altmap to really justify sticking with that as the preferred argument to pass. For those two functions it currently makes much more sense to pass the dev_pagemap pointer and then reference the altmap from there. Otherwise we are likely starting to look at something that would be more of a dirty hack where we are passing a unused altmap in order to get to the dev_pagemap so that we could populate the page.