From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay1.corp.sgi.com [137.38.102.111]) by oss.sgi.com (Postfix) with ESMTP id 928417F58 for ; Sun, 8 Feb 2015 10:04:11 -0600 (CST) Received: from cuda.sgi.com (cuda2.sgi.com [192.48.176.25]) by relay1.corp.sgi.com (Postfix) with ESMTP id 7009B8F8052 for ; Sun, 8 Feb 2015 08:04:11 -0800 (PST) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by cuda.sgi.com with ESMTP id GFdNvaDtsQYkHPDq (version=TLSv1 cipher=AES256-SHA bits=256 verify=NO) for ; Sun, 08 Feb 2015 08:04:07 -0800 (PST) Date: Sun, 8 Feb 2015 11:04:03 -0500 From: Brian Foster Subject: Re: [PATCH v3 01/18] xfs: add sparse inode chunk alignment superblock field Message-ID: <20150208160402.GA2927@bfoster.bfoster> References: <1423252385-3063-1-git-send-email-bfoster@redhat.com> <1423252385-3063-2-git-send-email-bfoster@redhat.com> <20150206224047.GO12722@dastard> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20150206224047.GO12722@dastard> 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 Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Dave Chinner Cc: xfs@oss.sgi.com On Sat, Feb 07, 2015 at 09:40:47AM +1100, Dave Chinner wrote: > On Fri, Feb 06, 2015 at 02:52:48PM -0500, Brian Foster wrote: > > Add sb_spinoalignmt to the superblock to specify sparse inode chunk > > alignment. This also currently represents the minimum allowable sparse > > chunk allocation size. > > > > Signed-off-by: Brian Foster > > --- > > fs/xfs/libxfs/xfs_format.h | 6 +++--- > > fs/xfs/libxfs/xfs_sb.c | 4 ++-- > > 2 files changed, 5 insertions(+), 5 deletions(-) > > > > diff --git a/fs/xfs/libxfs/xfs_format.h b/fs/xfs/libxfs/xfs_format.h > > index 8eb7189..051c24d 100644 > > --- a/fs/xfs/libxfs/xfs_format.h > > +++ b/fs/xfs/libxfs/xfs_format.h > > @@ -170,7 +170,7 @@ typedef struct xfs_sb { > > __uint32_t sb_features_log_incompat; > > > > __uint32_t sb_crc; /* superblock crc */ > > - __uint32_t sb_pad; > > + xfs_extlen_t sb_spinoalignmt;/* sparse inode chunk alignment */ > > That's a mounthful. sb_spino_align is a bit easier to read, IMO. > Ok. > > @@ -282,7 +282,7 @@ typedef enum { > > XFS_SBS_LOGSECTLOG, XFS_SBS_LOGSECTSIZE, XFS_SBS_LOGSUNIT, > > XFS_SBS_FEATURES2, XFS_SBS_BAD_FEATURES2, XFS_SBS_FEATURES_COMPAT, > > XFS_SBS_FEATURES_RO_COMPAT, XFS_SBS_FEATURES_INCOMPAT, > > - XFS_SBS_FEATURES_LOG_INCOMPAT, XFS_SBS_CRC, XFS_SBS_PAD, > > + XFS_SBS_FEATURES_LOG_INCOMPAT, XFS_SBS_CRC, XFS_SBS_SPINOALIGNMT, > > XFS_SBS_PQUOTINO, XFS_SBS_LSN, > > XFS_SBS_FIELDCOUNT > > } xfs_sb_field_t; > > These are gone in the for-next tree. > The per-field logging stuff is gone... this apparently still exists. It looks like it goes away as part of the icsb rework so this will drop naturally whenever that goes in. Brian > Cheers, > > Dave. > -- > Dave Chinner > david@fromorbit.com _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs