Re: [PATCH 69/73] ext2: Add fallthru support [ver #2]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

On Tue, Feb 21, 2012 at 06:06:11PM +0000, David Howells wrote:
> From: Valerie Aurora <vaurora@xxxxxxxxxx>
> Add support for fallthru directory entries to ext2.

As I mentioned, I wonder if it makes sense combine the patches for
whiteout and fallthrough director entries into a single patch.  Given
that the two patches modify the same functions, and in some cases
second modifies lines added or modified by first, it just makes life
easier if the two are folded together.

> --- a/include/linux/ext2_fs.h
> +++ b/include/linux/ext2_fs.h
> @@ -506,11 +506,14 @@ struct ext2_super_block {
>  #define EXT2_FEATURE_INCOMPAT_META_BG		0x0010
> +/* ext3/4 incompat flags take up the intervening constants */

... and the codepoint 0x2000 in the INCOMPAT mask has since already
been assigned.

As I mentioned in a comment to the previous patch, any objections if
you combine these two fields into a single ROCOMPAT feature?


						- Ted
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at

[Linux Ext4 Filesystem]     [Ecryptfs]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Photo]     [Yosemite]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [CEPH Filesystem]

  Powered by Linux