From owner-svn-src-head@FreeBSD.ORG Sat Sep 5 00:05:04 2009 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 768BB1065670; Sat, 5 Sep 2009 00:05:04 +0000 (UTC) (envelope-from bms@incunabulum.net) Received: from out1.smtp.messagingengine.com (out1.smtp.messagingengine.com [66.111.4.25]) by mx1.freebsd.org (Postfix) with ESMTP id 403448FC18; Sat, 5 Sep 2009 00:05:04 +0000 (UTC) Received: from compute2.internal (compute2.internal [10.202.2.42]) by gateway1.messagingengine.com (Postfix) with ESMTP id 60BA6618C8; Fri, 4 Sep 2009 20:05:03 -0400 (EDT) Received: from heartbeat1.messagingengine.com ([10.202.2.160]) by compute2.internal (MEProxy); Fri, 04 Sep 2009 20:05:03 -0400 DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=messagingengine.com; h=message-id:date:from:mime-version:to:cc:subject:references:in-reply-to:content-type:content-transfer-encoding; s=smtpout; bh=l/Df04kmcZRCKy3O0voCZLSXogc=; b=qmX/6M4cFgircA3SHDe0Ozrgvm8cyA5RaXzulDYIRGVndXqLWZGIP8AyQNN5vzeoSCMIxBf0OfhiFhlHh6YInnAFjV4jYh1+UXuBovRNbuJru3vMC6Dm0kSQ6PZ2CkQIR/aR6U7tD/VCMMzIPoiuD3ddgmXnzMGPWHL63neVtT0= X-Sasl-enc: KwF4aIWVkGS3Q9a2ipchXtKpV/07uyMS1ALxXrY3/gEe 1252109102 Received: from [192.168.123.18] (82-35-112-254.cable.ubr07.dals.blueyonder.co.uk [82.35.112.254]) by mail.messagingengine.com (Postfix) with ESMTPSA id 80BF32E39C; Fri, 4 Sep 2009 20:05:02 -0400 (EDT) Message-ID: <4AA1AB29.9030704@incunabulum.net> Date: Sat, 05 Sep 2009 01:04:57 +0100 From: Bruce Simpson User-Agent: Thunderbird 2.0.0.23 (Windows/20090812) MIME-Version: 1.0 To: Robert Watson References: <200909032110.n83LAvUF044731@svn.freebsd.org> <4AA09B41.3040103@incunabulum.net> In-Reply-To: Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, "George V. Neville-Neil" , svn-src-all@freebsd.org, src-committers@freebsd.org Subject: Re: svn commit: r196797 - in head: sys/net sys/netinet usr.bin/netstat X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sat, 05 Sep 2009 00:05:04 -0000 Robert Watson wrote: > For sysctls, explicit versioning doesn't help too much -- what I've > been pondering for another structure was including spares and having a > capabilities field in the structure where flags are set when fields > are known by the kernel. That way userspace can tell if the kernel > knows about the field it cares about and therefore whether they're > worth looking at. One thing that Thrift did was to explicitly all number fields in their implementation of RPC. This is useful, because if anything changes over time, implementations can be updated to keep track of that. Whilst I agree that it's unlikely 'struct arpstat' would change, if it did, it would be great to have a length field, at a minimum, in front. sysctl can determine size at runtime, however, kmem interfaces cannot. That's the only real reason why I'd nudge about it. :-) For igmpstat, at least, the SSM drop adds a LOT of state to the network stack which wasn't there before, so having an explicit version field is warranted for backwards compatibility (and this is something Robert chewed me out about many months ago). Much respect, big yo'self up yo...