virtio-dev.lists.oasis-open.org archive mirror
 help / color / mirror / Atom feed
* [virtio-dev] [PATCH v10 00/10] Introduce device group and device management
@ 2023-03-02 13:04 Michael S. Tsirkin
  2023-03-02 13:04 ` [virtio-dev] [PATCH v10 01/10] virtio: document forward compatibility guarantees Michael S. Tsirkin
                   ` (13 more replies)
  0 siblings, 14 replies; 170+ messages in thread
From: Michael S. Tsirkin @ 2023-03-02 13:04 UTC (permalink / raw)
  To: virtio-comment, virtio-dev, jasowang, mst, cohuck, sgarzare,
	stefanha, nrupal.jani, Piotr.Uminski, hang.yuan
  Cc: virtio, Zhu Lingshan, pasic, Shahaf Shuler, Parav Pandit, Max Gurtovoy

Change log:

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).

This is out of RFC, since we have two commands which are useful
to discover supported group types (ATM can be none or SR-IOV).


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        | 540 +++++++++++++++++++++++++++++++++++++++++++++++
 content.tex      | 112 +++++++++-
 introduction.tex |   3 +
 3 files changed, 653 insertions(+), 2 deletions(-)
 create mode 100644 admin.tex

-- 
MST


---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org


^ permalink raw reply	[flat|nested] 170+ messages in thread
* [virtio-dev] [PATCH v10 00/10] Introduce device group and device management
@ 2023-02-09 12:13 Michael S. Tsirkin
  2023-02-09 12:13 ` [PATCH v10 01/10] virtio: document forward compatibility guarantees Michael S. Tsirkin
  0 siblings, 1 reply; 170+ messages in thread
From: Michael S. Tsirkin @ 2023-02-09 12:13 UTC (permalink / raw)
  To: virtio-comment, virtio-dev, jasowang, mst, cohuck, sgarzare,
	stefanha, nrupal.jani, Piotr.Uminski, hang.yuan
  Cc: virtio, Zhu Lingshan, pasic, Shahaf Shuler, Parav Pandit, Max Gurtovoy


Thanks to everyone who commented!  Here's v10 after a longish
hiccup - I hope this is close to being ready. I'm especially
interested to hear whether this is in a good enough shape to
serve as basis for vq transport.

Thanks!

Change log:

since v9:
	addressed review comments:
		multiple typo fixes
		made command list array 64 bit - easier future proofing
		clarifications around group type and admin command concepts
		lots of clarifications around structure
			trucation for forwards and backwards compatibility -
			hopefully things are clear now

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).

This is out of RFC, since we have two commands which are useful
to discover supported group types (ATM can be none or SR-IOV).


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   | 501 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 content.tex | 111 +++++++++++-
 2 files changed, 610 insertions(+), 2 deletions(-)
 create mode 100644 admin.tex

-- 
MST


---------------------------------------------------------------------
To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org
For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org


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

end of thread, other threads:[~2023-03-31 11:44 UTC | newest]

Thread overview: 170+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-02 13:04 [virtio-dev] [PATCH v10 00/10] Introduce device group and device management Michael S. Tsirkin
2023-03-02 13:04 ` [virtio-dev] [PATCH v10 01/10] virtio: document forward compatibility guarantees Michael S. Tsirkin
2023-03-02 18:39   ` [virtio-dev] " Parav Pandit
2023-03-02 23:43     ` [virtio-dev] " Michael S. Tsirkin
     [not found]   ` <m2leka0yvl.fsf@oracle.com>
2023-03-06 22:00     ` [virtio-dev] Re: [virtio] " Michael S. Tsirkin
2023-03-07 10:04       ` David Edmondson
2023-03-08 14:16         ` Cornelia Huck
2023-03-08 15:04           ` Michael S. Tsirkin
2023-03-02 13:04 ` [virtio-dev] [PATCH v10 02/10] admin: introduce device group and related concepts Michael S. Tsirkin
2023-03-02 18:39   ` [virtio-dev] " Parav Pandit
2023-03-02 23:44     ` [virtio-dev] " Michael S. Tsirkin
2023-03-02 19:40   ` [virtio-dev] Re: [virtio] " Stefan Hajnoczi
2023-03-06 17:00   ` David Edmondson
2023-03-02 13:05 ` [virtio-dev] [PATCH v10 03/10] admin: introduce group administration commands Michael S. Tsirkin
2023-03-02 18:40   ` [virtio-dev] " Parav Pandit
2023-03-02 20:19     ` [virtio-dev] " Stefan Hajnoczi
2023-03-03  0:01       ` Michael S. Tsirkin
2023-03-03 13:17         ` Stefan Hajnoczi
2023-03-03 13:19           ` Michael S. Tsirkin
     [not found]             ` <4f869944-4ccd-c51e-0f30-dc3ba15ffd52@nvidia.com>
2023-03-07 18:33               ` [virtio-dev] " Parav Pandit
2023-03-08  0:34               ` [virtio-dev] " Michael S. Tsirkin
2023-03-08  1:01                 ` [virtio-dev] " Parav Pandit
2023-03-08  1:06                   ` [virtio-dev] " Michael S. Tsirkin
2023-03-08  1:14                     ` [virtio-dev] " Parav Pandit
     [not found]                 ` <6668fd7a-3eb3-0447-9cbf-72d308b1336a@nvidia.com>
2023-03-08 12:07                   ` [virtio-dev] " Michael S. Tsirkin
     [not found]                     ` <992e9a1e-3799-842c-79f3-e66f5c823356@nvidia.com>
2023-03-08 14:43                       ` Cornelia Huck
2023-03-08 14:54                         ` Michael S. Tsirkin
2023-03-08 15:21                       ` Michael S. Tsirkin
     [not found]                   ` <2ef47f53-5978-4b6a-593d-4e94eba0b9ac@nvidia.com>
2023-03-09  6:38                     ` [virtio-dev] Re: [virtio-comment] " Michael S. Tsirkin
2023-03-02 23:47     ` [virtio-dev] " Michael S. Tsirkin
2023-03-07 18:26       ` [virtio-dev] " Parav Pandit
2023-03-02 20:10   ` [virtio-dev] " Stefan Hajnoczi
2023-03-02 23:57     ` Michael S. Tsirkin
2023-03-03 13:13       ` Stefan Hajnoczi
2023-03-03 13:18         ` Michael S. Tsirkin
2023-03-03 20:23           ` Stefan Hajnoczi
2023-03-07 11:31             ` Jiri Pirko
2023-03-08 12:03               ` Michael S. Tsirkin
2023-03-07 10:31   ` [virtio-dev] Re: [virtio] " David Edmondson
2023-03-02 13:05 ` [virtio-dev] [PATCH v10 04/10] admin: introduce virtio admin virtqueues Michael S. Tsirkin
2023-03-02 18:40   ` [virtio-dev] " Parav Pandit
2023-03-02 23:48     ` [virtio-dev] " Michael S. Tsirkin
2023-03-02 20:40   ` Stefan Hajnoczi
2023-03-03  0:05     ` Michael S. Tsirkin
2023-03-03 13:28       ` Stefan Hajnoczi
2023-03-03 13:37         ` Michael S. Tsirkin
2023-03-03 20:21           ` [virtio-dev] Re: [virtio] " Stefan Hajnoczi
2023-03-05  9:38             ` Michael S. Tsirkin
2023-03-06  0:03               ` Stefan Hajnoczi
2023-03-06  0:18                 ` Michael S. Tsirkin
     [not found]                   ` <20230306110340.GA35392@fedora>
2023-03-06 18:37                     ` Michael S. Tsirkin
2023-03-06 20:17                       ` Stefan Hajnoczi
2023-03-06 21:43                         ` Michael S. Tsirkin
2023-03-31 11:07                         ` Michael S. Tsirkin
2023-03-07  8:03                       ` [virtio-dev] Re: [virtio-comment] " Jiri Pirko
2023-03-07 14:39                         ` Stefan Hajnoczi
2023-03-07 15:07                           ` Jiri Pirko
2023-03-07 19:03                             ` Stefan Hajnoczi
2023-03-07 19:09                               ` [virtio-dev] " Parav Pandit
2023-03-08  5:17                                 ` [virtio-dev] Re: [virtio] " Jason Wang
2023-03-08 11:58                                   ` Stefan Hajnoczi
2023-03-08 11:59                                   ` Michael S. Tsirkin
2023-03-08 10:17                               ` [virtio-dev] " Jiri Pirko
2023-03-08 12:44                                 ` Stefan Hajnoczi
2023-03-08 12:57                                   ` [virtio-dev] Re: [virtio] " Jiri Pirko
2023-03-08 17:17                                     ` Stefan Hajnoczi
2023-03-07 16:13                         ` [virtio-dev] " Michael S. Tsirkin
2023-03-08 10:08                           ` Jiri Pirko
2023-03-08 11:44                             ` Michael S. Tsirkin
     [not found]                 ` <7f63fa0a-7deb-5875-6c6b-bfc651681653@redhat.com>
     [not found]                   ` <20230306112030.GB35392@fedora>
     [not found]                     ` <853c78d0-f752-05e9-d79d-811e82801627@nvidia.com>
