From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1MV8R2-0005rL-4x for qemu-devel@nongnu.org; Sun, 26 Jul 2009 14:27:00 -0400 Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1MV8Qx-0005qO-N8 for qemu-devel@nongnu.org; Sun, 26 Jul 2009 14:26:59 -0400 Received: from [199.232.76.173] (port=57760 helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1MV8Qx-0005qL-FE for qemu-devel@nongnu.org; Sun, 26 Jul 2009 14:26:55 -0400 Received: from fmmailgate03.web.de ([217.72.192.234]:55991) by monty-python.gnu.org with esmtp (Exim 4.60) (envelope-from ) id 1MV8Qw-0004HE-Ss for qemu-devel@nongnu.org; Sun, 26 Jul 2009 14:26:55 -0400 Message-ID: <4A6C9FD8.3020808@web.de> Date: Sun, 26 Jul 2009 20:26:32 +0200 From: Jan Kiszka MIME-Version: 1.0 References: <4A6C5903.4000708@codemonkey.ws> <4A6C6B9C.2080909@redhat.com> <4A6C71A2.70904@web.de> <4A6C7372.8050908@redhat.com> In-Reply-To: <4A6C7372.8050908@redhat.com> Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="------------enig49E4B6A884FEE94BF2AD77D9" Sender: jan.kiszka@web.de Subject: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Avi Kivity Cc: aliguori@us.ibm.com, qemu-devel@nongnu.org, Juan Quintela This is an OpenPGP/MIME signed message (RFC 2440 and 3156) --------------enig49E4B6A884FEE94BF2AD77D9 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Avi Kivity wrote: > On 07/26/2009 06:09 PM, Jan Kiszka wrote: >> Avi Kivity wrote: >> =20 >>> On 07/26/2009 04:41 PM, Juan Quintela wrote: >>> =20 >>>> The order in audio_drivers is important, that means you can have a >>>> different default sound driver. >>>> >>>> After some thinking, my current plan is exporting audio_drv_list >>>> to the Makefile, and then generating something similar to >>>> AUDIO_DRIVERS. >>>> >>>> Notice that the "ordering" problems not only happens with sound >>>> drivers, >>>> we have the same problems with graphics console, which one is the >>>> default one? It depends on what you have configured, and changing t= he >>>> default order is far from trivial. >>>> >>>> =20 >>> You could pass a goodness value to the registration function, reflect= ing >>> what we think the best driver is. Then at runtime select the best >>> driver that is supported on the system. >>> >>> =20 >> >> I think the easiest way is to forward the sorted driver list as string= >> to the audio layer and let it be processed there. Given such an >> extension, we could also allow passing a list of drivers via >> QEMU_AUDIO_DRV. >> =20 >=20 > Yes, that is better. Decouple the goodness from the driver, and allow > the user to override it. >=20 >> BTW, I still wonder why we have OSS and only OSS as the default driver= >> list under Linux. Most distros I came across kicked it out or pushed i= t >> to the end of their list (as it requires exclusive access). Would be a= >> chance to improve the default. At least we should extend the list by a= ll >> available drivers. >> >> =20 >=20 > I have some secondhand memories of pulse and alsa generating very bad > output. >=20 I have fresh memories that pulse still suffers on my Suse while alsa works smoothly (and that with default settings since malc's last major fixes). Fedora prefers pulse over alsa today (but keeps OSS in the list). Suse only provides alsa for qemu-0.10. Ubuntu keeps OSS as #1, but at least adds alsa and pulse to the list. Jan --------------enig49E4B6A884FEE94BF2AD77D9 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.9 (GNU/Linux) Comment: Using GnuPG with SUSE - http://enigmail.mozdev.org iEYEARECAAYFAkpsn+wACgkQniDOoMHTA+ntFwCcCpc5eD3FXYhTeeBwIPKMMUjp Or4An2OYLbTWBF0AT765EKcHi8u2dz1O =DJ38 -----END PGP SIGNATURE----- --------------enig49E4B6A884FEE94BF2AD77D9--