lkml.org 
[lkml]   [2007]   [Oct]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [2.6 patch] unexport icmpmsg_statistics
    From
    Date
    netdev-owner@vger.kernel.org wrote on 10/24/2007 12:14:37 PM:

    > On Wed, Oct 24, 2007 at 12:07:45PM -0700, David Stevens wrote:
    > > netdev-owner@vger.kernel.org wrote on 10/24/2007 09:24:10 AM:
    > >
    > > > This patch removes the unused EXPORT_SYMBOL(icmpmsg_statistics).
    > > >
    > > > Signed-off-by: Adrian Bunk <bunk@kernel.org>
    > > >
    > > > ---
    > > > 4ce74657ac0b1bdcb4c7bc359d05643f8cc4a08b
    > > > diff --git a/net/ipv4/icmp.c b/net/ipv4/icmp.c
    > > > index 272c69e..233de06 100644
    > > > --- a/net/ipv4/icmp.c
    > > > +++ b/net/ipv4/icmp.c
    > > > @@ -1104,5 +1104,4 @@ void __init icmp_init(struct net_proto_family
    > > *ops)
    > > > EXPORT_SYMBOL(icmp_err_convert);
    > > > EXPORT_SYMBOL(icmp_send);
    > > > EXPORT_SYMBOL(icmp_statistics);
    > > > -EXPORT_SYMBOL(icmpmsg_statistics);
    > > > EXPORT_SYMBOL(xrlim_allow);
    > >
    > > "icmpmsg_statistics" belongs with (and replaces some of the
    > > old...)
    > > "icmp_statistics". I'm not sure that any modules use it, but I think
    you
    > > should remove both or neither.
    >
    > icmp_statistics is used by the dccp_ipv4 and sctp modules.

    The only items left in icmp_statistics are "InMsgs, InErrs,
    OutMsgs, OutErrs",
    so if dccp and sctp are sending or receiving any in or out ICMP messages,
    they
    should be using the new macros (which reference icmpmsg_statistics, not
    icmp_statistics) to count them.
    I took a quick look at SCTP. I don't know if that's going through
    icmp_rcv()
    or not; if so, I think it's double-counting; if not, then it isn't
    counting the
    individual types (as it should), and it should have ICMPMSG macros doing
    that.

    So, again, icmpmsg_statistics either should stay exported, or
    neither
    icmpmsg_statistics nor icmp_statistics should be exported (depending on
    how
    SCTP and DCCP code is resolved). It's incorrect in the current code to
    incrememnt ICMP_MIB_INMSGS without incrementing one of the types too.

    +-DLS


    -
    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: 2007-10-24 22:15    [W:0.033 / U:30.212 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site