lkml.org 
[lkml]   [2004]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] s390 (8/10): zfcp fixes.
On Fri, Mar 12, 2004 at 09:29:16PM +0100, Martin Schwidefsky wrote:
>
>
>
>
> Hi Christoph,
>
> > > - Replace release function for device structures by kfree. Move struct
> > > device to the start of struct zfcp_port/zfcp_unit to make it work.
> >
> > That's ugly as hell. Actually even more ugly. It's not that ->release
> > is such a performance critical path that you absolutely need to avoid one
> > level of function calls. So please put a simple wrapper back instead of
> > the horrible casts and suddenly the silly placement restrictions are gone,
> > too.
>
> That it's ugly is true. But what's important is that it is required to get
> module ref-counting right. The release function is called after the last
> module_put has been done.

Huh? Is the scsi reference counting logic that messed up? If so, it
needs to be fixed.

And if so, why does it not show up with all of the other scsi drivers?

thanks,

greg k-h
-
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/

\
 
 \ /
  Last update: 2005-03-22 14:01    [W:0.029 / U:0.116 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site