sound: tasks to move vhost-device-sound from staging

[MatiasVara]

The purpose of this issue is to figure out what needs to be done to move vhost-device-sound from staging. Some of the tasks are:

• Tests coverage, e.g., Alsa tests. This is fixed by sound: add tests to Alsa and other test stuff #579.
• Pipewire bug, see Crash during vhost-device-sound test execution #538 and Crash during vhost-device-sound test execution #538 (comment)
• Problem when compiling audio backends by using musl, see https://buildkite.com/rust-vmm/vhost-device-ci/builds/2105#018c47e1-bd2a-400c-bff3-47ba8e77ece0. This is fixed by sound: skip musl builds #581.

Is there anything else we need to do?

Thanks.

[epilys]

Sorry if I'm repeating myself, I legitimately don't remember where we settled for this; do we want musl builds or not?

[Ablu]

Sorry if I'm repeating myself, I legitimately don't remember where we settled for this; do we want musl builds or not?

IIRC nobody was against disabling them where they do not make sense. But I am not sure whether we agreed on a way to do that.

[stefano-garzarella]

Sorry if I'm repeating myself, I legitimately don't remember where we settled for this; do we want musl builds or not?

IIRC we agreed on disable musl build for the sound device. The problem was how. @Ablu suggested to take a look at the gpio device. I tried to see if we can do something in Cargo.toml, but without success. So IMHO for now we can do something similar of what we did in gpio.

[stefano-garzarella]

About alsa test, is someone working on it? (otherwise, I think we can put it in the exclude_path)

[epilys]

See #579

[stsquad]

Can this be closed now?

[MatiasVara]

Can this be closed now?

We are thinking to keep it open until https://gitlab.freedesktop.org/pipewire/pipewire-rs/-/merge_requests/210 is merged, if that's OK.

[MatiasVara]

MR 210 has been merged, I'll close this issue after #606 is merged.

[stefano-garzarella]

#606 is now merged, closing this ;-)