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 linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 23 Nov 2011 19:04:28 -0500 (EST) From: David Miller <davem@...emloft.net> To: mypopydev@...il.com Cc: eric.dumazet@...il.com, netdev@...r.kernel.org Subject: Re: [PATCH v2] ipv4 : igmp : optimize timer modify logic in igmp_mod_timer() From: Jun Zhao <mypopydev@...il.com> Date: Thu, 24 Nov 2011 06:54:45 +0800 > On Wed, 2011-11-23 at 17:28 -0500, David Miller wrote: >> From: Jun Zhao <mypopydev@...il.com> >> Date: Thu, 24 Nov 2011 00:38:42 +0800 >> >> > When timer is pending and expires less-than-or-equal-to new delay, >> > we need not used del_timer()/add_timer(). >> > >> > Signed-off-by: Jun Zhao <mypopydev@...il.com> >> >> You did not answer Eric's question, why are you optimizing this >> less-used code path? > > 1). Oh, in the RFC 3376 $5.2, Page 23: Then your commit message is terrible. Your commit message, one the one hand, talks about optimizing the code. Your explanation here talks about RFC conformance. Your inconsistencies, and how you ignore important questions posed to you like Eric's (until I point it out to you) makes your work incredibly irritating to review and process. Your patch submissions need to be more well formed and your commit messages need to explain exactly what your goals are with your change and how those goals are being met by the patch you are proposing. When we read "optimize timer modify logic" how the heck are we supposed to know what this change is actually doing? Why should we think that we actually need your change? How am we supposed to figure out that you are fixing an RFC conformance issue? I'm sorry, this patch submission is junk. Don't send us junk. -- 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