[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <4AE81A76.5080600@codefidence.com>
Date: Wed, 28 Oct 2009 12:18:30 +0200
From: Gilad Ben-Yossef <gilad@...efidence.com>
To: William Allen Simpson <william.allen.simpson@...il.com>
CC: netdev@...r.kernel.org, ori@...sleep.com
Subject: Re: [PATCH v3 4/7] Add the no SACK route option feature
William Allen Simpson wrote:
> Gilad Ben-Yossef wrote:
>> Implement querying and acting upon the no sack bit in the features
>> field.
>>
>> Signed-off-by: Gilad Ben-Yossef <gilad@...efidence.com>
>> Sigend-off-by: Ori Finkelman <ori@...sleep.com>
>> Sigend-off-by: Yony Amit <yony@...sleep.com>
>>
> Please explain how this code turns SACK on when it is off globally?
>
> As both Eric and I asked?
It doesn't. Please see my discussion with Eric for the why.
In short, doing so introduce a very subtle change to what the existing
interface do today, which will break
backwards compatibility by changing the meaning of writing zero to the
relevant sysctl. I don't want to be hunt down by angry sys admins :-)
Thanks,
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
"The biggest risk you can take it is to take no risk."
-- Mark Zuckerberg and probably others
--
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