From: "Michael S. Tsirkin" <mst@redhat.com>
To: Parav Pandit <parav@nvidia.com>
Cc: "virtio-comment@lists.oasis-open.org"
<virtio-comment@lists.oasis-open.org>,
Shahaf Shuler <shahafs@nvidia.com>,
"virtio@lists.oasis-open.org" <virtio@lists.oasis-open.org>
Subject: Re: [virtio-comment] [PATCH requirements 1/7] net-features: Add requirements document for release 1.4
Date: Wed, 7 Jun 2023 16:50:18 -0400 [thread overview]
Message-ID: <20230607164939-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <DM8PR12MB5480C6DA479E4DEFAC5E1EC1DC53A@DM8PR12MB5480.namprd12.prod.outlook.com>
On Wed, Jun 07, 2023 at 08:39:11PM +0000, Parav Pandit wrote:
>
>
> > From: Michael S. Tsirkin <mst@redhat.com>
> > Sent: Wednesday, June 7, 2023 4:36 PM
> >
> > On Tue, Jun 06, 2023 at 11:08:12PM +0000, Parav Pandit wrote:
> > > We can debate and change from cvq to new cfgvq as part of the journey.
> > > It is fine.
> >
> > Just to stress, cvq has a bunch of drawbacks. For example it is hard to access
> > from the hypervisor since it's DMA from VF.
> > This is why I think our direction should be to add a vq transport that does all
> > config accesses over admin commands.
> We debate this before, so will write is below.
> > Or maybe I'm wrong.
> >
>
> > But, let's distinguish between requirements, and between requirements and
> > design. Getting supported counters is a requirement.
> > Doing this over cvq is a design. Reducing MMIO registers is a requirement. OK?
>
> Yes, I would write it as reducing MMIO and accessing it over vq of the own device is a requirement.
reducing mmio is a requirement (unrelated to counters btw, it's
general). access over vq is a design point I think.
--
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/
next prev parent reply other threads:[~2023-06-07 20:50 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-01 22:02 [virtio-comment] [PATCH requirements 0/7] virtio net new features requirements Parav Pandit
2023-06-01 22:02 ` [virtio-comment] [PATCH requirements 1/7] net-features: Add requirements document for release 1.4 Parav Pandit
2023-06-06 22:15 ` Michael S. Tsirkin
2023-06-06 22:28 ` Parav Pandit
2023-06-06 22:56 ` Michael S. Tsirkin
2023-06-06 23:08 ` Parav Pandit
2023-06-06 23:18 ` Michael S. Tsirkin
2023-06-07 9:03 ` [virtio-comment] Re: [virtio] " Xuan Zhuo
2023-06-07 20:35 ` Michael S. Tsirkin
2023-06-07 20:39 ` Parav Pandit
2023-06-07 20:50 ` Michael S. Tsirkin [this message]
2023-06-07 20:53 ` Parav Pandit
2023-06-07 9:31 ` Xuan Zhuo
2023-06-01 22:03 ` [virtio-comment] [PATCH requirements 2/7] net-features: Add low latency transmit queue requirements Parav Pandit
2023-06-06 22:25 ` Michael S. Tsirkin
2023-06-06 22:35 ` Parav Pandit
2023-06-01 22:03 ` [virtio-comment] [PATCH requirements 3/7] net-features: Add low latency receive " Parav Pandit
2023-06-06 22:33 ` Michael S. Tsirkin
2023-06-06 22:44 ` Parav Pandit
2023-06-06 23:03 ` Michael S. Tsirkin
2023-06-01 22:03 ` [virtio-comment] [PATCH requirements 4/7] net-features: Add notification coalescing requirements Parav Pandit
2023-06-06 22:36 ` Michael S. Tsirkin
2023-06-06 22:46 ` Parav Pandit
2023-06-06 23:06 ` Michael S. Tsirkin
2023-06-01 22:03 ` [virtio-comment] [PATCH requirements 5/7] net-features: Add n-tuple receive flow steering requirements Parav Pandit
2023-06-02 3:35 ` Heng Qi
2023-06-02 3:51 ` Parav Pandit
2023-06-02 4:39 ` [virtio-comment] Re: [virtio] " Heng Qi
2023-06-06 12:08 ` Heng Qi
2023-06-06 21:49 ` [virtio-comment] " Parav Pandit
2023-06-12 14:35 ` [virtio-comment] " Heng Qi
2023-06-12 17:26 ` [virtio-comment] " Parav Pandit
2023-06-13 2:28 ` Heng Qi
2023-06-13 8:57 ` [virtio-comment] " Michael S. Tsirkin
2023-06-13 9:16 ` Cornelia Huck
2023-06-13 11:33 ` [virtio-comment] " Parav Pandit
2023-06-07 2:47 ` Jason Wang
2023-06-07 3:22 ` Parav Pandit
2023-06-13 2:57 ` [virtio-comment] Re: [virtio] " Heng Qi
2023-06-13 4:16 ` [virtio-comment] " Parav Pandit
2023-06-13 5:04 ` [virtio-comment] " Heng Qi
2023-06-13 12:24 ` [virtio-comment] " Parav Pandit
2023-06-14 3:43 ` [virtio-comment] " Heng Qi
2023-06-14 3:48 ` [virtio-comment] " Parav Pandit
2023-06-14 3:53 ` Heng Qi
2023-06-01 22:03 ` [virtio-comment] [PATCH requirements 6/7] net-features: Add packet timestamp requirements Parav Pandit
2023-06-06 22:40 ` Michael S. Tsirkin
2023-06-06 22:51 ` Parav Pandit
2023-06-06 23:08 ` Michael S. Tsirkin
2023-06-01 22:03 ` [virtio-comment] [PATCH requirements 7/7] net-features: Add header data split requirements Parav Pandit
2023-06-06 22:41 ` Michael S. Tsirkin
2023-06-08 14:57 ` Parav Pandit
2023-06-02 3:06 ` [virtio-comment] Re: [virtio] [PATCH requirements 0/7] virtio net new features requirements Heng Qi
2023-06-06 22:49 ` [virtio-comment] " Michael S. Tsirkin
2023-06-06 22:56 ` Parav Pandit
2023-06-06 23:10 ` Michael S. Tsirkin
2023-06-07 2:49 ` Jason Wang
2023-06-07 3:33 ` Parav Pandit
-- strict thread matches above, loose matches on Subject: below --
2023-07-24 3:34 Parav Pandit
2023-07-24 3:34 ` [virtio-comment] [PATCH requirements 1/7] net-features: Add requirements document for release 1.4 Parav Pandit
2023-08-08 8:16 ` David Edmondson
2023-08-14 5:17 ` Parav Pandit
2023-08-14 11:53 ` David Edmondson
2023-08-14 11:56 ` David Edmondson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230607164939-mutt-send-email-mst@kernel.org \
--to=mst@redhat.com \
--cc=parav@nvidia.com \
--cc=shahafs@nvidia.com \
--cc=virtio-comment@lists.oasis-open.org \
--cc=virtio@lists.oasis-open.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.