Thread (3 messages) 3 messages, 3 authors, 2014-07-26

Re: [PATCH] debugfs: allow do_write() to handle files over 4GB

From: Darrick J. Wong <hidden>
Date: 2014-07-25 00:04:38

On Thu, Jul 24, 2014 at 05:49:19PM -0600, Andreas Dilger wrote:
Fix debugfs do_write() to correctly set i_size_high if the source
file is larger than 4GB.  Otherwise, only the low 32 bits of the
size is stored in i_size.
You might also want to set large_file if needed.  I wrote a helper to set an
inode's i_size* fields and turn on feature bits; it's been languishing at the
back end of the patchbomb for a while:
http://permalink.gmane.org/gmane.comp.file-systems.ext4/43861

--D
quoted hunk ↗ jump to hunk
Signed-off-by: Andreas Dilger <redacted>
---
 debugfs/debugfs.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/debugfs/debugfs.c b/debugfs/debugfs.c
index 1703aba..1184596 100644
--- a/debugfs/debugfs.c
+++ b/debugfs/debugfs.c
@@ -1719,6 +1719,8 @@ void do_write(int argc, char *argv[])
 		current_fs->now ? current_fs->now : time(0);
 	inode.i_links_count = 1;
 	inode.i_size = statbuf.st_size;
+	/* cast to (long long) to avoid shift warning if st_size is 32 bits */
+	inode.i_size_high = (long long)statbuf.st_size >> 32;
 	if (current_fs->super->s_feature_incompat &
 	    EXT3_FEATURE_INCOMPAT_EXTENTS) {
 		int i;
-- 
1.7.3.4

--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help