Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Split transport specific files #157

Closed
paravmellanox opened this issue Feb 3, 2023 · 1 comment
Closed

Split transport specific files #157

paravmellanox opened this issue Feb 3, 2023 · 1 comment
Labels
virtio-v1.3-cs01

Comments

@paravmellanox
Copy link
Contributor

paravmellanox commented Feb 3, 2023 •
edited
Loading

Problem statement:
Currently several tens of pages worth of transport specific content 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.

Link to patches: https://lists.oasis-open.org/archives/virtio-dev/202302/msg00578.html
@mstsirkin
Copy link
Contributor

BALLOT CREATED AT URL: https://www.oasis-open.org/committees/ballot.php?id=3763

@cohuck cohuck closed this as completed in 91a4699 Mar 10, 2023
cohuck pushed a commit that referenced this issue Mar 10, 2023
@paravmellanox @cohuck
transport-mmio: Split MMIO transport to its own file
9e88ba9 
Place MMIO transport specification in its own file to better maintain it.

Fixes: #157

Signed-off-by: Parav Pandit <[email protected]>
Signed-off-by: Cornelia Huck <[email protected]>
cohuck pushed a commit that referenced this issue Mar 10, 2023
@paravmellanox @cohuck
transport-ccw: Split Channel IO transport to its own file
0af264f 
Place Channel IO transport specification in its own file to
better maintain it.

Fixes: #157

Signed-off-by: Parav Pandit <[email protected]>
Signed-off-by: Cornelia Huck <[email protected]>
cohuck pushed a commit that referenced this issue Mar 10, 2023
@paravmellanox @cohuck
transport-pci: Fix spellings and white spaces
deb0aa0 
Now that we have individual files, fix reported spelling errors.

While at it, remove trailing white spaces.

Fixes: #157

Signed-off-by: Parav Pandit <[email protected]>
Signed-off-by: Cornelia Huck <[email protected]>
cohuck pushed a commit that referenced this issue Mar 10, 2023
@paravmellanox @cohuck
transport-mmio: Fix spellings and white spaces
ca97719 
Now that we have individual files, fix reported spelling errors.

While at it, remove trailing white spaces.

Fixes: #157

Signed-off-by: Parav Pandit <[email protected]>
Signed-off-by: Cornelia Huck <[email protected]>
cohuck pushed a commit that referenced this issue Mar 10, 2023
@paravmellanox @cohuck
transport-ccw: Fix spellings and white spaces
8797f4d 
Now that we have individual files, fix reported spelling errors.

While at it, remove extra white spaces.

Fixes: #157

Signed-off-by: Parav Pandit <[email protected]>
Signed-off-by: Cornelia Huck <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
virtio-v1.3-cs01
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mstsirkin @paravmellanox