Re: [PATCH v1 Resend] virtio-sound: add latency_bytes definition

["Michael S. Tsirkin" <mst@redhat.com>]

On Fri, Jun 21, 2024 at 03:37:29PM +0200, Matias Ezequiel Vara Larsen wrote:
> On Tue, May 21, 2024 at 05:03:25PM +0200, Matias Ezequiel Vara Larsen wrote:
> > This commit aims at providing a better definition of latency_bytes. To
> > do so, this commit defines latency_bytes as is defined in Crosvm.
> > 
> > Signed-off-by: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
> > ---
> > Changelog:
> > v0 -> v1:
> >  * specify that the delay is due to the device implementation.
> > ---
> >  device-types/sound/description.tex | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> > 
> > diff --git a/device-types/sound/description.tex b/device-types/sound/description.tex
> > index 54c9c8e..a138e5b 100644
> > --- a/device-types/sound/description.tex
> > +++ b/device-types/sound/description.tex
> > @@ -694,7 +694,10 @@ \subsubsection{PCM I/O Messages}\label{sec:Device Types / Sound Device / Device
> >  \begin{description}
> >  \item[\field{status}] contains VIRTIO_SND_S_OK if an operation is successful,
> >  and VIRTIO_SND_S_IO_ERR otherwise.
> > -\item[\field{latency_bytes}] indicates the current device latency.
> > +\item[\field{latency_bytes}] indicates the current additional delay caused by
> > +the device's implementation. For playback, this is the amount of bytes that
> > +must be consumed before the current request can be played. For capture, this is
> > +the latency in terms of bytes that the capture buffer was recorded.


the last "was recorded" seems agrammatical. Do you mean "has recorded"?

> >  \end{description}
> >  
> >  Since all buffers in the queue (with one exception) should be of the size
> > -- 
> > 2.41.0
> > 
> Dear TC,
> 
> I'd like to request a vote on solving issue 198.
> 
> Fixes: https://github.com/oasis-tcs/virtio-spec/issues/198
> 
> Matias


OK. Can you pls add a bit more detail about the issue and
the proposal in github?