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=-1.0 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no 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 8B3D6C5AE5C for ; Tue, 10 Sep 2019 20:26:29 +0000 (UTC) Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by mail.kernel.org (Postfix) with ESMTP id 3FA1B2084D for ; Tue, 10 Sep 2019 20:26:29 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 3FA1B2084D 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 7277C6B0005; Tue, 10 Sep 2019 16:26:28 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 6D7E36B0006; Tue, 10 Sep 2019 16:26:28 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 614F56B0007; Tue, 10 Sep 2019 16:26:28 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from forelay.hostedemail.com (smtprelay0120.hostedemail.com [216.40.44.120]) by kanga.kvack.org (Postfix) with ESMTP id 3FB2F6B0005 for ; Tue, 10 Sep 2019 16:26:28 -0400 (EDT) Received: from smtpin14.hostedemail.com (10.5.19.251.rfc1918.com [10.5.19.251]) by forelay05.hostedemail.com (Postfix) with SMTP id C3949181AC9AE for ; Tue, 10 Sep 2019 20:26:27 +0000 (UTC) X-FDA: 75920143614.14.rifle01_1b3a6eeee5324 X-HE-Tag: rifle01_1b3a6eeee5324 X-Filterd-Recvd-Size: 4951 Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by imf48.hostedemail.com (Postfix) with ESMTP for ; Tue, 10 Sep 2019 20:26:26 +0000 (UTC) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga106.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 10 Sep 2019 13:26:25 -0700 X-IronPort-AV: E=Sophos;i="5.64,490,1559545200"; d="scan'208";a="175429665" Received: from ahduyck-desk1.jf.intel.com ([10.7.198.76]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 10 Sep 2019 13:26:24 -0700 Message-ID: Subject: Re: [PATCH v9 3/8] mm: Move set/get_pcppage_migratetype to mmzone.h From: Alexander Duyck To: Michal Hocko , Alexander Duyck Cc: virtio-dev@lists.oasis-open.org, kvm list , "Michael S. Tsirkin" , Catalin Marinas , David Hildenbrand , Dave Hansen , LKML , Matthew Wilcox , linux-mm , Andrew Morton , will@kernel.org, linux-arm-kernel@lists.infradead.org, Oscar Salvador , Yang Zhang , Pankaj Gupta , Konrad Rzeszutek Wilk , Nitesh Narayan Lal , Rik van Riel , lcapitulino@redhat.com, "Wang, Wei W" , Andrea Arcangeli , ying.huang@intel.com, Paolo Bonzini , Dan Williams , Fengguang Wu , "Kirill A. Shutemov" Date: Tue, 10 Sep 2019 13:26:24 -0700 In-Reply-To: <20190910174553.GC4023@dhcp22.suse.cz> References: <20190907172225.10910.34302.stgit@localhost.localdomain> <20190907172528.10910.37051.stgit@localhost.localdomain> <20190910122313.GW2063@dhcp22.suse.cz> <20190910174553.GC4023@dhcp22.suse.cz> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.30.5 (3.30.5-1.fc29) MIME-Version: 1.0 Content-Transfer-Encoding: 7bit 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 Tue, 2019-09-10 at 19:45 +0200, Michal Hocko wrote: > On Tue 10-09-19 07:46:50, Alexander Duyck wrote: > > On Tue, Sep 10, 2019 at 5:23 AM Michal Hocko wrote: > > > On Sat 07-09-19 10:25:28, Alexander Duyck wrote: > > > > From: Alexander Duyck > > > > > > > > In order to support page reporting it will be necessary to store and > > > > retrieve the migratetype of a page. To enable that I am moving the set and > > > > get operations for pcppage_migratetype into the mm/internal.h header so > > > > that they can be used outside of the page_alloc.c file. > > > > > > Please describe who is the user and why does it needs this interface. > > > This is really important because migratetype is an MM internal thing and > > > external users shouldn't really care about it at all. We really do not > > > want a random code to call those, especially the set_pcppage_migratetype. > > > > I was using it to store the migratetype of the page so that I could > > find the boundary list that contained the reported page as the array > > is indexed based on page order and migratetype. However on further > > discussion I am thinking I may just use page->index directly to index > > into the boundary array. Doing that I should be able to get a very > > slight improvement in lookup time since I am not having to pull order > > and migratetype and then compute the index based on that. In addition > > it becomes much more clear as to what is going on, and if needed I > > could add debug checks to verify the page is "Reported" and that the > > "Buddy" page type is set. > > Be careful though. A free page belongs to the page allocator and it is > free to reuse any fields for its purpose so using any of them nilly > willy is no go. If you need to stuff something like that then there > better be an api the allocator is aware of. My main objection is the > abuse migrate type. There might be other ways to express what you need. > Please make sure you clearly define that though. I will. Basically if the Reported is set then it will mean that the index value is in use and provides the index into the boundary array. The Reported flag will be cleared when the page is pulled from the buddy list and in the case of the page being allocated it is already overwritten by __rmqueue_smallest calling set_pcppage_migratetype which is what gave me the idea to just use that in the first place.