Re: [PATCH] s390 (2/7): common i/o layer.

From: Christoph Hellwig
Date: Mon Oct 06 2003 - 09:57:35 EST


On Mon, Oct 06, 2003 at 04:49:25PM +0200, Martin Schwidefsky wrote:
> > > Just checked. You right about chp_release which should do
> > > a kfree on the struct channel_path object. But the two
> > > other release functions are really dummy functions because
> > > cu3088_root_dev and iucv_root are static structures.
> >
> > Even in that case you're screwed in case they are in modules..
>
> Why? The root device are registered in the module init function
> and unregistered in the module exit function. I fail to see the
> problem.

You can still have a reference to the object when the module is unloaded.

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