From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:59095) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vd5LP-0005pP-QT for qemu-devel@nongnu.org; Sun, 03 Nov 2013 16:36:34 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vd5LJ-0003wR-Q4 for qemu-devel@nongnu.org; Sun, 03 Nov 2013 16:36:27 -0500 Received: from cantor2.suse.de ([195.135.220.15]:49626 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vd5LJ-0003wM-GM for qemu-devel@nongnu.org; Sun, 03 Nov 2013 16:36:21 -0500 Message-ID: <5276C1D0.5040103@suse.de> Date: Sun, 03 Nov 2013 22:36:16 +0100 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1383230256-22637-1-git-send-email-afaerber@suse.de> <52764F63.5090501@suse.de> <527667F6.3010003@suse.de> <52769B63.7080701@suse.de> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PULL v2 00/56] QOM devices patch queue 2013-10-31 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell , Anthony Liguori Cc: "Mian M. Hamayun" , "Michael S. Tsirkin" , Marcel Apfelbaum , qemu-devel , Stefan Hajnoczi , Paolo Bonzini , "Edgar E. Iglesias" Am 03.11.2013 19:58, schrieb Peter Maydell: > On 3 November 2013 18:52, Andreas F=C3=A4rber wrote: >> I believe that info is already outdated as per IRC. A patch to ossaudi= o >> 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. >=20 > 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=3Dpa 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 --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg