On Dec 12, 2013, at 8:26 AM, David Sterba <[email protected]> wrote: > Set the EXTENT_DATA_COMPRESSED flag together with EXTENT_ENCODED as > defined by fiemap spec. > > Signed-off-by: David Sterba <[email protected]> > --- > fs/btrfs/extent_io.c | 9 +++++++-- > 1 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/fs/btrfs/extent_io.c b/fs/btrfs/extent_io.c > index 5ea0ef5..8a28f15 100644 > --- a/fs/btrfs/extent_io.c > +++ b/fs/btrfs/extent_io.c > > @@ -4220,9 +4222,12 @@ int extent_fiemap(struct inode *inode, struct > fiemap_extent_info *fieinfo, > > if (ref_cnt > 1) > flags |= FIEMAP_EXTENT_SHARED; > + em_phys_len = em->block_len; > } > - if (test_bit(EXTENT_FLAG_COMPRESSED, &em->flags)) > + if (test_bit(EXTENT_FLAG_COMPRESSED, &em->flags)) { > flags |= FIEMAP_EXTENT_ENCODED; > + flags |= FIEMAP_EXTENT_DATA_COMPRESSED;
(minor nit) Could combine these (not sure it makes a difference to the
compiler):
flags |= (FIEMAP_EXTENT_ENCODED |
FIEMAP_EXTENT_DATA_COMPRESSED);
Cheers, Andreas
signature.asc
Description: Message signed with OpenPGP using GPGMail