2023-03-06 16:25                       ` [virtio-dev] " Stefan Hajnoczi
2023-03-07 19:04                         ` [virtio-dev] " Parav Pandit
     [not found]                         ` <e74483a4-38fa-99b5-86b8-785f0b98d029@nvidia.com>
2023-03-08 14:13                           ` [virtio-dev] Re: [virtio-comment] " Stefan Hajnoczi
2023-03-08 16:21                             ` [virtio-dev] " Parav Pandit
     [not found]                             ` <18ddbf69-19a6-3c6b-9e42-aaae66e20bcf@nvidia.com>
2023-03-08 17:15                               ` [virtio-dev] " Stefan Hajnoczi
2023-03-08 17:21                                 ` Michael S. Tsirkin
2023-03-09 12:35                                   ` Stefan Hajnoczi
2023-03-09 13:55                                     ` Michael S. Tsirkin
2023-03-09 15:56                                       ` Stefan Hajnoczi
     [not found]       ` <e8d41902-b794-66e9-8f15-e8617435047c@redhat.com>
2023-03-06 16:22         ` [virtio-dev] " Jiri Pirko
     [not found]       ` <027fff1b-8ed7-abc0-2331-b188b8822bf4@nvidia.com>
2023-03-06 22:49         ` [virtio-dev] Re: [virtio-comment] " Michael S. Tsirkin
     [not found]           ` <e12ab469-4586-f0f5-b084-8f2d7c031e11@nvidia.com>
2023-03-08 21:09             ` Michael S. Tsirkin
2023-03-08 21:17               ` [virtio-dev] " Parav Pandit
2023-03-09  7:28               ` [virtio-dev] " Jiri Pirko
2023-03-07  7:56         ` Jiri Pirko
     [not found]   ` <ZAXfegxCfvfLwiJT@nanopsycho>
2023-03-06 18:40     ` [virtio-dev] Re: [virtio] " Michael S. Tsirkin
2023-03-07  7:36       ` Jiri Pirko
2023-03-07 16:30         ` Michael S. Tsirkin
2023-03-08 10:05           ` [virtio-dev] Re: [virtio-comment] " Jiri Pirko
2023-03-08 11:50             ` Michael S. Tsirkin
2023-03-08 12:08               ` Jiri Pirko
     [not found]                 ` <18d51bc0-d759-1a05-cb7c-3d46c4ed2f1a@nvidia.com>
2023-03-08 18:01                   ` David Edmondson
2023-03-08 18:19                     ` Jiri Pirko
2023-03-08 21:25                     ` [virtio-dev] " Parav Pandit
2023-03-09  7:30                       ` [virtio-dev] " Jiri Pirko
2023-03-09 13:04                         ` [virtio-dev] " Parav Pandit
2023-03-09 13:57                           ` [virtio-dev] " Michael S. Tsirkin
2023-03-09 14:02                           ` [virtio-dev] " David Edmondson
2023-03-09 14:11                             ` Parav Pandit
2023-03-09 14:22                               ` [virtio-dev] " Michael S. Tsirkin
2023-03-09 14:30                                 ` [virtio-dev] " Parav Pandit
2023-03-09 14:43                                   ` [virtio-dev] " Michael S. Tsirkin
2023-03-09 16:53                                     ` [virtio-dev] " Parav Pandit
2023-03-09 17:14                                       ` [virtio-dev] " Michael S. Tsirkin
2023-03-09 17:16                                         ` [virtio-dev] " Parav Pandit
2023-03-08 21:45             ` Parav Pandit
2023-03-09  7:37               ` [virtio-dev] " Jiri Pirko
2023-03-09 12:36                 ` [virtio-dev] " Parav Pandit
2023-03-09 16:32                   ` [virtio-dev] " Jiri Pirko
     [not found]     ` <17471553-8981-eef9-656b-098e1acdda14@nvidia.com>
2023-03-06 18:44       ` [virtio-dev] " Michael S. Tsirkin
2023-03-07 10:41   ` David Edmondson
2023-03-02 13:05 ` [virtio-dev] [PATCH v10 05/10] pci: add admin vq registers to virtio over pci Michael S. Tsirkin
2023-03-02 20:51   ` [virtio-dev] " Stefan Hajnoczi
2023-03-02 13:05 ` [virtio-dev] [PATCH v10 06/10] mmio: document ADMIN_VQ as reserved Michael S. Tsirkin
2023-03-02 18:40   ` [virtio-dev] " Parav Pandit
2023-03-02 23:51     ` [virtio-dev] " Michael S. Tsirkin
2023-03-02 23:51     ` Michael S. Tsirkin
2023-03-03  8:34     ` Michael S. Tsirkin
     [not found]       ` <ZAXB44F3MS9CxIiK@nanopsycho>
2023-03-06 18:46         ` [virtio-dev] Re: [virtio] " Michael S. Tsirkin
2023-03-07  7:40           ` [virtio-dev] Re: [virtio-comment] " Jiri Pirko
2023-03-07 18:52       ` [virtio-dev] " Parav Pandit
2023-03-08 16:24         ` [virtio-dev] Re: [virtio-comment] " Cornelia Huck
2023-03-08 16:37           ` [virtio-dev] " Parav Pandit
2023-03-08 16:30         ` [virtio-dev] " Michael S. Tsirkin
2023-03-08 18:21           ` [virtio-dev] Re: [virtio] " Jiri Pirko
2023-03-02 20:52   ` [virtio-dev] " Stefan Hajnoczi
2023-03-02 13:05 ` [virtio-dev] [PATCH v10 07/10] ccw: " Michael S. Tsirkin
2023-03-02 20:53   ` [virtio-dev] " Stefan Hajnoczi
2023-03-02 13:05 ` [virtio-dev] [PATCH v10 08/10] admin: command list discovery Michael S. Tsirkin
2023-03-02 21:09   ` [virtio-dev] " Stefan Hajnoczi
2023-03-31 11:39     ` Michael S. Tsirkin
2023-03-07 10:54   ` [virtio-dev] " David Edmondson
     [not found]   ` <ZAXbBgN2jw8RhE/3@nanopsycho>
