From owner-svn-src-head@FreeBSD.ORG Mon Aug 26 14:00:22 2013 Return-Path: Delivered-To: svn-src-head@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1 with cipher ADH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTP id D4F6ACC4 for ; Mon, 26 Aug 2013 14:00:22 +0000 (UTC) (envelope-from andre@freebsd.org) Received: from c00l3r.networx.ch (c00l3r.networx.ch [62.48.2.2]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.freebsd.org (Postfix) with ESMTPS id 2D94C2636 for ; Mon, 26 Aug 2013 14:00:21 +0000 (UTC) Received: (qmail 6651 invoked from network); 26 Aug 2013 14:42:32 -0000 Received: from c00l3r.networx.ch (HELO [127.0.0.1]) ([62.48.2.2]) (envelope-sender ) by c00l3r.networx.ch (qmail-ldap-1.03) with SMTP for ; 26 Aug 2013 14:42:32 -0000 Message-ID: <521B5F71.4030507@freebsd.org> Date: Mon, 26 Aug 2013 16:00:17 +0200 From: Andre Oppermann User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:17.0) Gecko/20130801 Thunderbird/17.0.8 MIME-Version: 1.0 To: Gleb Smirnoff Subject: Re: svn commit: r254527 - in head/sys: net80211 netinet sys References: <20130826094329.GP4574@glebius.int.ru> In-Reply-To: <20130826094329.GP4574@glebius.int.ru> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Cc: svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.14 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: Mon, 26 Aug 2013 14:00:22 -0000 On 26.08.2013 11:43, Gleb Smirnoff wrote: > A> Author: andre > A> Date: Mon Aug 19 14:25:11 2013 > A> New Revision: 254527 > A> URL: http://svnweb.freebsd.org/changeset/base/254527 > A> > A> Log: > A> Reorder the mbuf defines to make more sense and group related flags > A> together. > A> > A> Add M_FLAG_PRINTF for use with printf(9) %b indentifier. > A> > A> Use the generic mbuf flags print names in the net80211 code and adjust > A> the protocol specific bits for their new positions. > A> > A> Change SCTP M_PROTO mapping from 5 to 1 to fit within the 16bit field > A> they use internally to store some additional information. > A> > A> Discussed with: trociny, glebius > > The first part: "reorder the mbuf flag defines" wasn't actually discussed. It was part of the patch to the email I sent you, and a couple of others, to which you replied with no objection to the M_FLAGS cleanup. It wasn't explicitly spelled out but very visible in the patch. > Tossing values of M_BCAST, M_MCAST, M_VLANTAG for no value except code > tidyness isn't a safe idea. These are the flags that device drivers use. > After this change binary drivers are still loadable but would be buggy. As I already explained elsewhere, on -current only KLD's compiled at the same time/revision as the kernel are supported. It has been this way as long as I can remember. There never was any guarantee, not even implicit, that a KLD from r254000 will continue to work with a kernel from r254527. In fact, if it were, it would be almost impossible to develop anything on -current. On -stable we do have this ABI+API guarantee, so a KLD from 9.0 is supposed to work on 9.2 as well and we generally keep that promise. > We allow ourselves ABI changes in head, but this doesn't mean that we > should do them for no important reason. The "important" reason are the later updates to the mbuf headers and that it simply doesn't make sense to keep the defines spaghetti while the whole thing is being overhauled. It seems from a technical point of view both arguments, maintain bits vs. reordering, are rather weak and prone to bike-shedding. ;) It certainly doesn't make sense to change again and revert it just for the sake of it. -- Andre