[PATCH 5/8] xfsprogs: Drop root inode refrerence in libxfs_umount()

Christoph Hellwig hch at infradead.org
Sun Nov 13 06:07:04 CST 2011


On Thu, Nov 10, 2011 at 02:35:15PM -0600, Alex Elder wrote:
> From: Kevan Rehm <kfr at sgi.com>
> 
> Routine libxfs_umount() did not call libxfs_iput for the m_rootip
> inode, so updates made to that inode could be lost.  This adds the
> missing call, and re-initializes the m_rootip pointer to be null.
> 
> Since the root inode reference is now dropped by libxfs_umount(), it
> should *not* be dropped in mkfs parseproto().

Looks fine in general, but as mentioned before live would be a lot
easier if we simply made the root inode a local variable in
libxfs_mount.

In fact I wonder if we even want to do that, there seems very little
reason to even do the root inode iget there - the only caller requesting
it will fall back if it fails.




More information about the xfs mailing list