2023-03-08 11:54     ` Michael S. Tsirkin
2023-03-08 12:41       ` Jiri Pirko
2023-03-31 11:43         ` [virtio-dev] Re: [virtio-comment] " Michael S. Tsirkin
2023-03-08 12:38   ` Jiri Pirko
2023-03-10  8:14   ` [virtio-dev] " Zhu, Lingshan
2023-03-02 13:05 ` [virtio-dev] [PATCH v10 09/10] admin: conformance clauses Michael S. Tsirkin
2023-03-07 11:04   ` [virtio-dev] Re: [virtio-comment] " David Edmondson
2023-03-08 11:58     ` Michael S. Tsirkin
2023-03-08 12:59       ` David Edmondson
2023-03-08 13:05         ` [virtio-dev] Re: [virtio] " Jiri Pirko
2023-03-08 13:22           ` Michael S. Tsirkin
2023-03-08 13:44           ` David Edmondson
2023-03-08 14:02             ` Jiri Pirko
2023-03-08 14:12               ` Michael S. Tsirkin
2023-03-10  9:10   ` [virtio-dev] " Zhu, Lingshan
2023-03-10  9:13     ` Michael S. Tsirkin
2023-03-10 18:00       ` Zhu Lingshan
2023-03-10  9:34     ` Michael S. Tsirkin
2023-03-02 13:05 ` [virtio-dev] [PATCH v10 10/10] ccw: document more reserved features Michael S. Tsirkin
2023-03-02 21:12   ` [virtio-dev] " Stefan Hajnoczi
2023-03-02 13:38 ` [virtio-dev] RE: [PATCH v10 00/10] Introduce device group and device management Parav Pandit
2023-03-02 23:27   ` [virtio-dev] " Michael S. Tsirkin
2023-03-02 18:39 ` [virtio-dev] " Parav Pandit
2023-03-06 16:40 ` [virtio-dev] Re: [virtio-comment] " Jiri Pirko
2023-03-06 22:48   ` Michael S. Tsirkin
2023-03-07  7:17     ` Jiri Pirko
2023-03-07 17:15       ` Michael S. Tsirkin
     [not found] ` <ZAXcqqdwfoLokT2l@nanopsycho>
2023-03-06 22:54   ` Michael S. Tsirkin
2023-03-07  7:21     ` Jiri Pirko
2023-03-07 17:20       ` Michael S. Tsirkin
2023-03-07 19:31         ` [virtio-dev] " Parav Pandit
2023-03-08  5:11           ` [virtio-dev] " Jason Wang
2023-03-08 12:02             ` [virtio-dev] " Parav Pandit
2023-03-10  8:32           ` [virtio-dev] " Zhu, Lingshan
2023-03-08  9:49         ` [virtio-dev] Re: [virtio] " Jiri Pirko
2023-03-08 16:30     ` [virtio-dev] " Cornelia Huck
2023-03-08 17:22       ` Michael S. Tsirkin
2023-03-08 18:15       ` Jiri Pirko
  -- strict thread matches above, loose matches on Subject: below --
2023-02-09 12:13 [virtio-dev] " Michael S. Tsirkin
2023-02-09 12:13 ` [PATCH v10 01/10] virtio: document forward compatibility guarantees Michael S. Tsirkin
2023-02-11 18:50   ` [virtio-dev] " Parav Pandit

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).