From owner-cvs-all@FreeBSD.ORG Thu Jan 24 10:58:18 2008 Return-Path: Delivered-To: cvs-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C8D7016A475 for ; Thu, 24 Jan 2008 10:58:18 +0000 (UTC) (envelope-from andre@freebsd.org) Received: from c00l3r.networx.ch (c00l3r.networx.ch [62.48.2.2]) by mx1.freebsd.org (Postfix) with ESMTP id 1F74F13C4D5 for ; Thu, 24 Jan 2008 10:58:17 +0000 (UTC) (envelope-from andre@freebsd.org) Received: (qmail 98697 invoked from network); 24 Jan 2008 10:19:32 -0000 Received: from localhost (HELO [127.0.0.1]) ([127.0.0.1]) (envelope-sender ) by c00l3r.networx.ch (qmail-ldap-1.03) with SMTP for ; 24 Jan 2008 10:19:32 -0000 Message-ID: <47986F4D.6070208@freebsd.org> Date: Thu, 24 Jan 2008 11:58:21 +0100 From: Andre Oppermann User-Agent: Thunderbird 1.5.0.14 (Windows/20071210) MIME-Version: 1.0 To: Kip Macy References: <200711200656.lAK6u4bc021279@repoman.freebsd.org> <4797B77E.2090605@freebsd.org> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: Mike Silbersack , kmacy@freebsd.org, cvs-src@freebsd.org, cvs-all@freebsd.org, src-committers@freebsd.org, freebsd-net@freebsd.org Subject: Re: cvs commit: src/sys/netinet tcp_syncache.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 24 Jan 2008 10:58:18 -0000 Kip Macy wrote: > Did you talk to the original submitter? Note that FreeBSD's TCP stack > is for use in servers and is not intending as a validating TCP stack. > If you would like it to serve as such you would better served by > tracking down the ANVL tests that FreeBSD fails. Also note that there > is no MUST in the following sentence: > > > "For simplicity and symmetry, we specify that > timestamps always be sent and echoed in both directions." > > So it is clearly open to interpretation. No, it is not. RFC1323 was written in 1992 before RFCs contained the boiler plate definition of MUST, SHOULD, MAY and so on. I, at least as a non-native English speaker, find the sentence perfectly clear and without any doubt. The IETF TCPM working group comes to the same conclusion. And I suppose many native English speakers too. Despite that arguing over whether "always" lacks a "MUST" to make it really always always and never not you cited the wrong part of RFC1323 as reason to completely remove the check. That's what I'm complaining about. Everyone in FreeBSD, including you and me, should at least provide the correct citation and rationale for any code change irrespective of the eventual merit of the change itself which is a separate issue. -- Andre