From: "Andreas Färber" <afaerber@suse.de>
To: Peter Maydell <peter.maydell@linaro.org>,
Anthony Liguori <anthony@codemonkey.ws>
Cc: "Mian M. Hamayun" <m.hamayun@virtualopensystems.com>,
"Michael S. Tsirkin" <mst@redhat.com>,
Marcel Apfelbaum <marcel.a@redhat.com>,
qemu-devel <qemu-devel@nongnu.org>,
Stefan Hajnoczi <stefanha@redhat.com>,
Paolo Bonzini <pbonzini@redhat.com>,
"Edgar E. Iglesias" <edgar.iglesias@gmail.com>
Subject: Re: [Qemu-devel] [PULL v2 00/56] QOM devices patch queue 2013-10-31
Date: Sun, 03 Nov 2013 22:36:16 +0100 [thread overview]
Message-ID: <5276C1D0.5040103@suse.de> (raw)
In-Reply-To: <CAFEAcA8babqDwb5Gr6Mp+HkX+uBNKin9xDuw7GQOxqiLNLZ0FQ@mail.gmail.com>
Am 03.11.2013 19:58, schrieb Peter Maydell:
> On 3 November 2013 18:52, Andreas Färber <afaerber@suse.de> wrote:
>> I believe that info is already outdated as per IRC. A patch to ossaudio
>> has been submitted to change the default away from OSS for virgin
>> ./configure. Problem is lack of /dev/dsp on some distros and
>> ossaudio.c's error handling for that case.
>
> Oh, is it just those warnings? The audio system tends to
> spew warnings if you don't have your local system's audio
> configured properly, but that has been the case for literally
> years (ie as long as I've worked with QEMU). That's not a new
> bug by any stretch of the imagination.
Having installed, updated and set up a Fedora 19 VM without /dev/dsp, I
am indeed still getting slightly different output than Anthony:
GTESTER check-qtest-arm
oss: Could not initialize DAC
oss: Failed to open `/dev/dsp'
oss: Reason: No such file or directory
oss: Could not initialize DAC
oss: Failed to open `/dev/dsp'
oss: Reason: No such file or directory
audio: Failed to create voice `lm4549.out'
repeated several times.
So it seems I was right and you too, Peter, it's just annoyingly noisy
audio output but definitely no qtest failure.
With Anthony's proposed ossaudio patch both the oss and the audio lines
get suppressed.
On openSUSE with QEMU_AUDIO_DRV=pa I am seeing:
GTESTER check-qtest-arm
pulseaudio: set_sink_input_volume() failed
pulseaudio: Reason: Invalid argument
pulseaudio: set_sink_input_mute() failed
pulseaudio: Reason: Invalid argument
repeated several times, but still no test failure, just the noise.
Meanwhile Marcel and Michael have been working on resolving the test
failures I reported. Once we have agreed on a solution there, we'll need
a v3 since mst's pull included the original version of the qom patch
with incorrect documentation syntax that would now cause silent merge
conflicts - my qom-next has the delta as follow-up fix instead.
Regards,
Andreas
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
prev parent reply other threads:[~2013-11-03 21:36 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-10-31 14:37 [Qemu-devel] [PULL v2 00/56] QOM devices patch queue 2013-10-31 Andreas Färber
2013-10-31 21:03 ` Anthony Liguori
2013-11-03 13:28 ` Andreas Färber
2013-11-03 13:58 ` Anthony Liguori
2013-11-03 15:12 ` Andreas Färber
2013-11-03 16:32 ` Anthony Liguori
2013-11-03 18:47 ` Peter Maydell
2013-11-03 18:52 ` Andreas Färber
2013-11-03 18:58 ` Peter Maydell
2013-11-03 21:36 ` Andreas Färber [this message]
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=5276C1D0.5040103@suse.de \
--to=afaerber@suse.de \
--cc=anthony@codemonkey.ws \
--cc=edgar.iglesias@gmail.com \
--cc=m.hamayun@virtualopensystems.com \
--cc=marcel.a@redhat.com \
--cc=mst@redhat.com \
--cc=pbonzini@redhat.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=stefanha@redhat.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.