Because the mmap solution introduces extra overhead in kernel. Customers who don't want dmix feature are also affected by the performance down. So, a better way is to keep both solution in the driver and give the choice to customers.
Sonic -----Original Message----- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Mike Frysinger Sent: Tuesday, February 05, 2008 2:04 PM To: Cai, Cliff Cc: [EMAIL PROTECTED]; [email protected] Subject: Re: [Linux-kernel-commits] [4196] trunk/sound: bug[#3577] add dmixsupport for AD1981, enable mmap for the soundcard driver On Jan 31, 2008 8:28 PM, Cai, Cliff <[EMAIL PROTECTED]> wrote: > I added a new config file to the alsa lib that is AD1980-MMAP.conf > Because the we need a different config file to support mmap way. > And app find out the config file by the codec name. i dont think this is a scalable (or acceptable to mainline) solution. why dont we keep the codecs untouched and default to mmap (and thus dmix) enabled. then if people want to disable dmix/mmap, we document the relevant changes they need to make in the alsa config file for an individual codec. or if we want to get fancy, we can write an automatic piece of code in the alsa-lib makefile that will auto run sed/patch to convert the mmap verison of the config file to the non-mmap version. -mike _______________________________________________ Linux-kernel-commits mailing list [email protected] http://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits _______________________________________________ Linux-kernel-commits mailing list [email protected] http://blackfin.uclinux.org/mailman/listinfo/linux-kernel-commits
