lkml.org 
[lkml]   [2006]   [Feb]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: Linux v2.6.16-rc5
Andrew Morton wrote:

> Rene Herman <rene.herman@keyaccess.nl> wrote:

>> This one isn't in: http://lkml.org/lkml/2006/2/21/7
>>
>> Andrew did pick it up -- pnp-bus-type-fix.patch, named as being in the
>> 2.6.16 queue in his 2.6.16-rc4-mm2 announce:
>>
>> http://lkml.org/lkml/2006/2/24/66
>>
>> so it's probably okay. The other two patches from that same thread
>> already made it into -rc5 though, so thought I'd ping anyway. It does
>> really want to make 2.6.16. Many ISA-PnP drivers are quite severely
>> broken without (it's also a regression against 2.6.15).
>>
>
> Problem is, that patch was just a "here, try this" thing which Adam slung
> onto the mailing list - I have no idea whether it was compete or final or
> whether he wants it in 2.6.16 or what.

Adam? But something will need to go in. At the moment an entire bus
subsystem appears to be broken.

> No indication of what problem it's fixing, nor how, now what risk
> there is of breaking something else. It's just a lonely little diff
> at present.

The problem it fixed for me was that the CS4236 ALSA driver's private
PnP remove method was not being called at modprobe -r, which meant that
the card wasn't being freed at all, resulting in memory leaks, the
inability to reload the driver, and oopses, during modprobe -r and reboot.

All ALSA ISA card drivers, not just CS4236, use the same interface to
PnP (the pnp_card_driver struct) meaning they would all appear to be
broken in that exact same way as well. Or rather, _any_ ISA-PnP driver
using that pnp_card_driver interface (there's also drivers using the
pnp_driver interface -- those appear to be okay). CS4236 isn't doing
anything special...

The problem seems to be caused by the "bustype" driver model changes in
2.6.16, the same ones that made the sensors drivers complain about
private methods versus bustype methods which was fixed in -rc2. Adam
said that in fact not so much the teardown was broken, but the setup,
and the patch replaces a subsystem probe method with a bustype method.

As to the risk of it breaking anything else... I doubt it. Given that
the old method did not work _at all_ it seems this is simply the way to
do this.

Adam ofcourse is the PnP expert though...

Rene.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-02-28 00:33    [W:0.102 / U:0.136 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site