All of lore.kernel.org
 help / color / mirror / Atom feed
* [virtio-comment] [PATCH v12 00/10] Introduce device group and device management
@ 2023-04-24 16:44 ` Michael S. Tsirkin
  0 siblings, 0 replies; 80+ messages in thread
From: Michael S. Tsirkin @ 2023-04-24 16:44 UTC (permalink / raw)
  To: virtio-comment, virtio-dev, jasowang, mst, cohuck, sgarzare,
	stefanha, nrupal.jani, Piotr.Uminski, hang.yuan
  Cc: virtio, Jiri Pirko, Zhu Lingshan, pasic, Shahaf Shuler,
	Parav Pandit, Max Gurtovoy

comments on previous one have all been minor, so I hope
this means we are close to merging this.

Change log:

since 11:
	addressed lots of comments, all minor. consistency with
	outstanding number->index and queue->enqueue work
	i did not intentionally drop any reviewed-by tags
	as all changes are minor - if yours is missing it is
	because I forgot to record it, sorry

	one "breaking" change in response to stefan's comment:
	in patch 5, num_queues has been specified not to include admin
	queues: just regular ones.

since v10:
	addressed lots of comments by Jiri, Stefan. Cornelia, Lngshan, Parav, Max

since v9:
	addressed comments by Parav, Max, Cornelia, David and Zhu Lingshan:
		added link to errno header from Linux
		rename _MEM to _MEMBER
		admin vq num is zero based
		clarify who sends commands where
		minor english tweaks
		clarify command length
		specify interaction with sriov capability
		correct commit log - NumVFs can be 0

	i could not decide what should happen when VFs are
	disabled. for now did not specify.

since v8:
	addressed comments by Cornelia - as we agreed on list
	
since v7:
	make high level error codes match linux, with virtio specific codes
		in a separate field
	renamed _ACCEPT to _USE since that's what it does
	clarified forward compatibility and non pci transports
	support multiple admin vqs
	conformance statements
	lots of changes all over the place to I changed author from Max
	to myself. Don't need to take credit but also don't want
	to blame Max for my mistakes.

since v6:

	- removed some extentions intended for future use.
	  We'll do them when we get there.

	- brought back command list query from v5 in a simplified form -
	  it's here to address the case where a single parent
	  can address multiple groups, such as PF addressing
	  transport vq and sriov vfs.

	- attempt to make terminology more formal.
	In particular a term for whoever controls the group.
 	I am still going back
	and forth between "parent" and "owner" - owner might
	be better after all since it will work if we ever
	have a self group. For now it's parent.

TODO (maybe?) - probably ok to defer until this part is upstream:

	Add "all members" member id.

	Add commands for MSI, feature discovery.

	Add commands for transport vq.


My intent is to try and support both SR-IOV and SIOV
usecases with the same structure and maybe even the same
VQ.

For example, it might make sense to split creating/destroying
SIOV devices from the transport passing data from the guest - the
driver would then not negotiate VIRTIO_F_SR_IOV (which
then means auto-provisioning).

More ideas for use-cases:
virtio VF features query and configuration space provisioning
virtio VF resource (queues, msix vectors count) provisioning


Future directions (shouldn't block this patch)
- aborting commands - left for later. or is vq reset enough?
- should we rename structures from admin to group admin?


Michael S. Tsirkin (10):
  virtio: document forward compatibility guarantees
  admin: introduce device group and related concepts
  admin: introduce group administration commands
  admin: introduce virtio admin virtqueues
  pci: add admin vq registers to virtio over pci
  mmio: document ADMIN_VQ as reserved
  ccw: document ADMIN_VQ as reserved
  admin: command list discovery
  admin: conformance clauses
  ccw: document more reserved features

 admin.tex        | 587 +++++++++++++++++++++++++++++++++++++++++++++++
 content.tex      | 121 +++++++++-
 introduction.tex |   3 +
 3 files changed, 709 insertions(+), 2 deletions(-)
 create mode 100644 admin.tex

-- 
MST


This publicly archived list offers a means to provide input to the
OASIS Virtual I/O Device (VIRTIO) TC.

In order to verify user consent to the Feedback License terms and
to minimize spam in the list archive, subscription is required
before posting.

Subscribe: virtio-comment-subscribe@lists.oasis-open.org
Unsubscribe: virtio-comment-unsubscribe@lists.oasis-open.org
List help: virtio-comment-help@lists.oasis-open.org
List archive: https://lists.oasis-open.org/archives/virtio-comment/
Feedback License: https://www.oasis-open.org/who/ipr/feedback_license.pdf
List Guidelines: https://www.oasis-open.org/policies-guidelines/mailing-lists
Committee: https://www.oasis-open.org/committees/virtio/
Join OASIS: https://www.oasis-open.org/join/


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

end of thread, other threads:[~2023-05-05 15:25 UTC | newest]

Thread overview: 80+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-04-24 16:44 [virtio-comment] [PATCH v12 00/10] Introduce device group and device management Michael S. Tsirkin
2023-04-24 16:44 ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 01/10] virtio: document forward compatibility guarantees Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 02/10] admin: introduce device group and related concepts Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 21:02   ` [virtio-comment] " Max Gurtovoy
2023-04-24 21:47   ` Parav Pandit
2023-04-24 21:47     ` [virtio-dev] " Parav Pandit
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 03/10] admin: introduce group administration commands Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 21:29   ` [virtio-comment] " Max Gurtovoy
2023-04-24 21:33     ` Michael S. Tsirkin
2023-04-24 21:33       ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 22:07   ` [virtio-comment] " Parav Pandit
2023-04-24 22:07     ` [virtio-dev] " Parav Pandit
2023-04-25  6:20     ` [virtio-comment] " Michael S. Tsirkin
2023-04-25  6:20       ` [virtio-dev] " Michael S. Tsirkin
2023-04-25 13:25       ` [virtio-comment] " Parav Pandit
2023-04-25 13:25         ` [virtio-dev] " Parav Pandit
2023-04-25 13:31         ` [virtio-comment] " Michael S. Tsirkin
2023-04-25 13:31           ` [virtio-dev] " Michael S. Tsirkin
2023-04-25 13:38           ` [virtio-comment] " Parav Pandit
2023-04-25 13:38             ` [virtio-dev] " Parav Pandit
2023-04-25 20:04             ` [virtio-comment] " Michael S. Tsirkin
2023-04-25 20:04               ` [virtio-dev] " Michael S. Tsirkin
2023-04-25 20:18               ` [virtio-comment] " Parav Pandit
2023-04-25 20:18                 ` [virtio-dev] " Parav Pandit
2023-04-25 20:55                 ` [virtio-comment] " Michael S. Tsirkin
2023-04-25 20:55                   ` [virtio-dev] " Michael S. Tsirkin
2023-04-26 18:18                   ` [virtio-comment] " Parav Pandit
2023-04-26 18:18                     ` [virtio-dev] " Parav Pandit
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 04/10] admin: introduce virtio admin virtqueues Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 22:32   ` [virtio-comment] " Parav Pandit
2023-04-24 22:32     ` [virtio-dev] " Parav Pandit
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 05/10] pci: add admin vq registers to virtio over pci Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 22:06   ` [virtio-comment] " Max Gurtovoy
2023-04-24 22:14     ` Parav Pandit
2023-04-24 22:14       ` [virtio-dev] " Parav Pandit
2023-04-26 21:12       ` [virtio-comment] " Max Gurtovoy
2023-04-26 22:11         ` [virtio-comment] " Parav Pandit
2023-04-26 22:11           ` [virtio-dev] " Parav Pandit
2023-04-26 22:31           ` [virtio-comment] " Max Gurtovoy
2023-04-27  0:11             ` [virtio-comment] " Parav Pandit
2023-04-27  0:11               ` [virtio-dev] " Parav Pandit
2023-05-05 15:12               ` [virtio-comment] " Michael S. Tsirkin
2023-05-05 15:12                 ` [virtio-dev] " Michael S. Tsirkin
2023-05-05 15:14                 ` [virtio-comment] " Parav Pandit
2023-05-05 15:14                   ` [virtio-dev] " Parav Pandit
2023-04-27 17:57             ` [virtio-comment] " Michael S. Tsirkin
2023-04-27 17:57               ` [virtio-dev] " Michael S. Tsirkin
2023-04-27 18:50               ` [virtio-comment] " Max Gurtovoy
2023-05-05 15:22           ` Michael S. Tsirkin
2023-05-05 15:22             ` [virtio-dev] " Michael S. Tsirkin
2023-05-05 15:25             ` [virtio-comment] " Parav Pandit
2023-05-05 15:25               ` [virtio-dev] " Parav Pandit
2023-04-24 22:29   ` [virtio-comment] " Parav Pandit
2023-04-24 22:29     ` [virtio-dev] " Parav Pandit
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 06/10] mmio: document ADMIN_VQ as reserved Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 22:08   ` [virtio-comment] " Max Gurtovoy
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 07/10] ccw: " Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 22:09   ` [virtio-comment] " Max Gurtovoy
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 08/10] admin: command list discovery Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 22:27   ` [virtio-comment] " Parav Pandit
2023-04-24 22:27     ` [virtio-dev] " Parav Pandit
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 09/10] admin: conformance clauses Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 16:44 ` [virtio-comment] [PATCH v12 10/10] ccw: document more reserved features Michael S. Tsirkin
2023-04-24 16:44   ` [virtio-dev] " Michael S. Tsirkin
2023-04-24 22:17   ` [virtio-comment] " Parav Pandit
2023-04-24 22:17     ` [virtio-dev] " Parav Pandit
2023-04-24 21:34 ` [virtio-comment] Re: [PATCH v12 00/10] Introduce device group and device management Parav Pandit
2023-04-24 21:34   ` [virtio-dev] " Parav Pandit
2023-05-02  7:51 ` [virtio-comment] Re: [virtio] " David Edmondson
2023-05-02  7:51   ` [virtio-dev] " David Edmondson

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.