corresponding patches to [RFC]

Dominik Brodowski linux at brodo.de
Mon Jul 7 15:47:20 BST 2003


On Sun, Jul 06, 2003 at 11:34:29PM +0100, Russell King wrote:
> On Thu, Jun 12, 2003 at 09:47:51PM +0200, Dominik Brodowski wrote:
> > Just a few changes to the RFC, the most important is that for static mapped
> > sockets its now enough to do "echo 1 > /sys/.../settings/start" [once all
> > other things are done properly, of course...]
> > 
> > 	Dominik
> > 
> > BTW: these changes are only released with the GPL v.2
> 
> Appologies for the delay in getting to this.

No problem, have been re-working these patches a bit in the past few days;
hopefully I'll get to publish a complete new patchset today.

> Can you clear the license change with David Hinds please?  I suspect that
> without David saying "ok" to it, and maybe asking for us to change the
> MAINTAINERS entry, Linus may not accept the patch.

To clarify some things: I don't care whether it's GPL v.2 or GPL/MPL.
However, some of the code I want to patch in is clearly based on differenct
GPL-only code. That's why I consider this step to be necessary. Nonetheless,
I'll ask David.

> > +	.ktype = &ktype_pcmcia_settings,
> 
> Hmm, this is adding attributes to the "pcmcia" bus, placing them in
> a directory called /sysfs/bus/pcmcia/settings ?

Aww, this seems to be a very old patchset you're reviewing here...

> I'm also slightly worried about the lifetime of the pcmcia_bus object.
> Although bus_unregister() makes it disappear from the sysfs tree, any
> references to that object and any sub-object will cause other code in
> the kernel to keep a reference to the pcmcia_bus_type, which gets
> really messy when the module whose storage it was using has long
> since gone.

Any module using pcmcia_bus will increase the refcount of ds.ko. Doesn't
this solve this problem?

	Dominik



More information about the linux-pcmcia mailing list