* [PATCH] fuzz: specify audiodev for usb-audio
@ 2024-05-27 4:07 Alexander Bulekov
2024-05-27 5:34 ` Thomas Huth
0 siblings, 1 reply; 4+ messages in thread
From: Alexander Bulekov @ 2024-05-27 4:07 UTC (permalink / raw)
To: qemu-devel
Cc: Alexander Bulekov, Thomas Huth, Paolo Bonzini, Bandan Das,
Stefan Hajnoczi, Darren Kenny, Qiuhao Li, Laurent Vivier
Fixes test-failure on Fedora 40 CI.
Reported-by: Thomas Huth <thuth@redhat.com>
Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
---
tests/qtest/fuzz/generic_fuzz_configs.h | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/tests/qtest/fuzz/generic_fuzz_configs.h b/tests/qtest/fuzz/generic_fuzz_configs.h
index 4d7c8ca4ec..ef0ad95712 100644
--- a/tests/qtest/fuzz/generic_fuzz_configs.h
+++ b/tests/qtest/fuzz/generic_fuzz_configs.h
@@ -150,7 +150,8 @@ const generic_fuzz_config predefined_configs[] = {
"-chardev null,id=cd0 -chardev null,id=cd1 "
"-device usb-braille,chardev=cd0 -device usb-ccid -device usb-ccid "
"-device usb-kbd -device usb-mouse -device usb-serial,chardev=cd1 "
- "-device usb-tablet -device usb-wacom-tablet -device usb-audio",
+ "-device usb-tablet -device usb-wacom-tablet "
+ "-device usb-audio,audiodev=snd0 -audiodev none,id=snd0",
.objects = "*usb* *uhci* *xhci*",
},{
.name = "pc-i440fx",
--
2.43.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] fuzz: specify audiodev for usb-audio
2024-05-27 4:07 [PATCH] fuzz: specify audiodev for usb-audio Alexander Bulekov
@ 2024-05-27 5:34 ` Thomas Huth
2024-05-27 14:07 ` Alexander Bulekov
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Huth @ 2024-05-27 5:34 UTC (permalink / raw)
To: Alexander Bulekov, qemu-devel
Cc: Paolo Bonzini, Bandan Das, Stefan Hajnoczi, Darren Kenny,
Qiuhao Li, Laurent Vivier
On 27/05/2024 06.07, Alexander Bulekov wrote:
> Fixes test-failure on Fedora 40 CI.
>
> Reported-by: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
> ---
> tests/qtest/fuzz/generic_fuzz_configs.h | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/qtest/fuzz/generic_fuzz_configs.h b/tests/qtest/fuzz/generic_fuzz_configs.h
> index 4d7c8ca4ec..ef0ad95712 100644
> --- a/tests/qtest/fuzz/generic_fuzz_configs.h
> +++ b/tests/qtest/fuzz/generic_fuzz_configs.h
> @@ -150,7 +150,8 @@ const generic_fuzz_config predefined_configs[] = {
> "-chardev null,id=cd0 -chardev null,id=cd1 "
> "-device usb-braille,chardev=cd0 -device usb-ccid -device usb-ccid "
> "-device usb-kbd -device usb-mouse -device usb-serial,chardev=cd1 "
> - "-device usb-tablet -device usb-wacom-tablet -device usb-audio",
> + "-device usb-tablet -device usb-wacom-tablet "
> + "-device usb-audio,audiodev=snd0 -audiodev none,id=snd0",
> .objects = "*usb* *uhci* *xhci*",
> }
Reviewed-by: Thomas Huth <thuth@redhat.com>
The patch makes sense and I think we should include it, thanks! .. but I
still don't understand why the behavior of the fuzzing job was different
between Fedora 38 and 40, do you? Why does it complain about "no default
audio driver available" on F40 but works fine on F38, though both build jobs
include the Alsa, pulseaudio and OSS backends?
Thomas
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fuzz: specify audiodev for usb-audio
2024-05-27 5:34 ` Thomas Huth
@ 2024-05-27 14:07 ` Alexander Bulekov
2024-05-27 14:37 ` Alexander Bulekov
0 siblings, 1 reply; 4+ messages in thread
From: Alexander Bulekov @ 2024-05-27 14:07 UTC (permalink / raw)
To: Thomas Huth
Cc: qemu-devel, Paolo Bonzini, Bandan Das, Stefan Hajnoczi,
Darren Kenny, Qiuhao Li, Laurent Vivier
On 240527 0734, Thomas Huth wrote:
> On 27/05/2024 06.07, Alexander Bulekov wrote:
> > Fixes test-failure on Fedora 40 CI.
> >
> > Reported-by: Thomas Huth <thuth@redhat.com>
> > Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
> > ---
> > tests/qtest/fuzz/generic_fuzz_configs.h | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/tests/qtest/fuzz/generic_fuzz_configs.h b/tests/qtest/fuzz/generic_fuzz_configs.h
> > index 4d7c8ca4ec..ef0ad95712 100644
> > --- a/tests/qtest/fuzz/generic_fuzz_configs.h
> > +++ b/tests/qtest/fuzz/generic_fuzz_configs.h
> > @@ -150,7 +150,8 @@ const generic_fuzz_config predefined_configs[] = {
> > "-chardev null,id=cd0 -chardev null,id=cd1 "
> > "-device usb-braille,chardev=cd0 -device usb-ccid -device usb-ccid "
> > "-device usb-kbd -device usb-mouse -device usb-serial,chardev=cd1 "
> > - "-device usb-tablet -device usb-wacom-tablet -device usb-audio",
> > + "-device usb-tablet -device usb-wacom-tablet "
> > + "-device usb-audio,audiodev=snd0 -audiodev none,id=snd0",
> > .objects = "*usb* *uhci* *xhci*",
> > }
>
> Reviewed-by: Thomas Huth <thuth@redhat.com>
>
> The patch makes sense and I think we should include it, thanks! .. but I
> still don't understand why the behavior of the fuzzing job was different
> between Fedora 38 and 40, do you? Why does it complain about "no default
> audio driver available" on F40 but works fine on F38, though both build jobs
> include the Alsa, pulseaudio and OSS backends?
>
Yes that's strange, if the behavior is different.
The full config here is:
.args = "-machine q35 -nodefaults "
"-drive file=null-co://,if=none,format=raw,id=disk0 "
"-device qemu-xhci,id=xhci -device usb-tablet,bus=xhci.0 "
"-device usb-bot -device usb-storage,drive=disk0 "
"-chardev null,id=cd0 -chardev null,id=cd1 "
"-device usb-braille,chardev=cd0 -device usb-ccid -device usb-ccid "
"-device usb-kbd -device usb-mouse -device usb-serial,chardev=cd1 "
"-device usb-tablet -device usb-wacom-tablet "
"-device usb-audio"
This contains "nodefaults", which should always be causing the error,
since this change:
c753bf479a ("audio: disable default backends if -audio/-audiodev is used")
This adds audio to qemu_disable_default_devices, which should be called
by -nodefaults. I wonder if for some reason the order in which the
audiodev is configured and the default audiodev is disabled is different
between builds.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] fuzz: specify audiodev for usb-audio
2024-05-27 14:07 ` Alexander Bulekov
@ 2024-05-27 14:37 ` Alexander Bulekov
0 siblings, 0 replies; 4+ messages in thread
From: Alexander Bulekov @ 2024-05-27 14:37 UTC (permalink / raw)
To: Thomas Huth
Cc: qemu-devel, Paolo Bonzini, Bandan Das, Stefan Hajnoczi,
Darren Kenny, Qiuhao Li, Laurent Vivier
On 240527 1007, Alexander Bulekov wrote:
> On 240527 0734, Thomas Huth wrote:
> > On 27/05/2024 06.07, Alexander Bulekov wrote:
> > > Fixes test-failure on Fedora 40 CI.
> > >
> > > Reported-by: Thomas Huth <thuth@redhat.com>
> > > Signed-off-by: Alexander Bulekov <alxndr@bu.edu>
> > > ---
> > > tests/qtest/fuzz/generic_fuzz_configs.h | 3 ++-
> > > 1 file changed, 2 insertions(+), 1 deletion(-)
> > >
> > > diff --git a/tests/qtest/fuzz/generic_fuzz_configs.h b/tests/qtest/fuzz/generic_fuzz_configs.h
> > > index 4d7c8ca4ec..ef0ad95712 100644
> > > --- a/tests/qtest/fuzz/generic_fuzz_configs.h
> > > +++ b/tests/qtest/fuzz/generic_fuzz_configs.h
> > > @@ -150,7 +150,8 @@ const generic_fuzz_config predefined_configs[] = {
> > > "-chardev null,id=cd0 -chardev null,id=cd1 "
> > > "-device usb-braille,chardev=cd0 -device usb-ccid -device usb-ccid "
> > > "-device usb-kbd -device usb-mouse -device usb-serial,chardev=cd1 "
> > > - "-device usb-tablet -device usb-wacom-tablet -device usb-audio",
> > > + "-device usb-tablet -device usb-wacom-tablet "
> > > + "-device usb-audio,audiodev=snd0 -audiodev none,id=snd0",
> > > .objects = "*usb* *uhci* *xhci*",
> > > }
> >
> > Reviewed-by: Thomas Huth <thuth@redhat.com>
> >
> > The patch makes sense and I think we should include it, thanks! .. but I
> > still don't understand why the behavior of the fuzzing job was different
> > between Fedora 38 and 40, do you? Why does it complain about "no default
> > audio driver available" on F40 but works fine on F38, though both build jobs
> > include the Alsa, pulseaudio and OSS backends?
> >
>
> Yes that's strange, if the behavior is different.
> The full config here is:
>
> .args = "-machine q35 -nodefaults "
> "-drive file=null-co://,if=none,format=raw,id=disk0 "
> "-device qemu-xhci,id=xhci -device usb-tablet,bus=xhci.0 "
> "-device usb-bot -device usb-storage,drive=disk0 "
> "-chardev null,id=cd0 -chardev null,id=cd1 "
> "-device usb-braille,chardev=cd0 -device usb-ccid -device usb-ccid "
> "-device usb-kbd -device usb-mouse -device usb-serial,chardev=cd1 "
> "-device usb-tablet -device usb-wacom-tablet "
> "-device usb-audio"
>
> This contains "nodefaults", which should always be causing the error,
> since this change:
> c753bf479a ("audio: disable default backends if -audio/-audiodev is used")
>
> This adds audio to qemu_disable_default_devices, which should be called
> by -nodefaults. I wonder if for some reason the order in which the
> audiodev is configured and the default audiodev is disabled is different
> between builds.
>
It might be even simpler. The recent jobs on Fedora 38 don't seem to
ever even make it to the xhci test, because of a detected leak:
https://gitlab.com/qemu-project/qemu/-/jobs/6949417820#L4375
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-05-27 14:38 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-27 4:07 [PATCH] fuzz: specify audiodev for usb-audio Alexander Bulekov
2024-05-27 5:34 ` Thomas Huth
2024-05-27 14:07 ` Alexander Bulekov
2024-05-27 14:37 ` Alexander Bulekov
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).