Re: Linux v2.6.16-rc5

[Rene Herman <rene.herman@keyaccess.nl>]

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/