From: Lukas Wunner <lukas@wunner.de>
To: Bjorn Helgaas <helgaas@kernel.org>,
Dan Williams <dan.j.williams@intel.com>,
linux-pci@vger.kernel.org, linux-cxl@vger.kernel.org
Cc: Gregory Price <gregory.price@memverge.com>,
Ira Weiny <ira.weiny@intel.com>,
Jonathan Cameron <Jonathan.Cameron@huawei.com>,
Alison Schofield <alison.schofield@intel.com>,
Vishal Verma <vishal.l.verma@intel.com>,
Dave Jiang <dave.jiang@intel.com>,
"Li, Ming" <ming4.li@intel.com>, Hillf Danton <hdanton@sina.com>,
Ben Widawsky <bwidawsk@kernel.org>,
Alexey Kardashevskiy <aik@amd.com>,
Davidlohr Bueso <dave@stgolabs.net>,
linuxarm@huawei.com
Subject: [PATCH v4 00/17] Collection of DOE material
Date: Sat, 11 Mar 2023 15:40:00 +0100 [thread overview]
Message-ID: <cover.1678543498.git.lukas@wunner.de> (raw)
Collection of DOE material, v4:
Migrate to synchronous API, create mailboxes in PCI core instead of
in drivers, relax restrictions on request/response size.
This should probably go in via the cxl tree because Dave Jiang is
basing his cxl work for the next merge window on it.
The first 6 patches are fixes, so could be applied immediately.
Thanks!
Changes v3 -> v4:
* [PATCH v4 01/17] cxl/pci: Fix CDAT retrieval on big endian
* In pci_doe_discovery(), add request_pl_le / response_pl_le variables
to avoid typecasts in pci_doe_task initializer (Jonathan)
* In cxl_cdat_read_table(), use __le32 instead of u32 for "*data"
variable (Jonathan)
* Use sizeof(__le32) instead of sizeof(u32) (Jonathan)
* [PATCH v4 03/17] cxl/pci: Handle truncated CDAT entries
* Check for sizeof(*entry) instead of sizeof(struct cdat_entry_header)
for clarity (Jonathan)
* [PATCH v4 12/17] PCI/DOE: Create mailboxes on device enumeration
* Amend commit message with additional justification for the commit
(Alexey)
* [PATCH v4 16/17] cxl/pci: cxl/pci: Simplify CDAT retrieval error path
* Newly added patch in v4 on popular request (Jonathan, Dave)
* [PATCH v4 17/17] cxl/pci: Rightsize CDAT response allocation
* Amend commit message with spec reference to the Table Access
Response Header (Ira)
* In cxl_cdat_get_length(), check for sizeof(response) instead of
2 * sizeof(u32) for clarity
Link to v3:
https://lore.kernel.org/linux-pci/cover.1676043318.git.lukas@wunner.de/
Dave Jiang (1):
cxl/pci: Simplify CDAT retrieval error path
Lukas Wunner (16):
cxl/pci: Fix CDAT retrieval on big endian
cxl/pci: Handle truncated CDAT header
cxl/pci: Handle truncated CDAT entries
cxl/pci: Handle excessive CDAT length
PCI/DOE: Silence WARN splat with CONFIG_DEBUG_OBJECTS=y
PCI/DOE: Fix memory leak with CONFIG_DEBUG_OBJECTS=y
PCI/DOE: Provide synchronous API and use it internally
cxl/pci: Use synchronous API for DOE
PCI/DOE: Make asynchronous API private
PCI/DOE: Deduplicate mailbox flushing
PCI/DOE: Allow mailbox creation without devres management
PCI/DOE: Create mailboxes on device enumeration
cxl/pci: Use CDAT DOE mailbox created by PCI core
PCI/DOE: Make mailbox creation API private
PCI/DOE: Relax restrictions on request and response size
cxl/pci: Rightsize CDAT response allocation
.clang-format | 1 -
drivers/cxl/core/pci.c | 140 +++++++---------
drivers/cxl/cxlmem.h | 3 -
drivers/cxl/cxlpci.h | 14 ++
drivers/cxl/pci.c | 49 ------
drivers/pci/doe.c | 342 ++++++++++++++++++++++++++++++----------
drivers/pci/pci.h | 11 ++
drivers/pci/probe.c | 1 +
drivers/pci/remove.c | 1 +
include/linux/pci-doe.h | 62 +-------
include/linux/pci.h | 3 +
11 files changed, 350 insertions(+), 277 deletions(-)
--
2.39.1
next reply other threads:[~2023-03-11 14:44 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-03-11 14:40 Lukas Wunner [this message]
2023-03-11 14:40 ` [PATCH v4 01/17] cxl/pci: Fix CDAT retrieval on big endian Lukas Wunner
2023-03-15 16:39 ` Jonathan Cameron
2023-03-11 14:40 ` [PATCH v4 02/17] cxl/pci: Handle truncated CDAT header Lukas Wunner
2023-03-13 2:42 ` Sathyanarayanan Kuppuswamy
2023-03-11 14:40 ` [PATCH v4 03/17] cxl/pci: Handle truncated CDAT entries Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 04/17] cxl/pci: Handle excessive CDAT length Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 05/17] PCI/DOE: Silence WARN splat with CONFIG_DEBUG_OBJECTS=y Lukas Wunner
2023-03-21 3:42 ` Alexey Kardashevskiy
2023-03-21 9:05 ` Jonathan Cameron
2023-04-04 9:01 ` Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 06/17] PCI/DOE: Fix memory leak " Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 07/17] PCI/DOE: Provide synchronous API and use it internally Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 08/17] cxl/pci: Use synchronous API for DOE Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 09/17] PCI/DOE: Make asynchronous API private Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 10/17] PCI/DOE: Deduplicate mailbox flushing Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 11/17] PCI/DOE: Allow mailbox creation without devres management Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 12/17] PCI/DOE: Create mailboxes on device enumeration Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 13/17] cxl/pci: Use CDAT DOE mailbox created by PCI core Lukas Wunner
2023-04-22 2:51 ` [PATCH] cxl/port: Fix port to pci device assumptions in read_cdat_data() Dan Williams
2023-04-22 8:35 ` Lukas Wunner
2023-04-22 14:05 ` Lukas Wunner
2023-04-22 20:54 ` Dan Williams
2023-04-22 22:30 ` Lukas Wunner
2023-04-22 23:22 ` Dan Williams
2023-04-23 8:19 ` Lukas Wunner
2023-04-22 20:56 ` Dan Williams
2023-04-23 14:58 ` Jonathan Cameron
2023-04-23 15:07 ` Jonathan Cameron
2023-04-23 18:32 ` Dan Williams
2023-03-11 14:40 ` [PATCH v4 14/17] PCI/DOE: Make mailbox creation API private Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 15/17] PCI/DOE: Relax restrictions on request and response size Lukas Wunner
2023-03-11 14:40 ` [PATCH v4 16/17] cxl/pci: Simplify CDAT retrieval error path Lukas Wunner
2023-03-15 16:48 ` Jonathan Cameron
2023-03-11 14:40 ` [PATCH v4 17/17] cxl/pci: Rightsize CDAT response allocation Lukas Wunner
2023-03-13 19:55 ` [PATCH v4 00/17] Collection of DOE material Bjorn Helgaas
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=cover.1678543498.git.lukas@wunner.de \
--to=lukas@wunner.de \
--cc=Jonathan.Cameron@huawei.com \
--cc=aik@amd.com \
--cc=alison.schofield@intel.com \
--cc=bwidawsk@kernel.org \
--cc=dan.j.williams@intel.com \
--cc=dave.jiang@intel.com \
--cc=dave@stgolabs.net \
--cc=gregory.price@memverge.com \
--cc=hdanton@sina.com \
--cc=helgaas@kernel.org \
--cc=ira.weiny@intel.com \
--cc=linux-cxl@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=linuxarm@huawei.com \
--cc=ming4.li@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox