From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Sender: List-Post: List-Help: List-Unsubscribe: List-Subscribe: Received: from lists.oasis-open.org (oasis-open.org [10.110.1.242]) by lists.oasis-open.org (Postfix) with ESMTP id 08F659862BA for ; Tue, 30 Nov 2021 13:49:11 +0000 (UTC) From: Cornelia Huck In-Reply-To: <9053000.DQESySOnJA@silver> References: <875ys9j1a0.fsf@redhat.com> <9053000.DQESySOnJA@silver> Date: Tue, 30 Nov 2021 14:48:50 +0100 Message-ID: <8735ndiwjh.fsf@redhat.com> MIME-Version: 1.0 Subject: Re: [virtio-comment] Re: [PATCH v2 0/2] Add VIRTIO_RING_F_LARGE_INDIRECT_DESC Content-Type: text/plain To: Christian Schoenebeck , virtio-comment@lists.oasis-open.org Cc: Stefan Hajnoczi , Greg Kurz List-ID: On Tue, Nov 30 2021, Christian Schoenebeck wrote: > On Dienstag, 30. November 2021 13:06:31 CET Cornelia Huck wrote: >> On Mon, Nov 29 2021, Christian Schoenebeck wrote: >> > This is a follow-up of: >> > https://lists.oasis-open.org/archives/virtio-comment/202111/msg00059.html >> > >> > These two patches introduce a more fine graded control over the maximum >> > length of an indirect descriptor table. >> > >> > Associated Github Task: >> > https://github.com/oasis-tcs/virtio-spec/issues/122 >> > >> > v1 -> v2: >> > * "active/pending message(s)" -> "vring slot(s)" >> > >> > NOTE: I decided to use the term "vring slot(s)" over the previously >> > suggested term "buffer(s)" as I found tha latter too ambiguous in this >> > context. [patch 1] >> > >> > * "... but does not oppose a limit to the actual bulk data size >> > >> > being transmitted." >> > -> >> > " ... but does not oppose a limit to the actual bulk data size >> > being transmitted when indirect descriptors are used." [patch 1] >> > >> > * Add common configuration field "queue_indirect_size" and make it >> > >> > mandatory as of revision 3. [NEW patch 2] >> >> Now you have me confused: 'revision' is a ccw transport concept, but you >> only add support on the pci transport? It's fine to leave out ccw (and >> mmio) for now, but you should not talk about revision 3... > > Looks like I was too quick on this one. Right, the name virtio_pci_common_cfg > should have suggested to me that this was PCI specific. :) I simply assumed > this was a common config structure shared by all bus types. > > And I had no idea that you were referring with "revision" to a bus specific > mechanism either. I assumed virtio v1.0 = rev1, virtio v1.1 = rev2 and that > the upcoming spec to become v1.2 = rev3 (i.e. for all bus tyes). Apologies if I was too confusing with my comments :( > > So what is the desired approach to proceed on this overall task, should this > just be addressed on PCI for now as first step? Yes, I'd concentrate on PCI first. It should not be too hard to add this for the other transports. > > And what about the intended availability of this new virtio_pci_common_cfg > field "queue_indirect_size", should it be optional per se, independent of the > virtio version or rather a mandatory field in upcoming virtio version? We should keep that field depending upon the feature bit IMHO. > >> > Christian Schoenebeck (2): >> > Add VIRTIO_RING_F_LARGE_INDIRECT_DESC >> > Add common configuration field "queue_indirect_size" >> > >> > content.tex | 56 ++++++++++++++++++++++++++++++++++++++++++++++++++ >> > split-ring.tex | 7 ++++++- >> > 2 files changed, 62 insertions(+), 1 deletion(-) 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/