All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Williams <dan.j.williams@intel.com>
To: linux-cxl@vger.kernel.org
Cc: Jonathan Cameron <Jonathan.Cameron@huawei.com>,
	kernel test robot <lkp@intel.com>,
	Ben Widawsky <ben.widawsky@intel.com>,
	Jonathan.Cameron@huawei.com, vishal.l.verma@intel.com,
	alison.schofield@intel.com
Subject: [PATCH v5 0/6] CXL core reorganization
Date: Mon, 02 Aug 2021 10:29:33 -0700	[thread overview]
Message-ID: <162792537312.368511.15862521152871631343.stgit@dwillia2-desk3.amr.corp.intel.com> (raw)

Changes since v4 [1]:
- Fix ARCH=um compile errors reported by 0day caused by header
  filename collisions with "mem.h", moved to "cxlmem.h"

- Improve kdoc note for @fops attribute of 'struct
  cdevm_file_operations' (Jonathan)

- Include core.h in memdev.c to fix "no prior declaration" warnings
  (kbuild robot)

- Collect reviewed-by's from Jonathan

[1]: https://lore.kernel.org/r/162774927110.4094313.9743731504096769799.stgit@dwillia2-desk3.amr.corp.intel.com

---

Original Cover from Ben:

The main motivation of the patch series is to establish the cxl_core driver in
its own directory and modularize it. Specifically, the patch series aims to
achieve three things:
1. Move existing core functionality to a new directory.
2. Split existing core functionality into multiple files.
3. Migrate memdev functionality into core.

#1 is trivially accomplished with git mv. The file itself is renamed back to
bus.c since the goal is to break up core functionality into multiple files, and
so the name core.c doesn't make sense in that context.

#2 is also trivially accomplished via cut/paste.

#3 is slightly invasive in that it has certain functional changes to improve the
existing interfaces and make them more generic. The rest of the change is
cut/paste. This is also the only part of the series which has runtime functional
change in that some interfaces are removed from cxl_pci, moved into cxl_core,
and exported for other drivers to use.

---

Ben Widawsky (3):
      cxl: Move cxl_core to new directory
      cxl/core: Improve CXL core kernel docs
      cxl/core: Move memdev management to core

Dan Williams (3):
      cxl/core: Move pmem functionality
      cxl/core: Move register mapping infrastructure
      cxl/pci: Introduce cdevm_file_operations


 Documentation/driver-api/cxl/memory-devices.rst |    8 
 drivers/cxl/Makefile                            |    4 
 drivers/cxl/core/Makefile                       |    8 
 drivers/cxl/core/bus.c                          |  464 +----------------------
 drivers/cxl/core/core.h                         |   20 +
 drivers/cxl/core/memdev.c                       |  246 ++++++++++++
 drivers/cxl/core/pmem.c                         |  204 ++++++++++
 drivers/cxl/core/regs.c                         |  235 ++++++++++++
 drivers/cxl/cxlmem.h                            |   26 +
 drivers/cxl/pci.c                               |  259 +------------
 drivers/cxl/pmem.c                              |    2 
 11 files changed, 795 insertions(+), 681 deletions(-)
 create mode 100644 drivers/cxl/core/Makefile
 rename drivers/cxl/{core.c => core/bus.c} (58%)
 create mode 100644 drivers/cxl/core/core.h
 create mode 100644 drivers/cxl/core/memdev.c
 create mode 100644 drivers/cxl/core/pmem.c
 create mode 100644 drivers/cxl/core/regs.c
 rename drivers/cxl/{mem.h => cxlmem.h} (76%)

base-commit: ff1176468d368232b684f75e82563369208bc371

             reply	other threads:[~2021-08-02 17:29 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-02 17:29 Dan Williams [this message]
2021-08-02 17:29 ` [PATCH v5 1/6] cxl: Move cxl_core to new directory Dan Williams
2021-08-02 17:29 ` [PATCH v5 2/6] cxl/core: Improve CXL core kernel docs Dan Williams
2021-08-02 17:29 ` [PATCH v5 3/6] cxl/core: Move pmem functionality Dan Williams
2021-08-02 17:29 ` [PATCH v5 4/6] cxl/core: Move register mapping infrastructure Dan Williams
2021-08-03 14:25   ` [PATCH v6 " Dan Williams
2021-08-02 17:29 ` [PATCH v5 5/6] cxl/pci: Introduce cdevm_file_operations Dan Williams
2021-08-02 17:30 ` [PATCH v5 6/6] cxl/core: Move memdev management to core Dan Williams

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=162792537312.368511.15862521152871631343.stgit@dwillia2-desk3.amr.corp.intel.com \
    --to=dan.j.williams@intel.com \
    --cc=Jonathan.Cameron@huawei.com \
    --cc=alison.schofield@intel.com \
    --cc=ben.widawsky@intel.com \
    --cc=linux-cxl@vger.kernel.org \
    --cc=lkp@intel.com \
    --cc=vishal.l.verma@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.