[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [PATCH v8 7/9] ccw: document ADMIN_VQ as reserved
On Tue, Nov 22, 2022 at 09:50:35AM +0100, Cornelia Huck wrote: > On Mon, Nov 21 2022, "Michael S. Tsirkin" <mst@redhat.com> wrote: > > > OK I think for now the following is sufficient, right? > > Separately, we should make an effort to document that these > > features are transport specific and not supported on all transports: > > happily VIRTIO_F_RING_RESET is only used in drivers/virtio/virtio_pci_modern.c > > for now, and VIRTIO_PMEM_F_SHMEM_REGION seems to be unused by Linux. > > > > ccw: document more reserved features > > > > vq reset and shared memory are unsupported, too. > > > > Signed-off-by: Michael S. Tsirkin <mst@redhat.com> > > > > diff --git a/content.tex b/content.tex > > index 30f8efb..776fc56 100644 > > --- a/content.tex > > +++ b/content.tex > > @@ -2978,6 +2978,8 @@ \subsection{Features reserved for future use}\label{sec:Virtio Transport Options > > \begin{itemize} > > > > \item VIRTIO_F_ADMIN_VQ > > +\item VIRTIO_F_RING_RESET > > +\item Shared memory regions including VIRTIO_PMEM_F_SHMEM_REGION > > \item VIRTIO_F_SR_IOV (not applicable for this transport) > \item VIRTIO_F_NOTIFICATION_DATA > \item VIRTIO_F_NOTIF_CONFIG_DATA > > I think that would be all AFAICS. In fact VIRTIO_F_NOTIF_CONFIG_DATA is also not supported for MMIO. > > What about doing a separate github issue for patch 1 + the list of > reserved features for ccw as of now? We should be able to get that in > independently of the admin vq work. Yes - if it's me that's going to work on this then I'd rather make it a patch on top though, I'd like to work on ADMIN_VQ first as that has people waiting on it. Work on hardening spec wording is something that will never end ;) -- MST
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]