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 D1281C636D4 for ; Fri, 10 Feb 2023 21:53:48 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 6DEBC28001A; Fri, 10 Feb 2023 16:53:48 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 68F05280003; Fri, 10 Feb 2023 16:53:48 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 57DA628001A; Fri, 10 Feb 2023 16:53:48 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0015.hostedemail.com [216.40.44.15]) by kanga.kvack.org (Postfix) with ESMTP id 4A899280003 for ; Fri, 10 Feb 2023 16:53:48 -0500 (EST) Received: from smtpin04.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay08.hostedemail.com (Postfix) with ESMTP id 1D9BC140483 for ; Fri, 10 Feb 2023 21:53:48 +0000 (UTC) X-FDA: 80452734936.04.A72F27B Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by imf07.hostedemail.com (Postfix) with ESMTP id 5E61E40016 for ; Fri, 10 Feb 2023 21:53:45 +0000 (UTC) Authentication-Results: imf07.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=B30GSdpw; spf=pass (imf07.hostedemail.com: domain of dave.jiang@intel.com designates 192.55.52.88 as permitted sender) smtp.mailfrom=dave.jiang@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1676066026; 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=8DSTIqeVozndV94O9kuwaj4PrUKtYaICPTL150I43nw=; b=Dn23g2BgWoOvdY5nYQw7iiztBDMNI3w/s7jDOGftWmCpGd1KPhMDrrRXMNiT9XEJ+/XfzJ J6jUeykMml9f/fxscdyL/rDnOfzVe0Ud23MfowxrHQgWz7J2rLIeYwn14okdLqfF8H0aOf 5FimxD8FEWOhg+WUGhO7Uip2yMWcj3M= ARC-Authentication-Results: i=1; imf07.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=B30GSdpw; spf=pass (imf07.hostedemail.com: domain of dave.jiang@intel.com designates 192.55.52.88 as permitted sender) smtp.mailfrom=dave.jiang@intel.com; dmarc=pass (policy=none) header.from=intel.com ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1676066026; a=rsa-sha256; cv=none; b=7bYAg96I83iQowYYRCGF5yRvndGrKo1nRLeHIzATCLAX5WycGHxrNRThbq2s3ncDangjeI 0ANijT9y+bsaOqFMlRa2KU2CWiDlAg4HkR4aT0ze0G2GhBXUzmLuAWG784K/FhpCEf+05t t7ACR1i68HRL0VUf09LRve4EnvUMb8Q= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1676066025; x=1707602025; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=vx5sAtd67oqSJ5HQNum0Qt2Ezi+uWPrpz/heDpnKEx0=; b=B30GSdpwdlrI5yY9nfrOP+2iFdiuU6+z5E4Kh5FjFrZiy21qsu5mgBrm FRNn2U97eM30xvslRrJEJDJDFVQ8NP7jsNxKQEWyzg7JjvDdK6JYy6xha OUCWK2PTh0WrtBfeCu+KYdyYi5MkHaBZyKS3mprUMBPK4bzUwh1+aFK0M qPY1aTKK3mEj2WdroplJZWjBp+1nsgdrmq7jUEM1/Zcjuub15izQWwLu3 yiClL7uVVekcfNfK3qBrKiizplUKCUX+pSoXbB5eQ9Kh47ZlZ1x6Mjp28 TA0Ejx1BffMqBCgv8jCgqJrD4w3J8EII1LnxeED3ckVdtyHORrgfl0vfd w==; X-IronPort-AV: E=McAfee;i="6500,9779,10617"; a="357937358" X-IronPort-AV: E=Sophos;i="5.97,287,1669104000"; d="scan'208";a="357937358" Received: from orsmga006.jf.intel.com ([10.7.209.51]) by fmsmga101.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Feb 2023 13:53:43 -0800 X-IronPort-AV: E=McAfee;i="6500,9779,10617"; a="645765617" X-IronPort-AV: E=Sophos;i="5.97,287,1669104000"; d="scan'208";a="645765617" Received: from djiang5-mobl3.amr.corp.intel.com (HELO [10.213.190.133]) ([10.213.190.133]) by orsmga006-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 10 Feb 2023 13:53:42 -0800 Message-ID: <448c17f6-cdfa-9422-c35e-cf95e1b99604@intel.com> Date: Fri, 10 Feb 2023 14:53: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 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> From: Dave Jiang In-Reply-To: <167602001107.1924368.11562316181038595611.stgit@dwillia2-xfh.jf.intel.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 5E61E40016 X-Rspam-User: X-Stat-Signature: w9ewsp58w3hcfch3f5fho3ixfykpd7rm X-HE-Tag: 1676066025-312574 X-HE-Meta: U2FsdGVkX18Jki6Ak+PdRswCTKNp+O2XQx1G+Sn1IQdTkInQ5pIbrBi8A0Rr/UbMt6RGZdS5Ul8peAlDjP8jkOupnoncS+nCXHjKXk3zItXc7AWzgOLopDNvyA7y+B/NToEiVwxmki8c4G/pjpoRkVLLn8ICVZ+RZ3T/RDseJ3X//l7A2TvAHGSKmDJCWga/zSQ/C0ZBCI7JlMBMUtYXs2FvY4MxwxXIzf+o3Ky/9QX0iDh9Jot4Dse6O78gEtL8vzzizMqIgMzZOc9JFPDySat0QEjQWf0ifa9CFb0ByT0j39NM7N7w6zeNcL503zDBVn3D1fDR1CxXsGidKfe702o/qtaxD88hlwqTPx2U8HT0WbmuHA9iz1R5rxGYuN1Lk/Xj1oSPfRW/8GN9iRJjJ6XrOgbSYmVqUN7QbarP20++aSml/HL5pMJPbG23Sipu+DybpR4EQBYpVWhn7QkHTW7BZZUsit8Cb1Z2uP6exSEXrlZzRba7s/Md1VvZWLONAWt5fLFB1Yz64bK+RCLX1z30QB0uL253pqLyvFwSuChbgTxwr6T6ZMJ9h05fD4TPr8aBQSewEMzuOAKT4mR7ylOrd2w3jXXrmvcsj1Z2g+Pk8A4DEeZfXGqADVLb0jliZ1Y0nnyf0rxoSs50ojJbG9eysW476enm7ZqVhgR5JgFP/pFeYahz47Cw0NccDIXgD53iRzt0c6m3hiyNal3gDoH7CFE+PDBN/c8sEINY2lHTdQ/c7bJA7hmHhdKKrIRo2s4iqx9y+BhRzL+Hw6p2HccaXbrrT0tq4tqSh/qD7BuAlHQdOKtrAzMWlXnta+1S8n5h1pwGMxBgdbQ5I4eWPf1UXgJfaEkJY7quQOzz1x3uJPMJoMfqwWqXLl6/WiSqNuEXQK40tQug9xOHIc80JliqMYJf/5ZuqR1KwbnfnjOAeJLqhiCKaGydnkkcIsC7aLzJ51288jSZbdlEZ3c Fw6a84/E fgdcaFhtT04kKTgAFbLA6fCUd/TPjUH/viL/JQE5ny4UJLpwe7CsSXZjJpfuizBB93KB+4tZiD6iPsQ7VjE4pFOprlV72dR9exSBQOqzFdZdBmLP3UhcZy4FGh7OJpCtorOl9a60PFP2qtaw0p+EQcIjEotUIEancHWkOeM63VnOT+8KfKnR7uerQ0wdwGZmnmXQD2vN0A6DSERyeRq4NBPM8RhWx/gNEkoPmUyI79cqflHx6bkH3tgY1aj7Xy5BDsU0+xGIjJ+2iwQM= 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: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 > --- > 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); >