[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170527082338.GC32125@infradead.org>
Date: Sat, 27 May 2017 01:23:38 -0700
From: Christoph Hellwig <hch@...radead.org>
To: Hyunchul Lee <hyc.lee@...il.com>
Cc: Richard Weinberger <richard@....at>,
Artem Bityutskiy <dedekind1@...il.com>,
adrian.hunter@...el.com, linux-kernel@...r.kernel.org,
linux-fsdevel@...r.kernel.org, linux-mtd@...ts.infradead.org,
kernel-team@....com
Subject: Re: [PATCH] ubifs: Add freeze support
> +static int ubifs_freeze_super(struct super_block *sb)
> +{
> + struct ubifs_info *c = sb->s_fs_info;
> + int err;
> +
> + dbg_gen("starting");
> + /* freeze_super always succeeds if file system is in read-only.
> + * however if there are errors, UBIFS is switched to read-only mode.
> + * so @ro_error should be checked.
> + */
> + err = freeze_super(sb);
> + if (!err && c->ro_error) {
> + thaw_super(sb);
> + return -EIO;
> + }
> + return err;
This is just broken. First ubifs should still properly propagate
the errors, and second freezing/unfreezing read only file systems is
perfectly valid, and third the freeze_super method is a special
hack for gfs2 that should not gain additional users.
Powered by blists - more mailing lists