[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <4AE40FA1.5040309@codefidence.com>
Date: Sun, 25 Oct 2009 10:43:13 +0200
From: Gilad Ben-Yossef <gilad@...efidence.com>
To: William Allen Simpson <william.allen.simpson@...il.com>
CC: netdev@...r.kernel.org
Subject: Re: [PATCH v2 5/8] Allow disabling TCP timestamp options per route
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: gilad@...efidence.com
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
--
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