Commit 13205fb9 authored by Jens Axboe's avatar Jens Axboe Committed by Al Viro
Browse files

ntfs: remove old debug check for dirty data in ntfs_put_super() 



This should not trigger anymore, so kill it.

Acked-by: default avatarAnton Altaparmakov <aia21@cam.ac.uk>
Signed-off-by: default avatarJens Axboe <jens.axboe@oracle.com>
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
parent 28ad0c11

fs/ntfs/super.c

+3 −30
Original line number Diff line number Diff line
@@ -2376,39 +2376,12 @@ static void ntfs_put_super(struct super_block *sb) 
		vol->mftmirr_ino = NULL;

	}

	/*

	 * If any dirty inodes are left, throw away all mft data page cache

	 * pages to allow a clean umount.  This should never happen any more

	 * due to mft.c::ntfs_mft_writepage() cleaning all the dirty pages as

	 * the underlying mft records are written out and cleaned.  If it does,

	 * happen anyway, we want to know...

	 * We should have no dirty inodes left, due to

	 * mft.c::ntfs_mft_writepage() cleaning all the dirty pages as

	 * the underlying mft records are written out and cleaned.

	 */

	ntfs_commit_inode(vol->mft_ino);

	write_inode_now(vol->mft_ino, 1);

	if (sb_has_dirty_inodes(sb)) {

		const char *s1, *s2;



		mutex_lock(&vol->mft_ino->i_mutex);

		truncate_inode_pages(vol->mft_ino->i_mapping, 0);

		mutex_unlock(&vol->mft_ino->i_mutex);

		write_inode_now(vol->mft_ino, 1);

		if (sb_has_dirty_inodes(sb)) {

			static const char *_s1 = "inodes";

			static const char *_s2 = "";

			s1 = _s1;

			s2 = _s2;

		} else {

			static const char *_s1 = "mft pages";

			static const char *_s2 = "They have been thrown "

					"away.  ";

			s1 = _s1;

			s2 = _s2;

		}

		ntfs_error(sb, "Dirty %s found at umount time.  %sYou should "

				"run chkdsk.  Please email "

				"linux-ntfs-dev@lists.sourceforge.net and say "

				"that you saw this message.  Thank you.", s1,

				s2);

	}

#endif /* NTFS_RW */



	iput(vol->mft_ino);