lkml.org 
[lkml]   [2003]   [Feb]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH/RFC] New module refcounting for net_proto_family
    At 07:46 PM 2/17/2003, David S. Miller wrote:

    >After talking to Alexey, I don't like this patch.
    >
    >The new module subsystem was supposed to deal with things
    >like this cleanly, and this patch is merely a hack to cover
    >up for it's shortcomings.
    No it's not. Are you guys saying that module refcounting in net/core/dev.c
    is a hack too ? Patch that I sent implements exactly the same logic.
    Grab module reference before creating net family socket and release
    module when socket is gone. Where is the hack here ?

    >To be honest, I'd rather just disallow module unloading or
    >let them stay buggy than put this hack into the tree.
    This comment makes no sense to me. Especially "let them stay buggy" part ;-).
    (I wonder what Alex could have told you).

    >Special hacks are for 2.4.x where things like full cleanups are not allowed.
    It's not a special hack. If it has problems let's fix them.
    I want to keep Bluetooth socket modules loadable and unloadable. And I'm sure
    Jean and other folks want's the same thing for IrDA and other subsystems with sockets.
    So, you guys would have to come with a better argument than "ohh it's an ugly hack" ;-).

    Alexey, could you please post what you told Dave here ?
    Rusty, any comments from you ?

    Here is the link to the patch that I sent
    http://marc.theaimsgroup.com/?l=linux-kernel&m=104308300808557&w=2

    and here is some explanation about protocol families
    http://marc.theaimsgroup.com/?l=linux-kernel&m=104317832227499&w=2

    Dave, Alex, seriously let's come up with the solution rather than ignoring
    the problem. I think my patch is a good solution. If you don't like overhead
    or whatever else let's fix that.

    Thanks
    Max

    -
    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 13:33    [W:2.277 / U:0.580 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site