X-Spam-Checker-Version: SpamAssassin 3.4.0-r929098 (2010-03-30) on oss.sgi.com
X-Spam-Level: 
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=ham
	version=3.4.0-r929098
Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111])
	by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id o8EMKbkx150827
	for <xfs@oss.sgi.com>; Tue, 14 Sep 2010 17:20:37 -0500
Received: from cf--amer001e--3.americas.sgi.com (cf--amer001e--3.americas.sgi.com [137.38.100.5])
	by relay1.corp.sgi.com (Postfix) with ESMTP id A89008F8074;
	Tue, 14 Sep 2010 15:21:23 -0700 (PDT)
Received: from [128.162.232.162] ([128.162.232.162]) by cf--amer001e--3.americas.sgi.com with Microsoft SMTPSVC(6.0.3790.3959);
	 Tue, 14 Sep 2010 17:21:17 -0500
Subject: Re: [PATCH 10/18] xfs: store xfs_mount in the buftarg instead of
 in the xfs_buf
From: Alex Elder <aelder@sgi.com>
Reply-To: aelder@sgi.com
To: Dave Chinner <david@fromorbit.com>
Cc: xfs@oss.sgi.com
In-Reply-To: <1284461777-1496-11-git-send-email-david@fromorbit.com>
References: <1284461777-1496-1-git-send-email-david@fromorbit.com>
	 <1284461777-1496-11-git-send-email-david@fromorbit.com>
Content-Type: text/plain; charset="UTF-8"
Date: Tue, 14 Sep 2010 17:21:17 -0500
Message-ID: <1284502877.9701.93.camel@doink>
Mime-Version: 1.0
X-Mailer: Evolution 2.28.3 
Content-Transfer-Encoding: 7bit
X-OriginalArrivalTime: 14 Sep 2010 22:21:17.0724 (UTC) FILETIME=[26A425C0:01CB545B]
X-Virus-Scanned: ClamAV version 0.94.2, clamav-milter version 0.94.2 on oss.sgi.com
X-Virus-Status: Clean

On Tue, 2010-09-14 at 20:56 +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner@redhat.com>
> 
> Each buffer contains both a buftarg pointer and a mount pointer. If
> we add a mount pointer into the buftarg, we can avoid needing the
> b_mount field in every buffer and grab it from the buftarg when
> needed instead. This shrinks the xfs_buf by 8 bytes.

Looks good.

Reviewed-by: Alex Elder <aelder@sgi.com>


> Signed-off-by: Dave Chinner <dchinner@redhat.com>
> ---