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
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
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