I thought it wasn't possible to have inline data and an external xattr block? It doesn't make sense to do it that way compared to the normal inline xattr and external data block. Cheers, Andreas > On Mar 11, 2014, at 0:55, "Darrick J. Wong" <darrick.wong@xxxxxxxxxx> wrote: > > i_blocks covers the number of blocks allocated to an inode for data, > extents, and ACL blocks. Since it's possible for a file to have a > separate ACL block and inline data, we must be careful when expanding > an inline data file to adjust, not set, the value of i_blocks. > > Signed-off-by: Darrick J. Wong <darrick.wong@xxxxxxxxxx> > --- > lib/ext2fs/inline_data.c | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > > diff --git a/lib/ext2fs/inline_data.c b/lib/ext2fs/inline_data.c > index 72e8fa3..a9ec923 100644 > --- a/lib/ext2fs/inline_data.c > +++ b/lib/ext2fs/inline_data.c > @@ -372,7 +372,9 @@ ext2fs_inline_data_dir_expand(ext2_filsys fs, ext2_ino_t ino, > if (EXT2_HAS_INCOMPAT_FEATURE(fs->super, EXT3_FEATURE_INCOMPAT_EXTENTS)) > inode->i_flags |= EXT4_EXTENTS_FL; > inode->i_flags &= ~EXT4_INLINE_DATA_FL; > - ext2fs_iblk_set(fs, inode, 1); > + retval = ext2fs_iblk_add_blocks(fs, inode, 1); > + if (retval) > + goto errout; > inode->i_size = fs->blocksize; > retval = ext2fs_bmap2(fs, ino, inode, 0, BMAP_SET, 0, 0, &blk); > if (retval) > @@ -410,7 +412,6 @@ ext2fs_inline_data_file_expand(ext2_filsys fs, ext2_ino_t ino, > inode->i_flags |= EXT4_EXTENTS_FL; > } > inode->i_flags &= ~EXT4_INLINE_DATA_FL; > - ext2fs_iblk_set(fs, inode, 0); > inode->i_size = 0; > retval = ext2fs_write_inode(fs, ino, inode); > if (retval) > > -- > 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 -- 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
