public inbox for virtio-dev@lists.linux.dev
 help / color / mirror / Atom feed
From: Radu Ocica <rocica@blackberry.com>
To: Matias Ezequiel Vara Larsen <mvaralar@redhat.com>
Cc: "virtio-dev@lists.oasis-open.org"
	<virtio-dev@lists.oasis-open.org>,
	"anton.yakovlev@opensynergy.com" <anton.yakovlev@opensynergy.com>
Subject: Re: [virtio-dev] virtio-snd comments/questions
Date: Thu, 9 Nov 2023 16:46:45 +0000	[thread overview]
Message-ID: <37eaf0dc2bdb4a7b936618b48dd00ac2@blackberry.com> (raw)
In-Reply-To: <CAHYGQ0yXPY2NU25N3jh=MH7=PUjkWy5R-SmtaMOVP-VCPPuq3w@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2436 bytes --]


> Could you elaborate on this? Which audio backend would not support
> PAUSE? I am not very familiar with audio engines.

For instance in linux, alsa-lib has a flag that specifies pause support by a PCM stream:
#define SNDRV_PCM_INFO_PAUSE                       (1UL<<19)  /* pause ioctl is supported */
The snd_pcm_pause API can only be used successfully if the PCM stream has this bit set in its PCM info.
A similar concept exists in QNX sound.



>> This looks promising. I found it already at https://urldefense.com/v3/__https://github.com/torvalds/linux/blob/master/sound/virtio/__;!!JoeW-IhCUkS0Jg!bBq1USPrJoN1_CBJkW-XBknkCSEOUP40V1ctr_OU0tsYyEr9pXjSEu7n5ibc5BL4fKT3Utk-cpGw6FiWxDY$  and will give it a try. Is there a repository for this driver code where both this fix and the audio control support is available?
>>
>>

> What do you mean with audio control support? Are you talking about the
> device? The device implementation is at
https://urldefense.com/v3/__https://github.com/rust-vmm/vhost-device/tree/main/staging__;!!JoeW-IhCUkS0Jg!bBq1USPrJoN1_CBJkW-XBknkCSEOUP40V1ctr_OU0tsYyEr9pXjSEu7n5ibc5BL4fKT3Utk-cpGw1sNq6IY$ .

I meant the support for feature VIRTIO_SND_F_CTLS in the virtio_snd guest driver. A patch to add this support is available here: https://lists.oasis-open.org/archives/virtio-comment/202104/msg00013.html. However this patch is not present in the virtio_snd guest driver version here: https://urldefense.com/v3/__https://github.com/torvalds/linux/blob/master/sound/virtio/__;!!JoeW-IhCUkS0Jg!bBq1USPrJoN1_CBJkW-XBknkCSEOUP40V1ctr_OU0tsYyEr9pXjSEu7n5ibc5BL4fKT3Utk-cpGw6FiWxDY$ . I was wondering whether there is a repository where the virtio_snd driver has both the VIRTIO_SND_F_CTLS support and the ack callback patch.

----------------------------------------------------------------------
This transmission (including any attachments) may contain confidential information, privileged material (including material protected by the solicitor-client or other applicable privileges), or constitute non-public information. Any use of this information by anyone other than the intended recipient is prohibited. If you have received this transmission in error, please immediately reply to the sender and delete this information from your system. Use, dissemination, distribution, or reproduction of this transmission by unintended recipients is not authorized and may be unlawful.

[-- Attachment #2: Type: text/html, Size: 5077 bytes --]

  reply	other threads:[~2023-11-09 16:46 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-18 15:32 [virtio-dev] virtio-snd comments/questions Radu Ocica
2023-10-24 10:18 ` Matias Ezequiel Vara Larsen
2023-11-03 19:05   ` Radu Ocica
2023-11-06 15:09     ` Matias Ezequiel Vara Larsen
2023-11-09 16:46       ` Radu Ocica [this message]
2023-11-10  7:57         ` Manos Pitsidianakis
2023-11-13 11:16         ` Matias Ezequiel Vara Larsen
2023-11-22 18:59           ` Radu Ocica
2023-11-27 11:20             ` Matias Ezequiel Vara Larsen
2023-10-24 13:30 ` Matias Ezequiel Vara Larsen
2023-11-03 18:17   ` Radu Ocica
2023-11-06 15:18     ` Matias Ezequiel Vara Larsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=37eaf0dc2bdb4a7b936618b48dd00ac2@blackberry.com \
    --to=rocica@blackberry.com \
    --cc=anton.yakovlev@opensynergy.com \
    --cc=mvaralar@redhat.com \
    --cc=virtio-dev@lists.oasis-open.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox