public inbox for virtio-dev@lists.linux.dev
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Jason Wang <jasowang@redhat.com>
Cc: Parav Pandit <parav@nvidia.com>,
	"virtio-dev@lists.oasis-open.org"
	<virtio-dev@lists.oasis-open.org>,
	"cohuck@redhat.com" <cohuck@redhat.com>,
	"david.edmondson@oracle.com" <david.edmondson@oracle.com>,
	"sburla@marvell.com" <sburla@marvell.com>,
	Yishai Hadas <yishaih@nvidia.com>,
	Maor Gottlieb <maorg@nvidia.com>,
	"virtio-comment@lists.oasis-open.org"
	<virtio-comment@lists.oasis-open.org>,
	Shahaf Shuler <shahafs@nvidia.com>
Subject: [virtio-dev] Re: [virtio-comment] Re: [virtio-dev] Re: [PATCH v3 0/3] transport-pci: Introduce legacy registers access using AQ
Date: Fri, 9 Jun 2023 03:21:50 -0400	[thread overview]
Message-ID: <20230609031923-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <CACGkMEsN=5DcMsZa5hHGKb9kCRUVA2CcwZ4uif-E0HKaXiJ-fA@mail.gmail.com>

On Fri, Jun 09, 2023 at 02:27:01PM +0800, Jason Wang wrote:
> > I would like to keep the stateful interactions of 1.x device outside of 0.9.5.
> 
> I don't think this is a real problem, but let's see the drawbacks of
> this proposal:
> 
> 1) non-trivial changes of full new transport alike ABI
> 2) can't work for nesting
> 3) require vendor to implement legacy behaviour which can't be
> documented precisely
> 4) require admin virtqueue to work
> 
> We won't have the above issues if we use modern + legacy header/mac.
> 
> Thanks


All this is true. But it's by design. It makes the legacy thing
as isolated as possible. Because let's be frank we won't be
able to drop legacy support in like 10 years. But hardware
vendors will do that quickly if they can't sell hardware.

-- 
MST


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


  reply	other threads:[~2023-06-09  7:21 UTC|newest]

Thread overview: 82+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-06-02 20:36 [virtio-dev] [PATCH v3 0/3] transport-pci: Introduce legacy registers access using AQ Parav Pandit
2023-06-02 20:36 ` [virtio-dev] [PATCH v3 1/3] admin: Split opcode table rows with a line Parav Pandit
2023-06-02 20:36 ` [virtio-dev] [PATCH v3 2/3] transport-pci: Introduce legacy registers access commands Parav Pandit
2023-06-04 13:22   ` [virtio-dev] " Michael S. Tsirkin
2023-06-04 13:51     ` [virtio-dev] " Parav Pandit
2023-06-04 14:13       ` [virtio-dev] " Michael S. Tsirkin
2023-06-04 14:32         ` [virtio-dev] " Parav Pandit
2023-06-04 14:41           ` [virtio-dev] " Michael S. Tsirkin
2023-06-04 15:01             ` [virtio-dev] " Parav Pandit
2023-06-04 22:10               ` [virtio-dev] " Michael S. Tsirkin
2023-06-04 23:57                 ` [virtio-dev] " Parav Pandit
2023-06-08 18:34   ` [virtio-dev] " Michael S. Tsirkin
2023-06-08 18:55     ` [virtio-dev] " Parav Pandit
2023-06-08 19:00       ` [virtio-dev] " Michael S. Tsirkin
2023-06-08 19:04         ` [virtio-dev] " Parav Pandit
2023-06-02 20:36 ` [virtio-dev] [PATCH v3 3/3] transport-pci: Add legacy register access conformance section Parav Pandit
2023-06-04 13:34 ` [virtio-dev] Re: [PATCH v3 0/3] transport-pci: Introduce legacy registers access using AQ Michael S. Tsirkin
2023-06-04 13:41   ` [virtio-dev] " Parav Pandit
2023-06-04 13:55     ` [virtio-dev] " Michael S. Tsirkin
2023-06-04 14:10       ` [virtio-dev] " Parav Pandit
2023-06-04 14:23         ` [virtio-dev] " Michael S. Tsirkin
2023-06-04 14:48           ` [virtio-dev] " Parav Pandit
2023-06-04 14:53             ` [virtio-dev] " Michael S. Tsirkin
2023-06-04 15:07               ` [virtio-dev] " Parav Pandit
2023-06-04 21:48                 ` [virtio-dev] " Michael S. Tsirkin
2023-06-04 23:40                   ` [virtio-dev] " Parav Pandit
2023-06-05  5:51                     ` [virtio-dev] " Michael S. Tsirkin
2023-06-05 13:27                       ` [virtio-dev] " Parav Pandit
2023-06-05 13:50                         ` [virtio-dev] " Michael S. Tsirkin
2023-06-05 16:04                           ` [virtio-dev] " Parav Pandit
2023-06-05 21:57                             ` [virtio-dev] " Michael S. Tsirkin
2023-06-05 22:12                               ` Parav Pandit
2023-06-06 11:56                                 ` Michael S. Tsirkin
2023-06-06 20:15                                   ` Parav Pandit
2023-06-07  2:27                                   ` Jason Wang
2023-06-07  3:05                                     ` Parav Pandit
2023-06-07  6:54                                       ` Jason Wang
2023-06-07  8:54                                         ` Michael S. Tsirkin
2023-06-08 14:38                                         ` Parav Pandit
2023-06-08 14:44                                           ` Michael S. Tsirkin
2023-06-08 14:53                                             ` Parav Pandit
2023-06-08 15:03                                               ` Michael S. Tsirkin
2023-06-08 15:16                                                 ` Parav Pandit
2023-06-08 18:03                                                   ` Michael S. Tsirkin
2023-06-08 18:11                                                     ` Parav Pandit
2023-06-08 18:31                                                   ` Michael S. Tsirkin
2023-06-08 19:00                                                     ` Parav Pandit
2023-06-08 19:03                                                       ` Michael S. Tsirkin
2023-06-08 19:12                                                         ` Parav Pandit
2023-06-09  2:06                                           ` Jason Wang
2023-06-09  2:29                                             ` Parav Pandit
2023-06-09  2:42                                               ` Jason Wang
2023-06-09  2:53                                                 ` Parav Pandit
2023-06-09  2:56                                                   ` Jason Wang
2023-06-09  2:58                                                     ` [virtio-dev] RE: [virtio-comment] " Parav Pandit
2023-06-09  3:02                                                       ` [virtio-dev] " Jason Wang
2023-06-09  3:25                                                         ` [virtio-dev] " Parav Pandit
2023-06-09  6:27                                                           ` [virtio-dev] " Jason Wang
2023-06-09  7:21                                                             ` Michael S. Tsirkin [this message]
2023-06-09 17:11                                                               ` [virtio-dev] " Parav Pandit
2023-06-11  0:27                                                                 ` [virtio-dev] " Michael S. Tsirkin
2023-06-11  2:08                                                                   ` [virtio-dev] " Parav Pandit
2023-06-11  7:14                                                                     ` [virtio-dev] " Michael S. Tsirkin
2023-06-11 12:54                                                                       ` [virtio-dev] " Parav Pandit
2023-06-11 20:09                                                                         ` [virtio-dev] " Michael S. Tsirkin
2023-06-11 20:17                                                                           ` [virtio-dev] " Parav Pandit
2023-06-11 23:15                                                                             ` [virtio-dev] " Michael S. Tsirkin
2023-06-26  3:46                                                                   ` Jason Wang
2023-06-26  3:32                                                                 ` Jason Wang
2023-06-26  3:51                                                                   ` [virtio-dev] " Parav Pandit
2023-06-27  2:38                                                                     ` [virtio-dev] " Jason Wang
2023-06-27  3:17                                                                       ` [virtio-dev] " Parav Pandit
2023-06-27  4:33                                                                         ` [virtio-dev] " Jason Wang
2023-06-26  3:50                                                               ` Jason Wang
2023-06-26  3:55                                                                 ` [virtio-dev] " Parav Pandit
2023-06-26 10:49                                                                 ` [virtio-dev] " Michael S. Tsirkin
2023-06-09  7:15                                             ` Michael S. Tsirkin
2023-06-26  3:59                                               ` Jason Wang
2023-06-26  4:04                                                 ` [virtio-dev] RE: [virtio-comment] " Parav Pandit
2023-06-27  2:42                                                   ` [virtio-dev] " Jason Wang
2023-06-26  7:13                                                 ` Michael S. Tsirkin
2023-06-07  8:57                                     ` Michael S. Tsirkin

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=20230609031923-mutt-send-email-mst@kernel.org \
    --to=mst@redhat.com \
    --cc=cohuck@redhat.com \
    --cc=david.edmondson@oracle.com \
    --cc=jasowang@redhat.com \
    --cc=maorg@nvidia.com \
    --cc=parav@nvidia.com \
    --cc=sburla@marvell.com \
    --cc=shahafs@nvidia.com \
    --cc=virtio-comment@lists.oasis-open.org \
    --cc=virtio-dev@lists.oasis-open.org \
    --cc=yishaih@nvidia.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox