Responsibility for byte swapping the extents information rests with the low-level extent code, which translates the on-disk extents information to the abstract extent format. The on-disk format will eventually get more complicated, in order to add support for 64-bit block numbers, bit-compressed extents, etc. So to avoid needing to expose all of that complexity in swapfs.c, the in-memory contents of i_blocks will not be byte-swapped and will be identical to the on-disk format. Signed-off-by: "Theodore Ts'o" <tytso@xxxxxxx> --- lib/ext2fs/swapfs.c | 4 ++++ 1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/lib/ext2fs/swapfs.c b/lib/ext2fs/swapfs.c index bd0844d..52ca6a0 100644 --- a/lib/ext2fs/swapfs.c +++ b/lib/ext2fs/swapfs.c @@ -158,7 +158,11 @@ void ext2fs_swap_inode_full(ext2_filsys fs, struct ext2_inode_large *t, if (!hostorder) has_data_blocks = ext2fs_inode_data_blocks(fs, (struct ext2_inode *) t); + if (hostorder && (f->i_flags & EXT4_EXTENTS_FL)) + has_data_blocks = 0; t->i_flags = ext2fs_swab32(f->i_flags); + if (hostorder && (t->i_flags & EXT4_EXTENTS_FL)) + has_data_blocks = 0; t->i_dir_acl = ext2fs_swab32(f->i_dir_acl); if (!islnk || has_data_blocks ) { for (i = 0; i < EXT2_N_BLOCKS; i++) -- 1.5.3.1.19.gb5ef6 - To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html