Skip to content

cxl: dport fixes from sashiko reports

Dave Jiang linux-cxl sashiko-reviews stable 4 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v4 Latest3 patches 4 repliesIn review
  2. v33 patches 5 repliesSuperseded
  3. v22 patches 5 repliesSuperseded
Earlier versions · 1
VersionPostedPatchesRepliesStatus
v12 patches2 repliesSuperseded

Changes in v4 from the cover letter

- Drop the unused dport output parameter in patch 2. (Sashiko, Alison)
- Add review tags from Alison and Ming.

Patches in v4 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3cxl/port: Clear cached dport pointers when a dport is removeddrivers/cxl/core/port.c+29 -00 replies
2/3cxl/core: Hold the dport host lock across dport lookup and usedrivers/cxl/core/core.h drivers/cxl/core/pci.c drivers/cxl/core/port.c +4 more+95 -401 reply
3/3cxl/port: Attach endpoints to a dport under the lock that pins itdrivers/cxl/core/port.c+62 -531 reply

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by.

PatchJonathan CameronLi MingGregory Price (Meta)Alison Schofield
1/3R-b R-b R-b R-b
2/3R-b R-b
3/3R-b R-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 4 reviewers.

Read the v4 thread

Cover letter [PATCH v4 0/3] cxl: dport fixes from sashiko reports

Couple dport fixes resulted from sashiko pre-existing issue reports. There are no depedencies between the two patches.

v4:
- Drop the unused dport output parameter in patch 2. (Sashiko, Alison)
- Add review tags from Alison and Ming.

v3:
- Add patch 3 to attach endpoints under the dport's port lock. (Ming)
- Add review tags from Ming and Gregory to patch 1.

v2:
- Addressed couple things from Jonathan. See patch for revlog.

Dave Jiang (3):
cxl/port: Clear cached dport pointers when a dport is removed cxl/core: Hold the dport host lock across dport lookup and use cxl/port: Attach endpoints to a dport under the lock that pins it

 drivers/cxl/core/core.h    |   6 +-
 drivers/cxl/core/pci.c     |   8 +-
 drivers/cxl/core/port.c    | 206 ++++++++++++++++++++++++-------------
 drivers/cxl/core/ras_rch.c |  12 ++-
 drivers/cxl/cxl.h          |  22 +++-
 drivers/cxl/mem.c          |  14 ++-
 drivers/cxl/pci.c          |  11 +-
 7 files changed, 186 insertions(+), 93 deletions(-)

base-commit: fd73f4a6659897191fa0d40695fe370925dd3780

Signature · 1 line
2.54.0