From owner-svn-src-all@FreeBSD.ORG Wed Jan 7 21:04:31 2015 Return-Path: Delivered-To: svn-src-all@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by hub.freebsd.org (Postfix) with ESMTPS id D3192700; Wed, 7 Jan 2015 21:04:31 +0000 (UTC) Received: from cyrus.watson.org (cyrus.watson.org [198.74.231.69]) by mx1.freebsd.org (Postfix) with ESMTP id A9DB11F26; Wed, 7 Jan 2015 21:04:31 +0000 (UTC) Received: from [10.0.1.17] (host86-177-234-167.range86-177.btcentralplus.com [86.177.234.167]) by cyrus.watson.org (Postfix) with ESMTPSA id 4998946B39; Wed, 7 Jan 2015 16:04:30 -0500 (EST) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 8.1 \(1993\)) Subject: Re: svn commit: r276750 - in head: share/man/man9 sys/contrib/ipfilter/netinet sys/dev/an sys/dev/bge sys/dev/ce sys/dev/cm sys/dev/cp sys/dev/cs sys/dev/ctau sys/dev/ed sys/dev/ex sys/dev/fe sys/dev/h... From: "Robert N. M. Watson" In-Reply-To: <20150107204853.GH15484@FreeBSD.org> Date: Wed, 7 Jan 2015 21:04:29 +0000 Content-Transfer-Encoding: quoted-printable Message-Id: References: <201501061259.t06CxcTc096488@svn.freebsd.org> <20150107174430.GQ1949@funkthat.com> <20150107204853.GH15484@FreeBSD.org> To: Gleb Smirnoff X-Mailer: Apple Mail (2.1993) Cc: svn-src-head@freebsd.org, John-Mark Gurney , src-committers@freebsd.org, svn-src-all@freebsd.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.18-1 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 07 Jan 2015 21:04:31 -0000 On 7 Jan 2015, at 20:48, Gleb Smirnoff wrote: > R> > Shouldn't this come w/ a FreeBSD version bump for drivers to use? > R>=20 > R> Yes, probably. Old drivers will continue to work fine in not = checking the=20 > R> return value (for now), but drivers seeing backporting will = probably want a=20 > R> __FreeBSD_version ifdef. I'll do a commit to bump the version = number today. > R>=20 > R> (In my local tree, M_EXT is renamed _M_EXT unless MBUF_PRIVATE is = defined,=20 > R> which really is quite a significant KPI change -- I'm not yet sure = if I'm=20 > R> going to push that into FreeBSD 11 or not.) >=20 > IMO, you should do the push :) >=20 > The faster we refactor the mbuf KPI, the better. Better do the surgery = in > one fast cut rather then do it in an endless serie of small but = painful > cuts. >=20 > Note that I'm still hoping of pushing projects/ifnet to 11, which is > a much bigger KPI change from drivers. The problem is really in structuring the changes so that they can be = reviewed: they're very hard to test since they touch dozens (hundreds?) = of drivers making them almost impossible to test effectively. Instead, = I've been breaking them down into a series of less intrusive and easier = to review chunks. The MLEN/MHLEN/MCLBYTES change should go into = phabricator this evening, assuming a bunch of sanity tests run to = completion successfully over the next few hours. These changes are = really all just to facilitate underlying changes to the mbuf allocator = by hiding more of the implementation details from consumers, so I want = to get them done in as expedient a manner as I can -- after all, they = are a bit boring :-). Robert=