[lkml]   [2007]   [May]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH] Factor out common MODULE_INFO content from module*.h files.
    On Sun, 20 May 2007, Sam Ravnborg wrote:

    > On Sun, May 20, 2007 at 03:06:15PM -0400, Robert P. J. Day wrote:
    > >
    > > In order to eventually break the interdependency between the module.h
    > > and moduleparam.h header files, factor out the common MODULE_INFO
    > > content into a new header file.
    > The moduleinfo.h file looks redundant at first look.
    > Why not push relevant parts from moduleparam.h (the
    > MODULE_INFO bits) to module.h and let go of
    > the include of moduleparam.h in module.h (when you
    > have fixed the users)?
    > In this way we do not add an extra .h file.
    > And files that needs moduleparam.h will anyway always need module.h.
    > But not the other way around.

    no problem, i can go that way, too, but there's just one (admittedly
    picky) issue associated with that.

    based on the above, we would have:

    1) module.h handling all generic module content, and
    2) moduleparam.h would "#include" module.h and add the parameter

    fair enough, but note that, with that, if you wanted parameter
    support, you would need to include *only* "moduleparam.h". are you
    good with that? (as i said, it's picky, but you'd probably still have
    a lot of people who, through force of habit, would still #include both
    just because they think it's necessary. wouldn't hurt, of course,
    since module.h would be protected against multiple inclusion.)

    so if you're good with all of the above, i can do that.

    Robert P. J. Day
    Linux Consulting, Training and Annoying Kernel Pedantry
    Waterloo, Ontario, CANADA
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2007-05-20 21:55    [W:0.021 / U:5.168 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site