Skip to content

Split transport specific files

Parav Pandit virtio-comment virtio-dev 5 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v5 Latest6 patches 2 repliesIn review
  2. v46 patches 0 repliesSuperseded
  3. v36 patches 0 repliesSuperseded
Earlier versions · 2
VersionPostedPatchesRepliesStatus
v26 patches0 repliesSuperseded
v16 patches4 repliesSuperseded

Changes in v5 from the cover letter

- fix mmio few lines hunk was in wrong pci file v3->v4:
- added github issue number fixes tag to patches 4 to 6 v2->v3:
- updated subject for patches 4 to 6

Patches in v5 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6transport-pci: Split PCI transport to its own filecontent.tex transport-pci.tex+1150 -11480 replies
2/6transport-mmio: Split MMIO transport to its own filecontent.tex transport-mmio.tex+565 -5650 replies
3/6transport-ccw: Split Channel IO transport to its own filecontent.tex transport-ccw.tex+602 -6020 replies
4/6transport-pci: Fix spellings and white spacestransport-pci.tex+6 -60 replies
5/6transport-mmio: Fix spellings and white spacestransport-mmio.tex+45 -450 replies
6/6transport-ccw: Fix spellings and white spacestransport-ccw.tex+5 -50 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v5 yet.

Read the v5 thread

Cover letter [PATCH v5 0/6] Split transport specific files

Problem statement:
Currently several tens of pages worth of transport specification is situated in single file. Many parts of the specification are already maintained in separate files.

Such separate files enables better maintenance of the specification overall.

Solution:
Follow the approach similar to rest of the spec, and keep transport specific spec in their individual files.

This series is for fixing issue [1].

[1] Fixes: https://github.com/oasis-tcs/virtio-spec/issues/157

Please review.

Patch summary:
patch 1 moves pci transport to its own file
patch 2 moves mmio transport to its own file
patch 3 moves channel io transport to its own file patch 4 to 6 fixes spelling errors existed in the spec which are discovered during splitting the files

changelog:
v4->v5:
- fix mmio few lines hunk was in wrong pci file v3->v4:
- added github issue number fixes tag to patches 4 to 6 v2->v3:
- updated subject for patches 4 to 6
v1->v2:
- renamed transport-channelio.tex to transport-ccw.tex
- removed trailing white spaces in transport-ccw.tex

Parav Pandit (6):
transport-pci: Split PCI transport to its own file
transport-mmio: Split MMIO transport to its own file
transport-ccw: Split Channel IO transport to its own file
transport-pci: Fix spellings and white spaces
transport-mmio: Fix spellings and white spaces
transport-ccw: Fix spellings and white spaces

 content.tex        | 2318 +-------------------------------------------
 transport-ccw.tex  |  601 ++++++++++++
 transport-mmio.tex |  564 +++++++++++
 transport-pci.tex  | 1149 ++++++++++++++++++++++
 4 files changed, 2317 insertions(+), 2315 deletions(-)
 create mode 100644 transport-ccw.tex
 create mode 100644 transport-mmio.tex
 create mode 100644 transport-pci.tex
Signature · 1 line
2.26.2