bcachefs: Fix bio alloc in check_extent_checksum()

Kent Overstreet · Kent Overstreet · commit 0389c09b2fb7 · 2024-04-17T17:29:58.000-04:00
if the buffer is virtually mapped it won't be a single bvec

Signed-off-by: Kent Overstreet &lt;kent.overstreet@linux.dev&gt;
diff --git a/fs/bcachefs/backpointers.c b/fs/bcachefs/backpointers.c
@@ -470,7 +470,7 @@ static int check_extent_checksum(struct btree_trans *trans,
 		goto err;
 	}
 
-	bio = bio_alloc(ca->disk_sb.bdev, 1, REQ_OP_READ, GFP_KERNEL);
+	bio = bio_alloc(ca->disk_sb.bdev, buf_pages(data_buf, bytes), REQ_OP_READ, GFP_KERNEL);
 	bio->bi_iter.bi_sector = p.ptr.offset;
 	bch2_bio_map(bio, data_buf, bytes);
 	ret = submit_bio_wait(bio);

Original file line number	Diff line number	Diff line change
`@@ -470,7 +470,7 @@ static int check_extent_checksum(struct btree_trans *trans,`
`470`	`470`	`goto err;`
`471`	`471`	`}`
`472`	`472`
`473`		`- bio = bio_alloc(ca->disk_sb.bdev, 1, REQ_OP_READ, GFP_KERNEL);`
	`473`	`+ bio = bio_alloc(ca->disk_sb.bdev, buf_pages(data_buf, bytes), REQ_OP_READ, GFP_KERNEL);`
`474`	`474`	`bio->bi_iter.bi_sector = p.ptr.offset;`
`475`	`475`	`bch2_bio_map(bio, data_buf, bytes);`
`476`	`476`	`ret = submit_bio_wait(bio);`