virtio — archive
[Date Prev]
| [Thread Prev]
| [Thread Next]
| [Date Next]
— [Date Index]
| [Thread Index]
| [Month Index]
| [List Home]
[PATCH v8 0/9] Introduce device group and device management
On Tue, Nov 22, 2022 at 09:50:35AM +0100, Cornelia Huck wrote:
> On Mon, Nov 21 2022, "Michael S. Tsirkin" <[email protected]> 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 <[email protected]>
> >
>
> 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]
| [Month Index]
| [List Home]