[lkml]   [2007]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [ofa-general] [PATCH 1/5] ehca: Supports large page MRs
    I applied this, but I agree with

    > WARNING: externs should be avoided in .c files
    > #227: FILE: drivers/infiniband/hw/ehca/ehca_mrmw.c:67:
    > +extern int ehca_mr_largepage;
    > WARNING: externs should be avoided in .c files
    > #949: FILE: drivers/infiniband/hw/ehca/hcp_if.c:753:
    > + extern int ehca_debug_level;

    if you need to use a variable in more than one .c file, put the extern
    declaration in a common header that's included everywhere you use the
    variable, including the .c file that it is defined in. That way the
    compiler can see if you get confused about the type of the variable.

    When you get a chance, please post a follow-on patch to fix this.

    - R.
    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-07-21 06:09    [W:0.024 / U:3.516 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site