From mboxrd@z Thu Jan 1 00:00:00 1970 From: William Allen Simpson Subject: Re: [PATCH] TCPCT-1: adding a sysctl Date: Fri, 02 Oct 2009 18:00:07 -0400 Message-ID: <4AC677E7.8030606@gmail.com> References: <4AC57AC5.3080703@gmail.com> <20091001225705.788d38ba@nehalam> <4AC61505.8030701@gmail.com> <4AC63DD1.3030705@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit To: netdev@vger.kernel.org Return-path: Received: from mail-bw0-f210.google.com ([209.85.218.210]:47805 "EHLO mail-bw0-f210.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753712AbZJBWAK (ORCPT ); Fri, 2 Oct 2009 18:00:10 -0400 Received: by bwz6 with SMTP id 6so1381540bwz.37 for ; Fri, 02 Oct 2009 15:00:13 -0700 (PDT) In-Reply-To: <4AC63DD1.3030705@gmail.com> Sender: netdev-owner@vger.kernel.org List-ID: William Allen Simpson wrote: > William Allen Simpson wrote: >> This is a straightforward re-implementation of an earlier patch, that no >> longer applies cleanly, that was reviewed: >> >> http://thread.gmane.org/gmane.linux.network/102586 >> > In that thread, David Miller wrote: > > "This looks mostly fine to me. I would even advocate not using a config > option for this." > > It would make the code look cleaner, and with the sysctl instead, it > would probably be fine. But SYN cookies has both. > > Before I go much further, I'd like guidance. > Based on Andi's expressed desire for more complete code before reviewing, and the general utility of using a sysctl instead, I'll remove the config #ifdefs. Sorry, I was overly cautious....