Subject: Re: [PATCH v2 5/8] Allow disabling TCP timestamp options per route

From: Gilad Ben-Yossef <[email-masked]>
Date: Sun, 25 Oct 2009 10:43:13 +0200

Hi,

William Allen Simpson wrote:

>
>> diff --git a/include/linux/rtnetlink.h b/include/linux/rtnetlink.h
>> index 9c802a6..2ab8c75 100644
>> --- a/include/linux/rtnetlink.h
>> +++ b/include/linux/rtnetlink.h
>> @@ -378,7 +378,7 @@ enum
>>  
>>  #define RTAX_FEATURE_ECN    0x00000001
>>  #define RTAX_FEATURE_NO_SACK    0x00000002
>> -#define RTAX_FEATURE_TIMESTAMP    0x00000004
>> +#define RTAX_FEATURE_NO_TSTAMP    0x00000004
>>  #define RTAX_FEATURE_ALLFRAG    0x00000008
>>  
> I just realized that unlike NO_WSCALE, this change assumes removing the
> sysctl and defaulting on.
No, it doesn't. This patch does not change the sysctl behavior in any 
way. The RTAX_FEATURE_TIMESTAMP define is a currently a dead code not 
used anywhere AFAIK.

Gilad

-- 
Gilad Ben-Yossef
Chief Coffee Drinker & CTO
Codefidence Ltd.

Web:   http://codefidence.com
Cell:  +972-52-8260388
Skype: gilad_codefidence
Tel:   +972-8-9316883 ext. 201
Fax:   +972-8-9316884
Email: [email-masked]

Check out our Open Source technology and training blog - http://tuxology.net

	"Sorry cannot parse this, its too long to be true  :)"
	  -- Eric Dumazet on netdev mailing list