[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define
From: |
malc |
Subject: |
[Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define |
Date: |
Sun, 26 Jul 2009 22:21:15 +0400 (MSD) |
On Sun, 26 Jul 2009, Avi Kivity wrote:
> On 07/26/2009 06:09 PM, Jan Kiszka wrote:
> > Avi Kivity wrote:
> >
> > > On 07/26/2009 04:41 PM, Juan Quintela wrote:
> > >
> > > > 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 the
> > > > default order is far from trivial.
> > > >
> > > >
> > > You could pass a goodness value to the registration function, reflecting
> > > what we think the best driver is. Then at runtime select the best
> > > driver that is supported on the system.
> > >
> > >
> >
> > 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.
> >
>
> Yes, that is better. Decouple the goodness from the driver, and allow the
> user to override it.
>
> > 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 it
> > 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 all
> > available drivers.
Nothing has changed since we had our private e-mail exchange on the
subject a while ago. As for 'OSS and only OSS as the default' i'm not
sure what that really means, maybe this is an answer though - OSS is
the only thing that doesn't require additional headers and libraries,
and just works.
> >
> >
>
> I have some secondhand memories of pulse and alsa generating very bad output.
I have some firsthand memories of this.
--
mailto:address@hidden
- Re: [Qemu-devel] [PATCH 45/46] Remove AUDIO_DRIVERS define, (continued)
- Re: [Qemu-devel] [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Juan Quintela, 2009/07/26
- Re: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Avi Kivity, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Jan Kiszka, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Avi Kivity, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define,
malc <=
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Jan Kiszka, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, malc, 2009/07/26
- Re: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26
[Qemu-devel] [PATCH 21/46] rename HAVE_FNMATCH_H to CONFIG_FNMATCH, Juan Quintela, 2009/07/24
[Qemu-devel] [PATCH 02/46] Remove SP_CFLAGS and SP_LDFLAGS, Juan Quintela, 2009/07/24
[Qemu-devel] [PATCH 05/46] use same algorithm for testing and, Juan Quintela, 2009/07/24
[Qemu-devel] [PATCH 04/46] remove CONFIG_ from VNC_TLS_{LIBS, FLAGS}, Juan Quintela, 2009/07/24
[Qemu-devel] [PATCH 07/46] remove CONFIG_ from BLUEZ_{LIBS,FLAGS}, Juan Quintela, 2009/07/24
[Qemu-devel] [PATCH 01/46] change HOST_SOLARIS to CONFIG_SOLARIS{_VERSION}, Juan Quintela, 2009/07/24