From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay2.corp.sgi.com [137.38.102.29]) by oss.sgi.com (8.14.3/8.14.3/SuSE Linux 0.8) with ESMTP id qB3IFlw0217787 for ; Mon, 3 Dec 2012 12:15:47 -0600 Date: Mon, 3 Dec 2012 12:18:06 -0600 From: Ben Myers Subject: Re: [PATCH 4/4] xfs: fix sparse reported log CRC endian issue Message-ID: <20121203181806.GM27055@sgi.com> References: <1354068063-1692-1-git-send-email-david@fromorbit.com> <1354068063-1692-5-git-send-email-david@fromorbit.com> <20121128133059.GC2765@infradead.org> <20121128213110.GV6434@dastard> <20121129203250.GI27055@sgi.com> <20121130160330.GA5667@infradead.org> <20121130160432.GR24185@sgi.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20121130160432.GR24185@sgi.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: xfs-bounces@oss.sgi.com Errors-To: xfs-bounces@oss.sgi.com To: Christoph Hellwig Cc: xfs@oss.sgi.com On Fri, Nov 30, 2012 at 10:04:32AM -0600, Ben Myers wrote: > Hey Christoph, > > On Fri, Nov 30, 2012 at 11:03:30AM -0500, Christoph Hellwig wrote: > > On Thu, Nov 29, 2012 at 02:32:50PM -0600, Ben Myers wrote: > > > This seems reasonable to me, and the patch looks fine. Christoph, do have any > > > further objection? > > > > No, although adding the explanation as a comment would be useful. > > > > Reviewed-by: Christoph Hellwig > > Cool. I'll add it in then. committed to git://oss.sgi.com/xfs/xfs.git, master and for-next branches. -Ben _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs