[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [PATCH v2 1/4] Add virtio Admin Virtqueue
On 1/26/2022 4:40 PM, Michael S. Tsirkin wrote:
On Mon, Jan 24, 2022 at 11:39:15AM +0200, Max Gurtovoy wrote:+Regardless of device offering VIRTIO_F_IN_ORDER, admin queue command buffers +are used by the device in out of order manner.Instead of special-casing AQ I'd like to see a generic capability addressing this need. For example, TX for virtio net might benefit from this too. And I'd like to mention, again, VIRTIO_F_PARTIAL_ORDER proposal as one, arguably cleaner and more generic way to address this.
We already suggested a special capability for IN_ORDER for AQ and you asked to drop it. We drop it and agreed that AQ will be OOO.
Why are we going back here ?You also mentioned that this patch set is already big enough so why solve all the problems we can think of in this one ? Why mixing VIRTIO_F_PARTIAL_ORDER here ?
And if that's not adequate I'd like to address that as part of the PARTIAL_ORDER proposal, this kind of per-queue in order was definitely on the radar as it was formulated.
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]