c/mirror_zfs
1
0
Fork 0
mirror of https://git.proxmox.com/git/mirror_zfs.git synced 2024-12-26 03:09:34 +03:00
Code Issues Packages Projects Releases Wiki Activity

vdev_disk: don't touch vbio after its handed off to the kernel

Browse Source 
After IO is unplugged, it may complete immediately and vbio_completion
be called on interrupt context. That may interrupt or deschedule our
task. If its the last bio, the vbio will be freed. Then, we get
rescheduled, and try to write to freed memory through vbio->.

This patch just removes the the cleanup, and the corresponding assert.
These were leftovers from a previous iteration of vbio_submit() and were
always "belt and suspenders" ops anyway, never strictly required.

Sponsored-by: Klara, Inc.
Sponsored-by: Wasabi Technology, Inc
Reported-by: Rich Ercolani <rincebrain@gmail.com>
Signed-off-by: Rob Norris <rob.norris@klarasystems.com>
(cherry picked from commit 917ff75e95)
This commit is contained in:
Rob Norris 2024-04-04 09:17:07 +11:00 committed by Brian Behlendorf
parent deb7a84231
commit 28520cad25
1 changed files with 6 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
module/os/linux/zfs/vdev_disk.c
View File

@ -758,8 +758,6 @@ vbio_fill_cb(struct page *page, size_t off, size_t len, void *priv)
static void static void
vbio_submit(vbio_t *vbio, abd_t *abd, uint64_t size) vbio_submit(vbio_t *vbio, abd_t *abd, uint64_t size)
{ {
ASSERT(vbio->vbio_bdev);
/* /*
* We plug so we can submit the BIOs as we go and only unplug them when * We plug so we can submit the BIOs as we go and only unplug them when
* they are fully created and submitted. This is important; if we don't * they are fully created and submitted. This is important; if we don't
@ -777,12 +775,15 @@ vbio_submit(vbio_t *vbio, abd_t *abd, uint64_t size)
vbio->vbio_bio->bi_end_io = vbio_completion; vbio->vbio_bio->bi_end_io = vbio_completion;
vbio->vbio_bio->bi_private = vbio; vbio->vbio_bio->bi_private = vbio;
/*
* Once submitted, vbio_bio now owns vbio (through bi_private) and we
* can't touch it again. The bio may complete and vbio_completion() be
* called and free the vbio before this task is run again, so we must
* consider it invalid from this point.
*/
vdev_submit_bio(vbio->vbio_bio); vdev_submit_bio(vbio->vbio_bio);
blk_finish_plug(&plug); blk_finish_plug(&plug);
vbio->vbio_bio = NULL;
vbio->vbio_bdev = NULL;
} }
/* IO completion callback */ /* IO completion callback */