From: Colin Guthrie <cguthrie@mandriva.org>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: Tejun Heo <teheo@suse.de>, Greg KH <greg@kroah.com>,
Al Viro <viro@ftp.linux.org.uk>, Takashi Iwai <tiwai@suse.de>,
Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] sound: make OSS device number claiming optional
Date: Wed, 05 Aug 2009 10:24:09 +0100 [thread overview]
Message-ID: <4A794FB9.30403@mandriva.org> (raw)
In-Reply-To: <20090805101551.6ee053e5@lxorguk.ukuu.org.uk>
'Twas brillig, and Alan Cox at 05/08/09 10:15 did gyre and gimble:
> On Wed, 05 Aug 2009 15:40:42 +0900
> Tejun Heo <teheo@suse.de> wrote:
>
>> If any OSS support is enabled, regardless of built-in or module,
>> sound_core claims full OSS major number (that is, the old 0-255
>> region) to trap open attempts and request sound module automatically.
>> This feature is redundant as chrdev already has such mechanism and no
>> longer used by modern distros. This preemptive claiming prevents
>> alternative OSS implementation.
>>
>> This patch makes the preclaiming optional and adds a config option
>> SOUND_OSS_CORE_PRECLAIM and kernel parameter soundcore.preclaim_oss to
>> control whether preclaim is enabled or not. This allows distros and
>> developers to try new things without rebuilding kernel.
>
> This looks like a random private devel hack sand I don't see why it's
> appropriate for mainstream, especially as these "new things" don't exist
> mainstream either.
While I'm not familiar enough with the kernel itself to comment on *how*
the patch works, the principle is quite important here.
As far as I know most distros enable snd-*-oss module loading via a
modprobe trick (e.g. in user space) so the change here shouldn't affect
this approach.
However, I certainly want to experiment with osspd and while I hope it's
going to be a good solution generally, it'll only make sense if the user
chooses to use pulseaudio.
If the user decided they want to use pure alsa, then they'll have to
drop back to using snd-*-oss for that instead.
This is all something I'd like to enable without making the user change
kernels based on a preference.
So if this patch is not accepted, I'd still like to see the kernel not
claim this device number, and leave it up to userspace to load the modules.
Sorry if this is not all factually accurate, but I think i've got the
gist right! :)
--
Colin Guthrie
cguthrie(at)mandriva.org
http://colin.guthr.ie/
Day Job:
Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
Mandriva Linux Contributor [http://www.mandriva.com/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
next prev parent reply other threads:[~2009-08-05 9:33 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-05 6:35 [PATCH 1/2] chrdev: implement __[un]register_chrdev() Tejun Heo
2009-08-05 6:40 ` [PATCH 2/2] sound: make OSS device number claiming optional Tejun Heo
2009-08-05 9:15 ` Alan Cox
2009-08-05 9:24 ` Colin Guthrie [this message]
2009-08-05 9:59 ` Alan Cox
2009-08-05 10:14 ` Takashi Iwai
2009-08-05 10:26 ` Alan Cox
2009-08-05 10:45 ` Takashi Iwai
2009-08-05 11:15 ` Alan Cox
2009-08-05 11:34 ` Tejun Heo
2009-08-05 12:35 ` Tejun Heo
2009-08-05 13:11 ` Alan Cox
2009-08-05 14:16 ` Tejun Heo
2009-08-05 9:32 ` Tejun Heo
2009-08-05 10:00 ` Alan Cox
2009-08-05 11:27 ` Tejun Heo
2009-08-05 12:48 ` Alan Cox
2009-08-05 14:13 ` Tejun Heo
2009-08-05 14:29 ` Alan Cox
2009-08-05 16:02 ` Tejun Heo
2009-08-05 16:33 ` Alan Cox
2009-08-05 16:38 ` Alan Cox
2009-08-05 16:52 ` Tejun Heo
2009-08-05 17:01 ` Alan Cox
2009-08-06 5:55 ` Tejun Heo
2009-08-05 7:04 ` [PATCH 1/2] chrdev: implement __[un]register_chrdev() Takashi Iwai
2009-08-05 7:11 ` Tejun Heo
2009-08-05 7:20 ` Takashi Iwai
2009-08-05 7:30 ` Tejun Heo
2009-08-05 9:01 ` [PATCH 1/2 UPDATED] " Tejun Heo
2009-08-05 16:16 ` [PATCH 1/2] " Greg KH
2009-08-05 16:30 ` Tejun Heo
2009-08-05 16:49 ` Greg KH
2009-08-05 17:01 ` Tejun Heo
2009-08-05 17:15 ` Greg KH
2009-08-06 5:52 ` Tejun Heo
2009-08-06 8:13 ` Tejun Heo
2009-08-06 19:58 ` Greg KH
2009-08-07 2:34 ` Tejun Heo
2009-08-07 4:05 ` Greg KH
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=4A794FB9.30403@mandriva.org \
--to=cguthrie@mandriva.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=greg@kroah.com \
--cc=linux-kernel@vger.kernel.org \
--cc=teheo@suse.de \
--cc=tiwai@suse.de \
--cc=viro@ftp.linux.org.uk \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox