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=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,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 340A8C282DD for ; Mon, 10 Jun 2019 11:02:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 11A8E20862 for ; Mon, 10 Jun 2019 11:02:11 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2389191AbfFJLCK (ORCPT ); Mon, 10 Jun 2019 07:02:10 -0400 Received: from cloudserver094114.home.pl ([79.96.170.134]:57174 "EHLO cloudserver094114.home.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2388982AbfFJLCK (ORCPT ); Mon, 10 Jun 2019 07:02:10 -0400 Received: from 79.184.253.190.ipv4.supernova.orange.pl (79.184.253.190) (HELO kreacher.localnet) by serwer1319399.home.pl (79.96.170.134) with SMTP (IdeaSmtpServer 0.83.213) id 87edd61c9de09749; Mon, 10 Jun 2019 13:02:08 +0200 From: "Rafael J. Wysocki" To: Dan Williams Cc: linux-kernel@vger.kernel.org, Len Brown , Keith Busch , peterz@infradead.org, vishal.l.verma@intel.com, dave.hansen@linux.intel.com, ard.biesheuvel@linaro.org, linux-nvdimm@lists.01.org, x86@kernel.org, linux-efi@vger.kernel.org Subject: Re: [PATCH v3 01/10] acpi/numa: Establish a new drivers/acpi/numa/ directory Date: Mon, 10 Jun 2019 13:02:07 +0200 Message-ID: <2852346.FjP3BEmj8C@kreacher> In-Reply-To: <155993563815.3036719.5542204455414415743.stgit@dwillia2-desk3.amr.corp.intel.com> References: <155993563277.3036719.17400338098057706494.stgit@dwillia2-desk3.amr.corp.intel.com> <155993563815.3036719.5542204455414415743.stgit@dwillia2-desk3.amr.corp.intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Sender: linux-efi-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-efi@vger.kernel.org On Friday, June 7, 2019 9:27:18 PM CEST Dan Williams wrote: > Currently hmat.c lives under an "hmat" directory which does not enhance > the description of the file. The initial motivation for giving hmat.c > its own directory was to delineate it as mm functionality in contrast to > ACPI device driver functionality. > > As ACPI continues to play an increasing role in conveying > memory location and performance topology information to the OS take the > opportunity to co-locate these NUMA relevant tables in a combined > directory. > > numa.c is renamed to srat.c and moved to drivers/acpi/numa/ along with > hmat.c. > > Cc: Len Brown > Cc: Keith Busch > Cc: "Rafael J. Wysocki" > Signed-off-by: Dan Williams A CC to linux-acpi is missing from this series. > --- > drivers/acpi/Kconfig | 9 +-------- > drivers/acpi/Makefile | 3 +-- > drivers/acpi/hmat/Makefile | 2 -- > drivers/acpi/numa/Kconfig | 7 ++++++- > drivers/acpi/numa/Makefile | 3 +++ > drivers/acpi/numa/hmat.c | 0 > drivers/acpi/numa/srat.c | 0 > 7 files changed, 11 insertions(+), 13 deletions(-) > delete mode 100644 drivers/acpi/hmat/Makefile > rename drivers/acpi/{hmat/Kconfig => numa/Kconfig} (72%) > create mode 100644 drivers/acpi/numa/Makefile > rename drivers/acpi/{hmat/hmat.c => numa/hmat.c} (100%) > rename drivers/acpi/{numa.c => numa/srat.c} (100%) > > diff --git a/drivers/acpi/Kconfig b/drivers/acpi/Kconfig > index 283ee94224c6..82c4a31c8701 100644 > --- a/drivers/acpi/Kconfig > +++ b/drivers/acpi/Kconfig > @@ -321,12 +321,6 @@ config ACPI_THERMAL > To compile this driver as a module, choose M here: > the module will be called thermal. > > -config ACPI_NUMA > - bool "NUMA support" > - depends on NUMA > - depends on (X86 || IA64 || ARM64) > - default y if IA64_GENERIC || IA64_SGI_SN2 || ARM64 > - > config ACPI_CUSTOM_DSDT_FILE > string "Custom DSDT Table file to include" > default "" > @@ -475,8 +469,7 @@ config ACPI_REDUCED_HARDWARE_ONLY > If you are unsure what to do, do not enable this option. > > source "drivers/acpi/nfit/Kconfig" > -source "drivers/acpi/hmat/Kconfig" > - > +source "drivers/acpi/numa/Kconfig" > source "drivers/acpi/apei/Kconfig" > source "drivers/acpi/dptf/Kconfig" > > diff --git a/drivers/acpi/Makefile b/drivers/acpi/Makefile > index 5d361e4e3405..f08a661274e8 100644 > --- a/drivers/acpi/Makefile > +++ b/drivers/acpi/Makefile > @@ -55,7 +55,6 @@ acpi-$(CONFIG_X86) += acpi_cmos_rtc.o > acpi-$(CONFIG_X86) += x86/apple.o > acpi-$(CONFIG_X86) += x86/utils.o > acpi-$(CONFIG_DEBUG_FS) += debugfs.o > -acpi-$(CONFIG_ACPI_NUMA) += numa.o > acpi-$(CONFIG_ACPI_PROCFS_POWER) += cm_sbs.o > acpi-y += acpi_lpat.o > acpi-$(CONFIG_ACPI_LPIT) += acpi_lpit.o > @@ -80,7 +79,7 @@ obj-$(CONFIG_ACPI_PROCESSOR) += processor.o > obj-$(CONFIG_ACPI) += container.o > obj-$(CONFIG_ACPI_THERMAL) += thermal.o > obj-$(CONFIG_ACPI_NFIT) += nfit/ > -obj-$(CONFIG_ACPI_HMAT) += hmat/ > +obj-$(CONFIG_ACPI_NUMA) += numa/ > obj-$(CONFIG_ACPI) += acpi_memhotplug.o > obj-$(CONFIG_ACPI_HOTPLUG_IOAPIC) += ioapic.o > obj-$(CONFIG_ACPI_BATTERY) += battery.o > diff --git a/drivers/acpi/hmat/Makefile b/drivers/acpi/hmat/Makefile > deleted file mode 100644 > index 1c20ef36a385..000000000000 > --- a/drivers/acpi/hmat/Makefile > +++ /dev/null > @@ -1,2 +0,0 @@ > -# SPDX-License-Identifier: GPL-2.0-only > -obj-$(CONFIG_ACPI_HMAT) := hmat.o > diff --git a/drivers/acpi/hmat/Kconfig b/drivers/acpi/numa/Kconfig > similarity index 72% > rename from drivers/acpi/hmat/Kconfig > rename to drivers/acpi/numa/Kconfig > index 95a29964dbea..d14582387ed0 100644 > --- a/drivers/acpi/hmat/Kconfig > +++ b/drivers/acpi/numa/Kconfig > @@ -1,4 +1,9 @@ > -# SPDX-License-Identifier: GPL-2.0 > +config ACPI_NUMA > + bool "NUMA support" > + depends on NUMA > + depends on (X86 || IA64 || ARM64) > + default y if IA64_GENERIC || IA64_SGI_SN2 || ARM64 > + > config ACPI_HMAT > bool "ACPI Heterogeneous Memory Attribute Table Support" > depends on ACPI_NUMA > diff --git a/drivers/acpi/numa/Makefile b/drivers/acpi/numa/Makefile > new file mode 100644 > index 000000000000..517a6c689a94 > --- /dev/null > +++ b/drivers/acpi/numa/Makefile > @@ -0,0 +1,3 @@ > +# SPDX-License-Identifier: GPL-2.0-only > +obj-$(CONFIG_ACPI_NUMA) += srat.o > +obj-$(CONFIG_ACPI_HMAT) += hmat.o > diff --git a/drivers/acpi/hmat/hmat.c b/drivers/acpi/numa/hmat.c > similarity index 100% > rename from drivers/acpi/hmat/hmat.c > rename to drivers/acpi/numa/hmat.c > diff --git a/drivers/acpi/numa.c b/drivers/acpi/numa/srat.c > similarity index 100% > rename from drivers/acpi/numa.c > rename to drivers/acpi/numa/srat.c > >