Andrew Morton wrote:
Rene Herman <[email protected]> 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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
