linux-ext4 - Re: [PATCH 1/4] e2fsprogs: Add undo I/O manager

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]

Message-ID: <20070803182831.GW6142@schatzie.adilger.int>
Date:	Fri, 3 Aug 2007 12:28:31 -0600
From:	Andreas Dilger <adilger@...sterfs.com>
To:	"Aneesh Kumar K.V" <aneesh.kumar@...ux.vnet.ibm.com>
Cc:	tytso@....edu, linux-ext4@...r.kernel.org
Subject: Re: [PATCH 1/4] e2fsprogs: Add undo I/O manager

On Aug 03, 2007  10:19 +0530, Aneesh Kumar K.V wrote:
> Andreas Dilger wrote:
> >I thought about this also, but in fact for most uses of the undo manager
> >we want to save the information at the start instead of the end, so it
> >is possible to undo e.g. a partial e2fsck that crashes before it finishes.
> >Only with mke2fs (and, I guess tune2fs -U) does the UUID change at the
> >end.
> 
> I am not sure whether saving the information at start is needed. I 
> understand that what we are looking for is the case when the application
> crashes without doing a io_channel_close. In that case i would say the
> user can use the --force option and replay the data from the tdb file.
> The UUID could very well be changed on the disk before the application
> crashed. So even if we save UUID at the start, there are cases where it
> won't match with the disk UUID.

While this is true, I don't think it is harmful to save the UUID at the
start.  The UUID changing is the rare case, so to make this safer saving
the UUID at the start and the end is best.

> That actually brings me to another change. I would be moving the block size
> recording changes from write_file_system_identity to a separate function 
> and will be calling it at the first write.

Definitely, yes.

Cheers, Andreas
--
Andreas Dilger
Principal Software Engineer
Cluster File Systems, Inc.

-
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