Re: [PATCH] s390 (8/10): zfcp fixes.

[Greg KH <greg@kroah.com>]

On Mon, Mar 15, 2004 at 10:18:00AM +0100, Martin Schwidefsky wrote:
> 
> So we need an external release function, one that isn't part of the zfcp
> module. This external release function is either a generic function for
> all these objects or a dedicated release function for each of the
> additional
> device objects. A dedicated release function would mean to define a release
> function somewhere in the kernel or another module just for the purpose of
> freeing an object defined by the zfcp module. This is even more gross than
> to use a generic release function. And the simplest release function is
> kfree.

This is not ok.  If you have to do something like this, I really suggest
that you not allow the "sub modules" be able to unload before the upper
module can.  In fact, why would you want to do such a thing?

I still really strongly object to this patch.  If it's a scsi problem,
fix it there, but odds are it's your driver's problem as no other scsi
driver needs this.

thanks,

greg k-h