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 Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id 94202C64ED8 for ; Fri, 10 Feb 2023 21:57:56 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 30B8B280023; Fri, 10 Feb 2023 16:57:56 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 2BC58280021; Fri, 10 Feb 2023 16:57:56 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 183C4280023; Fri, 10 Feb 2023 16:57:56 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0013.hostedemail.com [216.40.44.13]) by kanga.kvack.org (Postfix) with ESMTP id 084B6280021 for ; Fri, 10 Feb 2023 16:57:56 -0500 (EST) Received: from smtpin10.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay04.hostedemail.com (Postfix) with ESMTP id CF01D1A0648 for ; Fri, 10 Feb 2023 21:57:55 +0000 (UTC) X-FDA: 80452745310.10.5F0540B Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by imf27.hostedemail.com (Postfix) with ESMTP id A682240002 for ; Fri, 10 Feb 2023 21:57:52 +0000 (UTC) Authentication-Results: imf27.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="ViogKCs/"; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf27.hostedemail.com: domain of dave.jiang@intel.com designates 192.55.52.115 as permitted sender) smtp.mailfrom=dave.jiang@intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1676066272; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=/eq6SPnx9ZQpgXoGl2iYZ+tNFoIlBAio89ovDJEQhRI=; b=ojuOky5zzZsPAPEcSoqFCLkJPcCG5uLzVqTsU2Waq+HOz+yT15218bPs6r/2VAOB4snXWm zo83yXNqyhfDaXzZu7af6qhqkJxAoP9vNO+PMgIOuaVzq6S8dW0275fuSYbXr8jBXZ+5aR FTE1j5W1g0VVfBeDpmu4VJhWwLT7gUI= ARC-Authentication-Results: i=1; imf27.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b="ViogKCs/"; dmarc=pass (policy=none) header.from=intel.com; spf=pass (imf27.hostedemail.com: domain of dave.jiang@intel.com designates 192.55.52.115 as permitted sender) smtp.mailfrom=dave.jiang@intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1676066272; a=rsa-sha256; cv=none; b=0vn+y9YSrV4LbUffeJiYkPJJAffsEczuMqhigr3NiTynhIYB1dm0kEjds9x00VlpmsA1er vuS3HlogQ710cA0bRnQHpH/pfw0sZZYvwgD8pHzu+insJM/ePMnBWDufOFgr6xXgbc47/5 niTH5zaJ9Bpk5Sana6QYgTjvfJ+7zA0= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1676066272; x=1707602272; h=message-id:date:mime-version:subject:from:to:cc: references:in-reply-to:content-transfer-encoding; bh=vPxF1JseRSs4g26NzPX6yAqIVHYMWM5zfMZbpc6cPmA=; b=ViogKCs/OdtFHgA3SbrrvUH8fykGDv4SU2prAfG1Haon88GFyEx4YJVS lLCWfyL/yGOSTHagpIDR1EAzuVpwbVi9FXWZN/wkK+6hx/5fOgzL9E/g1 rPjaPcaERcwUZJDMSCcNDlGFib2moxvRr0AcQzOI2sTLI/xsdAP7XHplf 9O84aRF4AZdBK4J33SIOxpNVkLSgSawXLvF14Qj8N0NGCHs5Bl5Cf5iQY wluIsc+b3Zw0yI+uJbiza1Xms2OUrn5oZdF5Qs0wkkD5SASXVqs705GKg ffHtzmKy+7eB4DAuNUPps9O1YaJcW8OHVVljKBi7DvpCf40+NRZClYg0c Q==; X-IronPort-AV: E=McAfee;i="6500,9779,10617"; a="330544828" X-IronPort-AV: E=Sophos;i="5.97,287,1669104000"; d="scan'208";a="330544828" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Feb 2023 13:57:43 -0800 X-IronPort-AV: E=McAfee;i="6500,9779,10617"; a="700616193" X-IronPort-AV: E=Sophos;i="5.97,287,1669104000"; d="scan'208";a="700616193" Received: from djiang5-mobl3.amr.corp.intel.com (HELO [10.213.190.133]) ([10.213.190.133]) by orsmga001-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Feb 2023 13:57:42 -0800 Message-ID: <368f047a-b722-504f-41de-41b3f10cdeb8@intel.com> Date: Fri, 10 Feb 2023 14:57:42 -0700 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Firefox/102.0 Thunderbird/102.6.0 Subject: Re: [PATCH v2 15/20] dax/hmem: Move HMAT and Soft reservation probe initcall level Content-Language: en-US From: Dave Jiang To: Dan Williams , linux-cxl@vger.kernel.org Cc: Fan Ni , vishal.l.verma@intel.com, dave.hansen@linux.intel.com, linux-mm@kvack.org, linux-acpi@vger.kernel.org References: <167601992097.1924368.18291887895351917895.stgit@dwillia2-xfh.jf.intel.com> <167602001107.1924368.11562316181038595611.stgit@dwillia2-xfh.jf.intel.com> <448c17f6-cdfa-9422-c35e-cf95e1b99604@intel.com> In-Reply-To: <448c17f6-cdfa-9422-c35e-cf95e1b99604@intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Rspam-User: X-Rspamd-Server: rspam02 X-Rspamd-Queue-Id: A682240002 X-Stat-Signature: quermes9oi8ox6r38zfoy3y88nc18b6p X-HE-Tag: 1676066272-693880 X-HE-Meta: U2FsdGVkX19r8EoviM7ovHWaA3MGz+PTwfoc8RH4/LlJH4ghi9HCsH8mEY7kYitF/oI9pSthwe3cD0CUkKRlvl/VZDmXop8agpY41k+wXheJVZKSCPfECTi7ZlOyg4meoQruVGI+BmOMDyTvzVHGoQgzOLyUqHtVXTSXTrII65GtQ730cMRqs+1px6MRh2qLBC/9go/0ymtlFSzjm5EZonNhciAxSIIkayQJoCoKG6Bnio3oKHlQrmXxj01zOj0It02jHSVKH6PO5+fuCBiy3n7ogP9ZIaHRSuOvNkalgCl2g2ajzG7ADTtlc9A6n3TPwGnuwrqV3cBPHdcchgoAt5xh+LdGYdGPf2Bb5R7zNwhkKSM3ooOtEVitrVhpipPU4TFWFGV++IzHZ9GeVEo73xiBDSkO6l5hV4fdxb9r6dR91AKVRRWBkjqdPPRcTV1WgtH3Loe3oymBH/Zkebl+bSB4L80R+1BtUGqnfXNq3wkfDuSffMpUE0ZMG9v/ryo0Gff3cOavHhj6cDuWKBz9/nSWeD+78V/9pdPVC/TDtD8UuQfeg5zuuy+nDnshYXcY4K5rhRJNvMXQG0u9jWbvsurJZGYKW+K00iI0EqeMuvsTdg9289EskwMsqIKls5ktXrStGN5q1PEXFH10FuUbrlzZDtjdfjeKPrjL9Hsr1AoSTCB9GO2NHsseEKHJ5wmf4fhX7KqMnr7SyebDfbqp9uj9bMOROk0pAN7Ne/INqwsTa1DuzrFAqwJ2/I8o0rBmXuzpfT3YCSG2FGQcymCXDlVVNksBeu8pqBSP9B/xbDP3lb5JMEYxI5Rf0goWxe9LO2+91zZOeI81xqjBOM2tB4q5JiQltodjSn+Eo9ydOYRZkBtNVTq8UYTLq9hKRm2WpN2kRHIeUvZIEIcMppbKoDB56pcMIeHmw9K0BD3HoCEJUPoDE0aZDLbZ6z1H1KiBbVVcfDvQM/4LUaD/DUw PX4hQPNU /0gYqPysdDlfY+I+p2f0k3DRVZAznze6M+DUDust+rH5MBNjUo7laIhgJxdSVdW+sQ77fHUAo04TK68pIvgWBtVTNHYQ9r/o9cimIrOPmSMVT+j7TZWxMW6/ngBcYxWRoWJHuVy/a0sFr6QM48H39dnFENUC9d/0MDDb3stcrxC40VuKbZ+4MLsbnqh0It2LdjrK9OsfeYvoBoNUuz5wHXJD3VWQ1WO2gY/06jA19/+pMdvIpJ2JGuGBziBiWdstlA+QH8zicTN/Ni+Y= 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 2/10/23 2:53 PM, Dave Jiang wrote: > > > On 2/10/23 2:06 AM, Dan Williams wrote: >> In preparation for moving more filtering of "hmem" ranges into the >> dax_hmem.ko module, update the initcall levels. HMAT range registration >> moves to subsys_initcall() to be done before Soft Reservation probing, >> and Soft Reservation probing is moved to device_initcall() to be done >> before dax_hmem.ko initialization if it is built-in. >> >> Tested-by: Fan Ni >> Link: >> https://lore.kernel.org/r/167564542109.847146.10113972881782419363.stgit@dwillia2-xfh.jf.intel.com >> Signed-off-by: Dan Williams > > Reviewed-by: Dave Jiang Reviewed-by: Dave Jiang > >> --- >>   drivers/acpi/numa/hmat.c  |    2 +- >>   drivers/dax/hmem/Makefile |    3 ++- >>   drivers/dax/hmem/device.c |    2 +- >>   3 files changed, 4 insertions(+), 3 deletions(-) >> >> diff --git a/drivers/acpi/numa/hmat.c b/drivers/acpi/numa/hmat.c >> index 605a0c7053be..ff24282301ab 100644 >> --- a/drivers/acpi/numa/hmat.c >> +++ b/drivers/acpi/numa/hmat.c >> @@ -869,4 +869,4 @@ static __init int hmat_init(void) >>       acpi_put_table(tbl); >>       return 0; >>   } >> -device_initcall(hmat_init); >> +subsys_initcall(hmat_init); >> diff --git a/drivers/dax/hmem/Makefile b/drivers/dax/hmem/Makefile >> index 57377b4c3d47..d4c4cd6bccd7 100644 >> --- a/drivers/dax/hmem/Makefile >> +++ b/drivers/dax/hmem/Makefile >> @@ -1,6 +1,7 @@ >>   # SPDX-License-Identifier: GPL-2.0 >> -obj-$(CONFIG_DEV_DAX_HMEM) += dax_hmem.o >> +# device_hmem.o deliberately precedes dax_hmem.o for initcall ordering >>   obj-$(CONFIG_DEV_DAX_HMEM_DEVICES) += device_hmem.o >> +obj-$(CONFIG_DEV_DAX_HMEM) += dax_hmem.o >>   device_hmem-y := device.o >>   dax_hmem-y := hmem.o >> diff --git a/drivers/dax/hmem/device.c b/drivers/dax/hmem/device.c >> index 903325aac991..20749c7fab81 100644 >> --- a/drivers/dax/hmem/device.c >> +++ b/drivers/dax/hmem/device.c >> @@ -104,4 +104,4 @@ static __init int hmem_init(void) >>    * As this is a fallback for address ranges unclaimed by the ACPI HMAT >>    * parsing it must be at an initcall level greater than hmat_init(). >>    */ >> -late_initcall(hmem_init); >> +device_initcall(hmem_init); >>