iomap: iomap_write_failed fix

Andreas Gruenbacher · Darrick J. Wong · commit b71450e2cc4b · 2022-05-08T14:47:42.000-07:00
The @LenD parameter of truncate_pagecache_range() should be the offset of the last byte of the hole, not the first byte beyond it. Fixes: ae259a9 ("fs: introduce iomap infrastructure") Signed-off-by: Andreas Gruenbacher <agruenba@redhat.com> Reviewed-by: Darrick J. Wong <djwong@kernel.org> Signed-off-by: Darrick J. Wong <djwong@kernel.org>
diff --git a/fs/iomap/buffered-io.c b/fs/iomap/buffered-io.c
@@ -531,7 +531,8 @@ iomap_write_failed(struct inode *inode, loff_t pos, unsigned len)
 	 * write started inside the existing inode size.
 	 */
 	if (pos + len > i_size)
-		truncate_pagecache_range(inode, max(pos, i_size), pos + len);
+		truncate_pagecache_range(inode, max(pos, i_size),
+					 pos + len - 1);
 }
 
 static int iomap_read_folio_sync(loff_t block_start, struct folio *folio,

Original file line number	Diff line number	Diff line change
`@@ -531,7 +531,8 @@ iomap_write_failed(struct inode *inode, loff_t pos, unsigned len)`
`531`	`531`	`* write started inside the existing inode size.`
`532`	`532`	`*/`
`533`	`533`	`if (pos + len > i_size)`
`534`		`- truncate_pagecache_range(inode, max(pos, i_size), pos + len);`
	`534`	`+ truncate_pagecache_range(inode, max(pos, i_size),`
	`535`	`+ pos + len - 1);`
`535`	`536`	`}`
`536`	`537`
`537`	`538`	`static int iomap_read_folio_sync(loff_t block_start, struct folio *folio,`