From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bombadil.infradead.org ([198.137.202.9]:33481 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S966404AbcKOKYG (ORCPT ); Tue, 15 Nov 2016 05:24:06 -0500 Date: Tue, 15 Nov 2016 02:24:05 -0800 From: Christoph Hellwig Subject: Re: [PATCH 05/10] libxfs: clean up _dir2_data_freescan Message-ID: <20161115102405.GU1607@infradead.org> References: <147830459629.27129.4541676333173489075.stgit@birch.djwong.org> <147830462738.27129.11184668870098090677.stgit@birch.djwong.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <147830462738.27129.11184668870098090677.stgit@birch.djwong.org> Sender: linux-xfs-owner@vger.kernel.org List-ID: List-Id: xfs To: "Darrick J. Wong" Cc: david@fromorbit.com, linux-xfs@vger.kernel.org On Fri, Nov 04, 2016 at 05:10:27PM -0700, Darrick J. Wong wrote: > Refactor the implementations of xfs_dir2_data_freescan into a > routine that takes the raw directory block parameters and > a second function that figures out the raw parameters from the > directory inode. This enables us to use the exact same code > for both userspace and the kernel, since repair knows exactly > which directory block geometry parameters it needs. Looks fine, Reviewed-by: Christoph Hellwig