aboutsummaryrefslogtreecommitdiffstats
path: root/module/os/linux
diff options
context:
space:
mode:
authorRob Norris <[email protected]>2024-04-10 13:14:13 +1000
committerBrian Behlendorf <[email protected]>2024-04-12 08:53:48 -0700
commit7ad2616d378f31fb9b15507b7038e445b80a4842 (patch)
treee0b59f9458992def2f92ad5bb833921f0bce5221 /module/os/linux
parentd0d9dccc61d2ea9776497d2372a1d43cf8fe3d24 (diff)
vdev_disk: fix alignment check when buffer has non-zero starting offset
If a linear buffer spans multiple pages, and the first page has a non-zero starting offset, the checker would not include the offset, and so would think there was an alignment gap at the end of the first page, rather than at the start. That is, for a 16K buffer spread across five pages with an initial 512B offset: [.XXXXXXX][XXXXXXXX][XXXXXXXX][XXXXXXXX][XXXXXXX.] It would be interpreted as: [XXXXXXX.][XXXXXXXX]... And be rejected as misaligned. Since it's already a linear ABD, the "linearising" copy would just reuse the buffer as-is, and the second check would failing, tripping the VERIFY in vdev_disk_io_rw(). This commit fixes all this by including the offset in the check for end-of-page alignment. Sponsored-by: Klara, Inc. Sponsored-by: Wasabi Technology, Inc. Signed-off-by: Rob Norris <[email protected]> (cherry picked from commit 1bf649cb0a1cc6e48dce848611ba327eb283000e)
Diffstat (limited to 'module/os/linux')
-rw-r--r--module/os/linux/zfs/vdev_disk.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/module/os/linux/zfs/vdev_disk.c b/module/os/linux/zfs/vdev_disk.c
index 12157d3b6..223b41068 100644
--- a/module/os/linux/zfs/vdev_disk.c
+++ b/module/os/linux/zfs/vdev_disk.c
@@ -865,7 +865,7 @@ vdev_disk_check_pages_cb(struct page *page, size_t off, size_t len, void *priv)
* Note if we're taking less than a full block, so we can check it
* above on the next call.
*/
- s->end = len & s->bmask;
+ s->end = (off+len) & s->bmask;
/* All blocks after the first must start on a block size boundary. */
if (s->npages != 0 && (off & s->bmask) != 0)