Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757242AbXJXUxY (ORCPT ); Wed, 24 Oct 2007 16:53:24 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1753745AbXJXUxD (ORCPT ); Wed, 24 Oct 2007 16:53:03 -0400 Received: from atlrel7.hp.com ([156.153.255.213]:56657 "EHLO atlrel7.hp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752922AbXJXUxB (ORCPT ); Wed, 24 Oct 2007 16:53:01 -0400 Message-ID: <471FB08D.9070506@hp.com> Date: Wed, 24 Oct 2007 16:52:29 -0400 From: Vlad Yasevich User-Agent: Thunderbird 2.0.0.4 (X11/20070604) MIME-Version: 1.0 To: David Stevens Cc: Adrian Bunk , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, netdev-owner@vger.kernel.org Subject: Re: [2.6 patch] unexport icmpmsg_statistics References: In-Reply-To: X-Enigmail-Version: 0.95.3 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1640 Lines: 42 David Stevens wrote: > I took a look at the DCCP references, and I think they're just > incrementing the wrong MIB variable -- e.g., it's incrementing > ICMP_MIB_INERRORS when the skb length is less than the > header indicates. That's not an ICMP_MIB_INERRORS error, > that's an IPSTATS_MIB_INHDRERRORS error. ICMP_MIB_INERRORS > is when you receive an ICMP error packet; an IP header error > is something else entirely. Looking at icmp_rcv(), ICMP_MIB_INERRORS is incremented if: a) checksum fails b) no enough room in skb for icmp header c) type out of bound and other error conditions while processing ICMP packet. Are all of these wrong as well? There are other places that increment this statistic for errors during processing. > > That's followed by a failed lookup incrementing ICMP_MIB_INERRORS > which should be an unknown port error in the transport MIB (assuming > it has one-- it's not an ICMP error; could be an IP error, if the address > isn't local, rather than unknown port). > > In SCTP, it appears to have similar problems. SCTP errors are not > ICMP errors, though it perhaps should be calling icmp_send() to > send one to the offending host for some of the cases. I'll skip the insufficient buffer space statistic yet, since, per above, it's not clear which one should be used. Others, I agree are move ULP errors, but the mibs don't account for those yet. -vlad - 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/