On Tue, Sep 25, 2012 at 05:37:15AM -0400, Christoph Hellwig wrote: > On Thu, Sep 20, 2012 at 10:32:39AM -0300, Carlos Maiolino wrote: > > With the changes made on xfs_set_inode64(), to make it behave as > > xfs_set_inode32() (now leaving to the caller the responsibility to update > > mp->m_maxagi), we use the return value of xfs_set_inode64() to update > > mp->m_maxagi during remount. > > > > Signed-off-by: Carlos Maiolino <cmaiolino@xxxxxxxxxx> > > Shouldn;t this be part of the previous patch? > > Otherwise looks good. > dchinner suggested to keep small changes into different patches, once the xfs_set_inode64() was added previously, adding a separate patche to add xfs_set_inode32() and another one to change the behaviour of both functions looks reasonable to me > _______________________________________________ > xfs mailing list > xfs@xxxxxxxxxxx > http://oss.sgi.com/mailman/listinfo/xfs -- --Carlos _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
