[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [virtio-dev] virtio-snd comments/questions
Hi Matias, On 07.11.2023 00:18, Matias Ezequiel Vara Larsen wrote:
On Fri, Nov 03, 2023 at 06:17:17PM +0000, Radu Ocica wrote:I also think that the driver should not send a STOP command after a failed START command. There is no explicit description of any states in the virtio sound device specification. We have both adopted an interpretation that the state is determined by the last successful pcm control request and the transitions specified for pcm control requests assume success of these control requests. I wish this was clarified explicitly in the spec, to not allow room for interpretation.I agree. I think what is missed in the spec is to define the behavior of the PCM command cycle in case a command fails. I worked on a RFC to improve that section, feel free to comment at https://ddec1-0-en-ctp.trendmicro.com:443/wis/clicktime/v1/query?url=https%3a%2f%2flists.oasis%2dopen.org%2farchives%2fvirtio%2dcomment%2f202307%2fmsg00158.html&umid=76a8a4e7-2580-42db-b254-1053a2900dcf&auth=53c7c7de28b92dfd96e93d9dd61a23e634d2fbec-69dce88acbdf64019491f6342547f0e136609169. I think an overall solution would be to implement the PCM state machine in the driver thus control messages would be sent only if the stream is in the correct state.
Such a state machine already exists and is implemented in the ALSA subsystem of the kernel. The spec tries to simulate exactly this one. What you could do is to check it and fix possible issues in the spec. Kind regards,
Matias --------------------------------------------------------------------- To unsubscribe, e-mail: virtio-dev-unsubscribe@lists.oasis-open.org For additional commands, e-mail: virtio-dev-help@lists.oasis-open.org
-- Anton Yakovlev Senior Software Engineer OpenSynergy GmbH Rotherstr. 20, 10245 Berlin
[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]