Discussion of the implementations of VIRTIO specification
 help / color / mirror / Atom feed
* [PATCH v5 0/6] Split transport specific files
@ 2023-02-24 22:22 Parav Pandit
  2023-02-24 22:22 ` [virtio-dev] " Parav Pandit
                   ` (7 more replies)
  0 siblings, 8 replies; 16+ messages in thread
From: Parav Pandit @ 2023-02-24 22:22 UTC (permalink / raw)
  To: mst, virtio-dev, cohuck; +Cc: virtio-comment, shahafs, Parav Pandit

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

-- 
2.26.2


^ permalink raw reply	[flat|nested] 16+ messages in thread

end of thread, other threads:[~2023-03-03  7:45 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-02-24 22:22 [PATCH v5 0/6] Split transport specific files Parav Pandit
2023-02-24 22:22 ` [virtio-dev] " Parav Pandit
2023-02-24 22:22 ` [PATCH v5 1/6] transport-pci: Split PCI transport to its own file Parav Pandit
2023-02-24 22:22   ` [virtio-dev] " Parav Pandit
2023-02-24 22:22 ` [PATCH v5 2/6] transport-mmio: Split MMIO " Parav Pandit
2023-02-24 22:22   ` [virtio-dev] " Parav Pandit
2023-02-24 22:22 ` [PATCH v5 3/6] transport-ccw: Split Channel IO " Parav Pandit
2023-02-24 22:22   ` [virtio-dev] " Parav Pandit
2023-02-24 22:22 ` [PATCH v5 4/6] transport-pci: Fix spellings and white spaces Parav Pandit
2023-02-24 22:22   ` [virtio-dev] " Parav Pandit
2023-02-24 22:22 ` [PATCH v5 5/6] transport-mmio: " Parav Pandit
2023-02-24 22:22   ` [virtio-dev] " Parav Pandit
2023-02-24 22:22 ` [PATCH v5 6/6] transport-ccw: " Parav Pandit
2023-02-24 22:22   ` [virtio-dev] " Parav Pandit
2023-03-02 15:10 ` [virtio-dev] RE: [PATCH v5 0/6] Split transport specific files Parav Pandit
2023-03-03  7:45   ` [virtio-dev] " Michael S. Tsirkin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox