]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commitdiff
ocfs2: return non-zero st_blocks for inline data
authorJohn Haxby <john.haxby@oracle.com>
Thu, 14 Jan 2016 23:17:12 +0000 (15:17 -0800)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 15 Jan 2016 00:00:49 +0000 (16:00 -0800)
Some versions of tar assume that files with st_blocks == 0 do not
contain any data and will skip reading them entirely.  See also commit
9206c561554c ("ext4: return non-zero st_blocks for inline data").

Signed-off-by: John Haxby <john.haxby@oracle.com>
Reviewed-by: Mark Fasheh <mfasheh@suse.de>
Cc: Joel Becker <jlbec@evilplan.org>
Acked-by: Gang He <ghe@suse.com>
Reviewed-by: Junxiao Bi <junxiao.bi@oracle.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/ocfs2/file.c

index 0e5b4515f92e7a875a6f396db0d5648c9c157ad1..d6312793250927b1efacf4f09c18c38f7da75e5d 100644 (file)
@@ -1302,6 +1302,14 @@ int ocfs2_getattr(struct vfsmount *mnt,
        }
 
        generic_fillattr(inode, stat);
+       /*
+        * If there is inline data in the inode, the inode will normally not
+        * have data blocks allocated (it may have an external xattr block).
+        * Report at least one sector for such files, so tools like tar, rsync,
+        * others don't incorrectly think the file is completely sparse.
+        */
+       if (unlikely(OCFS2_I(inode)->ip_dyn_features & OCFS2_INLINE_DATA_FL))
+               stat->blocks += (stat->size + 511)>>9;
 
        /* We set the blksize from the cluster size for performance */
        stat->blksize = osb->s_clustersize;