linux-ext4 - Re: [PATCH] e2fsprogs: remove fragment support (V3)

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] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 11 Aug 2007 23:31:29 -0400
From:	Theodore Tso <tytso@....edu>
To:	Coly Li <coyli@...e.de>
Cc:	linux-ext4@...r.kernel.org, Andreas Dilger <adilger@...sterfs.com>
Subject: Re: [PATCH] e2fsprogs: remove fragment support (V3)

On Sat, Aug 11, 2007 at 01:13:39AM +0800, Coly Li wrote:
> -----BEGIN PGP SIGNED MESSAGE-----

If you're going to use PGP signatures, please use the MIME-PGP that
doesn't cause annoying quoting to take place, i.e.:

> - -   switch (os) {
> - -       case EXT2_OS_HURD:
> - -           frag = inode->osd2.hurd2.h_i_frag;

Also, please take a look at my recent patch series to remove support
for PPC big-endian byte-swapped filesystems.  Separate patches which
makes it obvious what is being removed and makes it easy to review are
much appreciated.

In addition, for now, I'd like to keep the code which checks the
remaining fragment fields in the inode and superblock since they act
as a safety check to make sure the filesystem is sane and we don't
have garbage there.  The checksum fields will obviate this need, but
keeping the checks there for ext2/ext3 filesystems seem like a good
idea.

Dropping the union is probably fine, since at this point it looks
pretty clear that both the Hurd and Masix is dead.  But let's do the
cleanups a little at a time, and I'd probably start with just removing
the cruft from the mke2fs options and man pages.

Regards,

    	       	   	  	      - 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