lkml.org 
[lkml]   [2009]   [Aug]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 2/2] sound: make OSS device number claiming optional
Hello, Alan.

Alan Cox wrote:
> On Wed, 05 Aug 2009 18:32:32 +0900
> Tejun Heo <teheo@suse.de> wrote:
>
>> Hello, Alan.
>>
>> Alan Cox wrote:
>>> This looks like a random private devel hack sand
>> Why so?
>>
>>> I don't see why it's appropriate for mainstream, especially as these
>>> "new things" don't exist mainstream either.
>> The new things is CUSE and it's already in mainstream.
>
> Then fix it to use the sound interfaces properly. We have a multiplexor
> for sound devices so they can be shared between OSS/ALSA/whatever. Use it.
> Ditto if you want CUSE to handle miscdevices use the miscdevice interface
> properly.

The only problem here is the now obsolete hackish way OSS module
auto-loading is handled. CUSE is being a regular chardev citizen.
The OSS multiplexing mechanism itself isn't a problem and the posted
patch doesn't distort it in any way but by preclaiming all OSS device
numbers, OSS requires its own different rule for no good reason. The
only actual difference here would be what module aliases are used.

Given the situation, I sure can add hacks to CUSE so that it doesn't
do regular chardev thing but does OSS specific stuff if OSS major is
detected. To me, this would be much painful exercise in spreading the
hack.

The proper solution would be extending chardev such that OSS sound
core can use its own custom module aliases for backward compatibility.
But given that OSS is about the only user left for such facility and
modern distros don't depend on the feature or can easily work around
it from userland, that would be an overkill.

Another option is to rip the custom module requesting code altogether.
It just doesn't matter at all anymore. modprobe is smart enough to
work around those issues with a couple lines of configuration changes.
We can add those alises to feature-removal-schedule.txt and then kill
it in a year or so. If OSS is still alive and well, I would have
chosen this path.

However, in-kernel OSS is largely dead or at least is dying a slow
death. Native OSS drivers are no longer in development and the only
left user is in-kernel emulation from ALSA, which I'm sure have its
places but is getting more and more inadequate for general desktop
usage due to lack of software multiplexing.

I don't see much reason to go through feature removal process for the
now obsolete OSS-specific module requesting feature. Allowing distros
to make the switch as they see fit and letting the code wither away is
the least painful way.

So, in summary, the hack here is how OSS does custom module requesting
and the proper solution would be removing the mis-feature according to
our feature removal protocol or reimplementing it in a non-hackish
way, both of which are unnecessarily painful and overkill at this
point.

Thanks.

--
tejun


\
 
 \ /
  Last update: 2009-08-05 13:29    [W:0.052 / U:1.532 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site