* [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).