lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 16 Jan 2008 16:10:45 -0800 From: David Stevens <dlstevens@...ibm.com> To: Herbert Xu <herbert@...dor.apana.org.au> Cc: davem@...emloft.net, herbert@...dor.apana.org.au, netdev@...r.kernel.org, netdev-owner@...r.kernel.org, wangchen@...fujitsu.com Subject: Re: [PATCH] ICMP: ICMP_MIB_OUTMSGS increment duplicated netdev-owner@...r.kernel.org wrote on 01/16/2008 03:17:29 PM: > Fair enough. How about moving this code back into icmp.c and just > add a new count call in raw.c? The push pending function is used on > the UDP fast path so the leaner it is the better. I started out with it there, but it certainly wasn't cleaner. You have to peek on the queue, which I didn't want to do for all the SMP issues that brings in, and then replicate all the code push_pending_frames does to get you a protocol header pointer in one buffer, and there are multiple ways in the raw path to get you there (so the counter code itself would appear in multiple places). I don't think the 2 instructions measurably impact anything in the fast path and that is the earliest common point for the multiple paths to generate ICMP output where the header and type are available without replicating code. So, simplicity is exactly why I put it there, rather than the "obvious" places at the higher layer. :-) It's probably the better, single place to put the other protocol "out messages" counters, too, since those probably also have multiple paths that end up here, but ICMPMsgStats was all that patch was after. +-DLS -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists