From owner-svn-src-head@FreeBSD.ORG  Sun Mar 17 10:45:46 2013
Return-Path: <owner-svn-src-head@FreeBSD.ORG>
Delivered-To: svn-src-head@freebsd.org
Received: from mx1.freebsd.org (mx1.freebsd.org
 [IPv6:2001:1900:2254:206a::19:1])
 by hub.freebsd.org (Postfix) with ESMTP id 90FAF740
 for <svn-src-head@freebsd.org>; Sun, 17 Mar 2013 10:45:46 +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 01C27B0C
 for <svn-src-head@freebsd.org>; Sun, 17 Mar 2013 10:45:45 +0000 (UTC)
Received: (qmail 84850 invoked from network); 17 Mar 2013 11:57:25 -0000
Received: from c00l3r.networx.ch (HELO [127.0.0.1]) ([62.48.2.2])
 (envelope-sender <andre@freebsd.org>)
 by c00l3r.networx.ch (qmail-ldap-1.03) with SMTP
 for <glebius@FreeBSD.org>; 17 Mar 2013 11:57:25 -0000
Message-ID: <51459ED3.4040304@freebsd.org>
Date: Sun, 17 Mar 2013 11:45:39 +0100
From: Andre Oppermann <andre@freebsd.org>
User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64;
 rv:17.0) Gecko/20130307 Thunderbird/17.0.4
MIME-Version: 1.0
To: Gleb Smirnoff <glebius@FreeBSD.org>
Subject: Re: svn commit: r248417 - head/sys/sys
References: <201303170739.r2H7djP1098888@svn.freebsd.org>
 <51458691.4090107@freebsd.org> <20130317093339.GT48089@FreeBSD.org>
In-Reply-To: <20130317093339.GT48089@FreeBSD.org>
Content-Type: text/plain; charset=KOI8-R; 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
 <svn-src-head.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/options/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/svn-src-head>
List-Post: <mailto:svn-src-head@freebsd.org>
List-Help: <mailto:svn-src-head-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/svn-src-head>,
 <mailto:svn-src-head-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 17 Mar 2013 10:45:46 -0000

On 17.03.2013 10:33, Gleb Smirnoff wrote:
> On Sun, Mar 17, 2013 at 10:02:09AM +0100, Andre Oppermann wrote:
> A> On 17.03.2013 08:39, Gleb Smirnoff wrote:
> A> > Author: glebius
> A> > Date: Sun Mar 17 07:39:45 2013
> A> > New Revision: 248417
> A> > URL: http://svnweb.freebsd.org/changeset/base/248417
> A> >
> A> > Log:
> A> >    Add MEXT_ALIGN() macro, similar to M_ALIGN() and MH_ALIGN(), but for
> A> >    mbufs with external buffer.
> A>
> A> While you are cleaning up the mbuf usage wouldn't it make sense to remove
> A> these macros, instead of adding new ones, and use m_align() which handles
> A> all these cases internally?
>
> I'm thinking about this. Maybe it is worth to request tail alignment as
> a flag to the allocating function itself?

IMHO that would overload the allocation function(s).  The explicit step of
doing m_align() for those who need it is fine and alerts the reader of what
is going on.  I'm all for simplification and unification, on the other hand
it shouldn't be taken too far creating new complexity on the other side.

-- 
Andre