linux-ext4 - Re: [PATCH] ext4: don't manipulate recovery flag when freezing no-journal fs

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Sat, 15 Aug 2015 10:50:47 -0400
From:	Theodore Ts'o <tytso@....edu>
To:	Jan Kara <jack@...e.cz>
Cc:	Eric Sandeen <sandeen@...hat.com>,
	ext4 development <linux-ext4@...r.kernel.org>,
	Stu Mark <smark@...to.com>
Subject: Re: [PATCH] ext4: don't manipulate recovery flag when freezing
 no-journal fs

On Tue, Aug 11, 2015 at 09:22:37PM +0200, Jan Kara wrote:
> On Wed 05-08-15 15:13:58, Eric Sandeen wrote:
> > ext4 started setting needs_recovery on filesystems without journals
> > when they are unfrozen.  This makes no sense, and in fact confuses
> > blkid to the point where it doesn't recognize the filesystem at all.
> > 
> > (freeze ext2; unfreeze ext2; run blkid; see no output; run dumpe2fs,
> > see needs_recovery set on fs w/ no journal).
> > 
> > To fix this, don't manipulate the INCOMPAT_RECOVER feature on
> > filesystems without journals.
> > 
> > Reported-by: Stu Mark <smark@...to.com>
> > Signed-off-by: Eric Sandeen <sandeen@...hat.com>
> 
> The patch looks good. You can add:
> 
> Reviewed-by: Jan Kara <jack@...e.com>

Thanks, applied.

> > Note, is there a reason that in ext4_freeze, if journal_flush
> > fails, we skip the ext4_commit_super call?  I didn't change that
> > here, but it seems odd.

The only reason to call ext4_commit_super() is to persistent the
changed INCOMPAT_RECOVER flag --- well, and to update the global free
inode/block counts.  OTOH, if the journal_flush() has failed, the fact
that the global free counts aren't up to date will be the least of
your problems as far as the consistency of the file system snapshot is
concerned....

						- Ted
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html