Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 1f792dd1 authored by Linus Torvalds's avatar Linus Torvalds
Browse files
Pull ext3 fix and quota cleanup from Jan Kara:
 "A fix of ext3 error reporting from fsync and a quota cleanup"

* 'for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/jack/linux-fs:
  quota: Convert use of typedef ctl_table to struct ctl_table
  ext3: Fix fsync error handling after filesystem abort.
parents c75e2475 e628753b
Loading
Loading
Loading
Loading
+6 −2
Original line number Diff line number Diff line
@@ -48,9 +48,13 @@ int ext3_sync_file(struct file *file, loff_t start, loff_t end, int datasync)

	trace_ext3_sync_file_enter(file, datasync);

	if (inode->i_sb->s_flags & MS_RDONLY)
	if (inode->i_sb->s_flags & MS_RDONLY) {
		/* Make sure that we read updated state */
		smp_rmb();
		if (EXT3_SB(inode->i_sb)->s_mount_state & EXT3_ERROR_FS)
			return -EROFS;
		return 0;

	}
	ret = filemap_write_and_wait_range(inode->i_mapping, start, end);
	if (ret)
		goto out;
+12 −1
Original line number Diff line number Diff line
@@ -174,6 +174,11 @@ static void ext3_handle_error(struct super_block *sb)
	if (test_opt (sb, ERRORS_RO)) {
		ext3_msg(sb, KERN_CRIT,
			"error: remounting filesystem read-only");
		/*
		 * Make sure updated value of ->s_mount_state will be visible
		 * before ->s_flags update.
		 */
		smp_wmb();
		sb->s_flags |= MS_RDONLY;
	}
	ext3_commit_super(sb, es, 1);
@@ -291,8 +296,14 @@ void ext3_abort(struct super_block *sb, const char *function,
	ext3_msg(sb, KERN_CRIT,
		"error: remounting filesystem read-only");
	EXT3_SB(sb)->s_mount_state |= EXT3_ERROR_FS;
	sb->s_flags |= MS_RDONLY;
	set_opt(EXT3_SB(sb)->s_mount_opt, ABORT);
	/*
	 * Make sure updated value of ->s_mount_state will be visible
	 * before ->s_flags update.
	 */
	smp_wmb();
	sb->s_flags |= MS_RDONLY;

	if (EXT3_SB(sb)->s_journal)
		journal_abort(EXT3_SB(sb)->s_journal, -EIO);
}
+3 −3
Original line number Diff line number Diff line
@@ -2585,7 +2585,7 @@ static int do_proc_dqstats(struct ctl_table *table, int write,
	return proc_dointvec(table, write, buffer, lenp, ppos);
}

static ctl_table fs_dqstats_table[] = {
static struct ctl_table fs_dqstats_table[] = {
	{
		.procname	= "lookups",
		.data		= &dqstats.stat[DQST_LOOKUPS],
@@ -2654,7 +2654,7 @@ static ctl_table fs_dqstats_table[] = {
	{ },
};

static ctl_table fs_table[] = {
static struct ctl_table fs_table[] = {
	{
		.procname	= "quota",
		.mode		= 0555,
@@ -2663,7 +2663,7 @@ static ctl_table fs_table[] = {
	{ },
};

static ctl_table sys_table[] = {
static struct ctl_table sys_table[] = {
	{
		.procname	= "fs",
		.mode		= 0555,