On 09/20/2012 09:32 AM, Carlos Maiolino wrote: > This patch set adds inode64 as the default allocation mode, but also includes 2 > patches to remove duplicated code and another one to make inode32 able to be > remounted. > > NOTE: This patch has as dependency "Make inode64 a remountable option" patch. > > V4 properly set pagf_metadata in patch 5 > Hi Carlos, With that fix, this patchset looks good to me, thanks. Reviewed-by: Brian Foster <bfoster@xxxxxxxxxx> Brian > V3 has a better factoring of each patch goal, including other dchinner's > suggestions in V2. > Also it contains a fix in how m_agirotor reset is made (patch 01) that I've > found while doing tests with these patches. > > Carlos Maiolino (6): > xfs: Fix m_agirotor reset during AG selection > xfs: make inode64 as the default allocation mode > xfs: reduce code duplication handling inode32/64 options > xfs: Fix mp->m_maxagi update during inode64 remount > xfs: add inode64->inode32 transition into xfs_set_inode32() > xfs: Make inode32 a remountable option > > fs/xfs/xfs_ialloc.c | 2 +- > fs/xfs/xfs_mount.c | 43 +++---------------- > fs/xfs/xfs_super.c | 117 ++++++++++++++++++++++++++++++++++++++++------------ > fs/xfs/xfs_super.h | 2 + > 4 files changed, 99 insertions(+), 65 deletions(-) > _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs