* [PATCH AUTOSEL 5.14 03/25] cxl: Move cxl_core to new directory [not found] <20210913223339.435347-1-sashal@kernel.org> @ 2021-09-13 22:33 ` Sasha Levin 2021-09-14 8:56 ` Jonathan Cameron 0 siblings, 1 reply; 6+ messages in thread From: Sasha Levin @ 2021-09-13 22:33 UTC (permalink / raw) To: linux-kernel, stable Cc: Ben Widawsky, kernel test robot, Jonathan Cameron, Dan Williams, Sasha Levin, linux-doc, linux-cxl From: Ben Widawsky <ben.widawsky@intel.com> [ Upstream commit 5161a55c069f53d88da49274cbef6e3c74eadea9 ] CXL core is growing, and it's already arguably unmanageable. To support future growth, move core functionality to a new directory and rename the file to represent just bus support. Future work will remove non-bus functionality. Note that mem.h is renamed to cxlmem.h to avoid a namespace collision with the global ARCH=um mem.h header. Reported-by: kernel test robot <lkp@intel.com> Signed-off-by: Ben Widawsky <ben.widawsky@intel.com> Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> Link: https://lore.kernel.org/r/162792537866.368511.8915631504621088321.stgit@dwillia2-desk3.amr.corp.intel.com Signed-off-by: Dan Williams <dan.j.williams@intel.com> Signed-off-by: Sasha Levin <sashal@kernel.org> --- Documentation/driver-api/cxl/memory-devices.rst | 2 +- drivers/cxl/Makefile | 4 +--- drivers/cxl/core/Makefile | 5 +++++ drivers/cxl/{core.c => core/bus.c} | 4 ++-- drivers/cxl/{mem.h => cxlmem.h} | 0 drivers/cxl/pci.c | 2 +- drivers/cxl/pmem.c | 2 +- 7 files changed, 11 insertions(+), 8 deletions(-) create mode 100644 drivers/cxl/core/Makefile rename drivers/cxl/{core.c => core/bus.c} (99%) rename drivers/cxl/{mem.h => cxlmem.h} (100%) diff --git a/Documentation/driver-api/cxl/memory-devices.rst b/Documentation/driver-api/cxl/memory-devices.rst index 487ce4f41d77..a86e2c7c551a 100644 --- a/Documentation/driver-api/cxl/memory-devices.rst +++ b/Documentation/driver-api/cxl/memory-devices.rst @@ -36,7 +36,7 @@ CXL Core .. kernel-doc:: drivers/cxl/cxl.h :internal: -.. kernel-doc:: drivers/cxl/core.c +.. kernel-doc:: drivers/cxl/core/bus.c :doc: cxl core External Interfaces diff --git a/drivers/cxl/Makefile b/drivers/cxl/Makefile index 32954059b37b..d1aaabc940f3 100644 --- a/drivers/cxl/Makefile +++ b/drivers/cxl/Makefile @@ -1,11 +1,9 @@ # SPDX-License-Identifier: GPL-2.0 -obj-$(CONFIG_CXL_BUS) += cxl_core.o +obj-$(CONFIG_CXL_BUS) += core/ obj-$(CONFIG_CXL_MEM) += cxl_pci.o obj-$(CONFIG_CXL_ACPI) += cxl_acpi.o obj-$(CONFIG_CXL_PMEM) += cxl_pmem.o -ccflags-y += -DDEFAULT_SYMBOL_NAMESPACE=CXL -cxl_core-y := core.o cxl_pci-y := pci.o cxl_acpi-y := acpi.o cxl_pmem-y := pmem.o diff --git a/drivers/cxl/core/Makefile b/drivers/cxl/core/Makefile new file mode 100644 index 000000000000..ad137f96e5c8 --- /dev/null +++ b/drivers/cxl/core/Makefile @@ -0,0 +1,5 @@ +# SPDX-License-Identifier: GPL-2.0 +obj-$(CONFIG_CXL_BUS) += cxl_core.o + +ccflags-y += -DDEFAULT_SYMBOL_NAMESPACE=CXL -I$(srctree)/drivers/cxl +cxl_core-y := bus.o diff --git a/drivers/cxl/core.c b/drivers/cxl/core/bus.c similarity index 99% rename from drivers/cxl/core.c rename to drivers/cxl/core/bus.c index a2e4d54fc7bc..0815eec23944 100644 --- a/drivers/cxl/core.c +++ b/drivers/cxl/core/bus.c @@ -6,8 +6,8 @@ #include <linux/pci.h> #include <linux/slab.h> #include <linux/idr.h> -#include "cxl.h" -#include "mem.h" +#include <cxlmem.h> +#include <cxl.h> /** * DOC: cxl core diff --git a/drivers/cxl/mem.h b/drivers/cxl/cxlmem.h similarity index 100% rename from drivers/cxl/mem.h rename to drivers/cxl/cxlmem.h diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c index 4cf351a3cf99..a945c5fda292 100644 --- a/drivers/cxl/pci.c +++ b/drivers/cxl/pci.c @@ -12,9 +12,9 @@ #include <linux/pci.h> #include <linux/io.h> #include <linux/io-64-nonatomic-lo-hi.h> +#include "cxlmem.h" #include "pci.h" #include "cxl.h" -#include "mem.h" /** * DOC: cxl pci diff --git a/drivers/cxl/pmem.c b/drivers/cxl/pmem.c index 0088e41dd2f3..9652c3ee41e7 100644 --- a/drivers/cxl/pmem.c +++ b/drivers/cxl/pmem.c @@ -6,7 +6,7 @@ #include <linux/ndctl.h> #include <linux/async.h> #include <linux/slab.h> -#include "mem.h" +#include "cxlmem.h" #include "cxl.h" /* -- 2.30.2 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH AUTOSEL 5.14 03/25] cxl: Move cxl_core to new directory 2021-09-13 22:33 ` [PATCH AUTOSEL 5.14 03/25] cxl: Move cxl_core to new directory Sasha Levin @ 2021-09-14 8:56 ` Jonathan Cameron 2021-09-14 8:57 ` Jonathan Cameron 0 siblings, 1 reply; 6+ messages in thread From: Jonathan Cameron @ 2021-09-14 8:56 UTC (permalink / raw) To: Sasha Levin Cc: linux-kernel, stable, Ben Widawsky, kernel test robot, Dan Williams, linux-doc, linux-cxl On Mon, 13 Sep 2021 18:33:17 -0400 Sasha Levin <sashal@kernel.org> wrote: > From: Ben Widawsky <ben.widawsky@intel.com> > > [ Upstream commit 5161a55c069f53d88da49274cbef6e3c74eadea9 ] > > CXL core is growing, and it's already arguably unmanageable. To support > future growth, move core functionality to a new directory and rename the > file to represent just bus support. Future work will remove non-bus > functionality. > > Note that mem.h is renamed to cxlmem.h to avoid a namespace collision > with the global ARCH=um mem.h header. Not a fix... I'm guessing this got picked up on the basis of the Reported-by: tag? I think that was added for a minor tweak as this went through review rather than referring to the whole patch. Jonathan > > Reported-by: kernel test robot <lkp@intel.com> > Signed-off-by: Ben Widawsky <ben.widawsky@intel.com> > Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> > Link: https://lore.kernel.org/r/162792537866.368511.8915631504621088321.stgit@dwillia2-desk3.amr.corp.intel.com > Signed-off-by: Dan Williams <dan.j.williams@intel.com> > Signed-off-by: Sasha Levin <sashal@kernel.org> > --- > Documentation/driver-api/cxl/memory-devices.rst | 2 +- > drivers/cxl/Makefile | 4 +--- > drivers/cxl/core/Makefile | 5 +++++ > drivers/cxl/{core.c => core/bus.c} | 4 ++-- > drivers/cxl/{mem.h => cxlmem.h} | 0 > drivers/cxl/pci.c | 2 +- > drivers/cxl/pmem.c | 2 +- > 7 files changed, 11 insertions(+), 8 deletions(-) > create mode 100644 drivers/cxl/core/Makefile > rename drivers/cxl/{core.c => core/bus.c} (99%) > rename drivers/cxl/{mem.h => cxlmem.h} (100%) > > diff --git a/Documentation/driver-api/cxl/memory-devices.rst b/Documentation/driver-api/cxl/memory-devices.rst > index 487ce4f41d77..a86e2c7c551a 100644 > --- a/Documentation/driver-api/cxl/memory-devices.rst > +++ b/Documentation/driver-api/cxl/memory-devices.rst > @@ -36,7 +36,7 @@ CXL Core > .. kernel-doc:: drivers/cxl/cxl.h > :internal: > > -.. kernel-doc:: drivers/cxl/core.c > +.. kernel-doc:: drivers/cxl/core/bus.c > :doc: cxl core > > External Interfaces > diff --git a/drivers/cxl/Makefile b/drivers/cxl/Makefile > index 32954059b37b..d1aaabc940f3 100644 > --- a/drivers/cxl/Makefile > +++ b/drivers/cxl/Makefile > @@ -1,11 +1,9 @@ > # SPDX-License-Identifier: GPL-2.0 > -obj-$(CONFIG_CXL_BUS) += cxl_core.o > +obj-$(CONFIG_CXL_BUS) += core/ > obj-$(CONFIG_CXL_MEM) += cxl_pci.o > obj-$(CONFIG_CXL_ACPI) += cxl_acpi.o > obj-$(CONFIG_CXL_PMEM) += cxl_pmem.o > > -ccflags-y += -DDEFAULT_SYMBOL_NAMESPACE=CXL > -cxl_core-y := core.o > cxl_pci-y := pci.o > cxl_acpi-y := acpi.o > cxl_pmem-y := pmem.o > diff --git a/drivers/cxl/core/Makefile b/drivers/cxl/core/Makefile > new file mode 100644 > index 000000000000..ad137f96e5c8 > --- /dev/null > +++ b/drivers/cxl/core/Makefile > @@ -0,0 +1,5 @@ > +# SPDX-License-Identifier: GPL-2.0 > +obj-$(CONFIG_CXL_BUS) += cxl_core.o > + > +ccflags-y += -DDEFAULT_SYMBOL_NAMESPACE=CXL -I$(srctree)/drivers/cxl > +cxl_core-y := bus.o > diff --git a/drivers/cxl/core.c b/drivers/cxl/core/bus.c > similarity index 99% > rename from drivers/cxl/core.c > rename to drivers/cxl/core/bus.c > index a2e4d54fc7bc..0815eec23944 100644 > --- a/drivers/cxl/core.c > +++ b/drivers/cxl/core/bus.c > @@ -6,8 +6,8 @@ > #include <linux/pci.h> > #include <linux/slab.h> > #include <linux/idr.h> > -#include "cxl.h" > -#include "mem.h" > +#include <cxlmem.h> > +#include <cxl.h> > > /** > * DOC: cxl core > diff --git a/drivers/cxl/mem.h b/drivers/cxl/cxlmem.h > similarity index 100% > rename from drivers/cxl/mem.h > rename to drivers/cxl/cxlmem.h > diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c > index 4cf351a3cf99..a945c5fda292 100644 > --- a/drivers/cxl/pci.c > +++ b/drivers/cxl/pci.c > @@ -12,9 +12,9 @@ > #include <linux/pci.h> > #include <linux/io.h> > #include <linux/io-64-nonatomic-lo-hi.h> > +#include "cxlmem.h" > #include "pci.h" > #include "cxl.h" > -#include "mem.h" > > /** > * DOC: cxl pci > diff --git a/drivers/cxl/pmem.c b/drivers/cxl/pmem.c > index 0088e41dd2f3..9652c3ee41e7 100644 > --- a/drivers/cxl/pmem.c > +++ b/drivers/cxl/pmem.c > @@ -6,7 +6,7 @@ > #include <linux/ndctl.h> > #include <linux/async.h> > #include <linux/slab.h> > -#include "mem.h" > +#include "cxlmem.h" > #include "cxl.h" > > /* ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH AUTOSEL 5.14 03/25] cxl: Move cxl_core to new directory 2021-09-14 8:56 ` Jonathan Cameron @ 2021-09-14 8:57 ` Jonathan Cameron 2021-09-14 15:05 ` Ben Widawsky 0 siblings, 1 reply; 6+ messages in thread From: Jonathan Cameron @ 2021-09-14 8:57 UTC (permalink / raw) To: Sasha Levin Cc: linux-kernel, stable, Ben Widawsky, kernel test robot, Dan Williams, linux-doc, linux-cxl On Tue, 14 Sep 2021 09:56:23 +0100 Jonathan Cameron <Jonathan.Cameron@Huawei.com> wrote: > On Mon, 13 Sep 2021 18:33:17 -0400 > Sasha Levin <sashal@kernel.org> wrote: > > > From: Ben Widawsky <ben.widawsky@intel.com> > > > > [ Upstream commit 5161a55c069f53d88da49274cbef6e3c74eadea9 ] > > > > CXL core is growing, and it's already arguably unmanageable. To support > > future growth, move core functionality to a new directory and rename the > > file to represent just bus support. Future work will remove non-bus > > functionality. > > > > Note that mem.h is renamed to cxlmem.h to avoid a namespace collision > > with the global ARCH=um mem.h header. > > Not a fix... > > I'm guessing this got picked up on the basis of the Reported-by: tag? > I think that was added for a minor tweak as this went through review rather > than referring to the whole patch. Or possibly because it was a precursor to the fix in the next patch. Hmm. Ben, Dan, does it make sense for these two to go into stable? Jonathan > > Jonathan > > > > > > Reported-by: kernel test robot <lkp@intel.com> > > Signed-off-by: Ben Widawsky <ben.widawsky@intel.com> > > Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> > > Link: https://lore.kernel.org/r/162792537866.368511.8915631504621088321.stgit@dwillia2-desk3.amr.corp.intel.com > > Signed-off-by: Dan Williams <dan.j.williams@intel.com> > > Signed-off-by: Sasha Levin <sashal@kernel.org> > > --- > > Documentation/driver-api/cxl/memory-devices.rst | 2 +- > > drivers/cxl/Makefile | 4 +--- > > drivers/cxl/core/Makefile | 5 +++++ > > drivers/cxl/{core.c => core/bus.c} | 4 ++-- > > drivers/cxl/{mem.h => cxlmem.h} | 0 > > drivers/cxl/pci.c | 2 +- > > drivers/cxl/pmem.c | 2 +- > > 7 files changed, 11 insertions(+), 8 deletions(-) > > create mode 100644 drivers/cxl/core/Makefile > > rename drivers/cxl/{core.c => core/bus.c} (99%) > > rename drivers/cxl/{mem.h => cxlmem.h} (100%) > > > > diff --git a/Documentation/driver-api/cxl/memory-devices.rst b/Documentation/driver-api/cxl/memory-devices.rst > > index 487ce4f41d77..a86e2c7c551a 100644 > > --- a/Documentation/driver-api/cxl/memory-devices.rst > > +++ b/Documentation/driver-api/cxl/memory-devices.rst > > @@ -36,7 +36,7 @@ CXL Core > > .. kernel-doc:: drivers/cxl/cxl.h > > :internal: > > > > -.. kernel-doc:: drivers/cxl/core.c > > +.. kernel-doc:: drivers/cxl/core/bus.c > > :doc: cxl core > > > > External Interfaces > > diff --git a/drivers/cxl/Makefile b/drivers/cxl/Makefile > > index 32954059b37b..d1aaabc940f3 100644 > > --- a/drivers/cxl/Makefile > > +++ b/drivers/cxl/Makefile > > @@ -1,11 +1,9 @@ > > # SPDX-License-Identifier: GPL-2.0 > > -obj-$(CONFIG_CXL_BUS) += cxl_core.o > > +obj-$(CONFIG_CXL_BUS) += core/ > > obj-$(CONFIG_CXL_MEM) += cxl_pci.o > > obj-$(CONFIG_CXL_ACPI) += cxl_acpi.o > > obj-$(CONFIG_CXL_PMEM) += cxl_pmem.o > > > > -ccflags-y += -DDEFAULT_SYMBOL_NAMESPACE=CXL > > -cxl_core-y := core.o > > cxl_pci-y := pci.o > > cxl_acpi-y := acpi.o > > cxl_pmem-y := pmem.o > > diff --git a/drivers/cxl/core/Makefile b/drivers/cxl/core/Makefile > > new file mode 100644 > > index 000000000000..ad137f96e5c8 > > --- /dev/null > > +++ b/drivers/cxl/core/Makefile > > @@ -0,0 +1,5 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +obj-$(CONFIG_CXL_BUS) += cxl_core.o > > + > > +ccflags-y += -DDEFAULT_SYMBOL_NAMESPACE=CXL -I$(srctree)/drivers/cxl > > +cxl_core-y := bus.o > > diff --git a/drivers/cxl/core.c b/drivers/cxl/core/bus.c > > similarity index 99% > > rename from drivers/cxl/core.c > > rename to drivers/cxl/core/bus.c > > index a2e4d54fc7bc..0815eec23944 100644 > > --- a/drivers/cxl/core.c > > +++ b/drivers/cxl/core/bus.c > > @@ -6,8 +6,8 @@ > > #include <linux/pci.h> > > #include <linux/slab.h> > > #include <linux/idr.h> > > -#include "cxl.h" > > -#include "mem.h" > > +#include <cxlmem.h> > > +#include <cxl.h> > > > > /** > > * DOC: cxl core > > diff --git a/drivers/cxl/mem.h b/drivers/cxl/cxlmem.h > > similarity index 100% > > rename from drivers/cxl/mem.h > > rename to drivers/cxl/cxlmem.h > > diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c > > index 4cf351a3cf99..a945c5fda292 100644 > > --- a/drivers/cxl/pci.c > > +++ b/drivers/cxl/pci.c > > @@ -12,9 +12,9 @@ > > #include <linux/pci.h> > > #include <linux/io.h> > > #include <linux/io-64-nonatomic-lo-hi.h> > > +#include "cxlmem.h" > > #include "pci.h" > > #include "cxl.h" > > -#include "mem.h" > > > > /** > > * DOC: cxl pci > > diff --git a/drivers/cxl/pmem.c b/drivers/cxl/pmem.c > > index 0088e41dd2f3..9652c3ee41e7 100644 > > --- a/drivers/cxl/pmem.c > > +++ b/drivers/cxl/pmem.c > > @@ -6,7 +6,7 @@ > > #include <linux/ndctl.h> > > #include <linux/async.h> > > #include <linux/slab.h> > > -#include "mem.h" > > +#include "cxlmem.h" > > #include "cxl.h" > > > > /* > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH AUTOSEL 5.14 03/25] cxl: Move cxl_core to new directory 2021-09-14 8:57 ` Jonathan Cameron @ 2021-09-14 15:05 ` Ben Widawsky 2021-09-14 15:40 ` Dan Williams 2021-09-14 17:00 ` Sasha Levin 0 siblings, 2 replies; 6+ messages in thread From: Ben Widawsky @ 2021-09-14 15:05 UTC (permalink / raw) To: Jonathan Cameron Cc: Sasha Levin, linux-kernel, stable, kernel test robot, Dan Williams, linux-doc, linux-cxl On 21-09-14 09:57:49, Jonathan Cameron wrote: > On Tue, 14 Sep 2021 09:56:23 +0100 > Jonathan Cameron <Jonathan.Cameron@Huawei.com> wrote: > > > On Mon, 13 Sep 2021 18:33:17 -0400 > > Sasha Levin <sashal@kernel.org> wrote: > > > > > From: Ben Widawsky <ben.widawsky@intel.com> > > > > > > [ Upstream commit 5161a55c069f53d88da49274cbef6e3c74eadea9 ] > > > > > > CXL core is growing, and it's already arguably unmanageable. To support > > > future growth, move core functionality to a new directory and rename the > > > file to represent just bus support. Future work will remove non-bus > > > functionality. > > > > > > Note that mem.h is renamed to cxlmem.h to avoid a namespace collision > > > with the global ARCH=um mem.h header. > > > > Not a fix... > > > > I'm guessing this got picked up on the basis of the Reported-by: tag? > > I think that was added for a minor tweak as this went through review rather > > than referring to the whole patch. > Or possibly because it was a precursor to the fix in the next patch. > > Hmm. Ben, Dan, does it make sense for these two to go into stable? > > Jonathan As of now, no, but having this will make future fixes much easier to cherry pick. > > > > > Jonathan > > > > > > > > > > Reported-by: kernel test robot <lkp@intel.com> > > > Signed-off-by: Ben Widawsky <ben.widawsky@intel.com> > > > Reviewed-by: Jonathan Cameron <Jonathan.Cameron@huawei.com> > > > Link: https://lore.kernel.org/r/162792537866.368511.8915631504621088321.stgit@dwillia2-desk3.amr.corp.intel.com > > > Signed-off-by: Dan Williams <dan.j.williams@intel.com> > > > Signed-off-by: Sasha Levin <sashal@kernel.org> > > > --- > > > Documentation/driver-api/cxl/memory-devices.rst | 2 +- > > > drivers/cxl/Makefile | 4 +--- > > > drivers/cxl/core/Makefile | 5 +++++ > > > drivers/cxl/{core.c => core/bus.c} | 4 ++-- > > > drivers/cxl/{mem.h => cxlmem.h} | 0 > > > drivers/cxl/pci.c | 2 +- > > > drivers/cxl/pmem.c | 2 +- > > > 7 files changed, 11 insertions(+), 8 deletions(-) > > > create mode 100644 drivers/cxl/core/Makefile > > > rename drivers/cxl/{core.c => core/bus.c} (99%) > > > rename drivers/cxl/{mem.h => cxlmem.h} (100%) > > > > > > diff --git a/Documentation/driver-api/cxl/memory-devices.rst b/Documentation/driver-api/cxl/memory-devices.rst > > > index 487ce4f41d77..a86e2c7c551a 100644 > > > --- a/Documentation/driver-api/cxl/memory-devices.rst > > > +++ b/Documentation/driver-api/cxl/memory-devices.rst > > > @@ -36,7 +36,7 @@ CXL Core > > > .. kernel-doc:: drivers/cxl/cxl.h > > > :internal: > > > > > > -.. kernel-doc:: drivers/cxl/core.c > > > +.. kernel-doc:: drivers/cxl/core/bus.c > > > :doc: cxl core > > > > > > External Interfaces > > > diff --git a/drivers/cxl/Makefile b/drivers/cxl/Makefile > > > index 32954059b37b..d1aaabc940f3 100644 > > > --- a/drivers/cxl/Makefile > > > +++ b/drivers/cxl/Makefile > > > @@ -1,11 +1,9 @@ > > > # SPDX-License-Identifier: GPL-2.0 > > > -obj-$(CONFIG_CXL_BUS) += cxl_core.o > > > +obj-$(CONFIG_CXL_BUS) += core/ > > > obj-$(CONFIG_CXL_MEM) += cxl_pci.o > > > obj-$(CONFIG_CXL_ACPI) += cxl_acpi.o > > > obj-$(CONFIG_CXL_PMEM) += cxl_pmem.o > > > > > > -ccflags-y += -DDEFAULT_SYMBOL_NAMESPACE=CXL > > > -cxl_core-y := core.o > > > cxl_pci-y := pci.o > > > cxl_acpi-y := acpi.o > > > cxl_pmem-y := pmem.o > > > diff --git a/drivers/cxl/core/Makefile b/drivers/cxl/core/Makefile > > > new file mode 100644 > > > index 000000000000..ad137f96e5c8 > > > --- /dev/null > > > +++ b/drivers/cxl/core/Makefile > > > @@ -0,0 +1,5 @@ > > > +# SPDX-License-Identifier: GPL-2.0 > > > +obj-$(CONFIG_CXL_BUS) += cxl_core.o > > > + > > > +ccflags-y += -DDEFAULT_SYMBOL_NAMESPACE=CXL -I$(srctree)/drivers/cxl > > > +cxl_core-y := bus.o > > > diff --git a/drivers/cxl/core.c b/drivers/cxl/core/bus.c > > > similarity index 99% > > > rename from drivers/cxl/core.c > > > rename to drivers/cxl/core/bus.c > > > index a2e4d54fc7bc..0815eec23944 100644 > > > --- a/drivers/cxl/core.c > > > +++ b/drivers/cxl/core/bus.c > > > @@ -6,8 +6,8 @@ > > > #include <linux/pci.h> > > > #include <linux/slab.h> > > > #include <linux/idr.h> > > > -#include "cxl.h" > > > -#include "mem.h" > > > +#include <cxlmem.h> > > > +#include <cxl.h> > > > > > > /** > > > * DOC: cxl core > > > diff --git a/drivers/cxl/mem.h b/drivers/cxl/cxlmem.h > > > similarity index 100% > > > rename from drivers/cxl/mem.h > > > rename to drivers/cxl/cxlmem.h > > > diff --git a/drivers/cxl/pci.c b/drivers/cxl/pci.c > > > index 4cf351a3cf99..a945c5fda292 100644 > > > --- a/drivers/cxl/pci.c > > > +++ b/drivers/cxl/pci.c > > > @@ -12,9 +12,9 @@ > > > #include <linux/pci.h> > > > #include <linux/io.h> > > > #include <linux/io-64-nonatomic-lo-hi.h> > > > +#include "cxlmem.h" > > > #include "pci.h" > > > #include "cxl.h" > > > -#include "mem.h" > > > > > > /** > > > * DOC: cxl pci > > > diff --git a/drivers/cxl/pmem.c b/drivers/cxl/pmem.c > > > index 0088e41dd2f3..9652c3ee41e7 100644 > > > --- a/drivers/cxl/pmem.c > > > +++ b/drivers/cxl/pmem.c > > > @@ -6,7 +6,7 @@ > > > #include <linux/ndctl.h> > > > #include <linux/async.h> > > > #include <linux/slab.h> > > > -#include "mem.h" > > > +#include "cxlmem.h" > > > #include "cxl.h" > > > > > > /* > > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH AUTOSEL 5.14 03/25] cxl: Move cxl_core to new directory 2021-09-14 15:05 ` Ben Widawsky @ 2021-09-14 15:40 ` Dan Williams 2021-09-14 17:00 ` Sasha Levin 1 sibling, 0 replies; 6+ messages in thread From: Dan Williams @ 2021-09-14 15:40 UTC (permalink / raw) To: Sasha Levin Cc: Jonathan Cameron, Linux Kernel Mailing List, stable, kernel test robot, Linux Doc Mailing List, linux-cxl, Ben Widawsky On Tue, Sep 14, 2021 at 8:06 AM Ben Widawsky <ben.widawsky@intel.com> wrote: > > On 21-09-14 09:57:49, Jonathan Cameron wrote: > > On Tue, 14 Sep 2021 09:56:23 +0100 > > Jonathan Cameron <Jonathan.Cameron@Huawei.com> wrote: > > > > > On Mon, 13 Sep 2021 18:33:17 -0400 > > > Sasha Levin <sashal@kernel.org> wrote: > > > > > > > From: Ben Widawsky <ben.widawsky@intel.com> > > > > > > > > [ Upstream commit 5161a55c069f53d88da49274cbef6e3c74eadea9 ] > > > > > > > > CXL core is growing, and it's already arguably unmanageable. To support > > > > future growth, move core functionality to a new directory and rename the > > > > file to represent just bus support. Future work will remove non-bus > > > > functionality. > > > > > > > > Note that mem.h is renamed to cxlmem.h to avoid a namespace collision > > > > with the global ARCH=um mem.h header. > > > > > > Not a fix... > > > > > > I'm guessing this got picked up on the basis of the Reported-by: tag? > > > I think that was added for a minor tweak as this went through review rather > > > than referring to the whole patch. > > Or possibly because it was a precursor to the fix in the next patch. > > > > Hmm. Ben, Dan, does it make sense for these two to go into stable? > > > > Jonathan > > As of now, no, but having this will make future fixes much easier to cherry > pick. Sasha, please drop this. The CXL subsystem is still in major feature development. I would rather manually backport small fixes rather than backport major code movement just to make small fix backport easier. Let me know if there was a specific fix autosel was trying to resolve and we'll take a look at whether it makes sense to do a custom backport for -stable. ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH AUTOSEL 5.14 03/25] cxl: Move cxl_core to new directory 2021-09-14 15:05 ` Ben Widawsky 2021-09-14 15:40 ` Dan Williams @ 2021-09-14 17:00 ` Sasha Levin 1 sibling, 0 replies; 6+ messages in thread From: Sasha Levin @ 2021-09-14 17:00 UTC (permalink / raw) To: Ben Widawsky Cc: Jonathan Cameron, linux-kernel, stable, kernel test robot, Dan Williams, linux-doc, linux-cxl On Tue, Sep 14, 2021 at 08:05:58AM -0700, Ben Widawsky wrote: >On 21-09-14 09:57:49, Jonathan Cameron wrote: >> On Tue, 14 Sep 2021 09:56:23 +0100 >> Jonathan Cameron <Jonathan.Cameron@Huawei.com> wrote: >> >> > On Mon, 13 Sep 2021 18:33:17 -0400 >> > Sasha Levin <sashal@kernel.org> wrote: >> > >> > > From: Ben Widawsky <ben.widawsky@intel.com> >> > > >> > > [ Upstream commit 5161a55c069f53d88da49274cbef6e3c74eadea9 ] >> > > >> > > CXL core is growing, and it's already arguably unmanageable. To support >> > > future growth, move core functionality to a new directory and rename the >> > > file to represent just bus support. Future work will remove non-bus >> > > functionality. >> > > >> > > Note that mem.h is renamed to cxlmem.h to avoid a namespace collision >> > > with the global ARCH=um mem.h header. >> > >> > Not a fix... >> > >> > I'm guessing this got picked up on the basis of the Reported-by: tag? >> > I think that was added for a minor tweak as this went through review rather >> > than referring to the whole patch. >> Or possibly because it was a precursor to the fix in the next patch. >> >> Hmm. Ben, Dan, does it make sense for these two to go into stable? >> >> Jonathan > >As of now, no, but having this will make future fixes much easier to cherry It was picked because the next patch depends on it. I'll just drop both if you don't want the next one. Thanks! -- Thanks, Sasha ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-09-14 17:00 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20210913223339.435347-1-sashal@kernel.org>
2021-09-13 22:33 ` [PATCH AUTOSEL 5.14 03/25] cxl: Move cxl_core to new directory Sasha Levin
2021-09-14 8:56 ` Jonathan Cameron
2021-09-14 8:57 ` Jonathan Cameron
2021-09-14 15:05 ` Ben Widawsky
2021-09-14 15:40 ` Dan Williams
2021-09-14 17:00 ` Sasha Levin
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).