diff options
author | Jens Axboe <axboe@kernel.dk> | 2018-09-28 11:40:17 -0400 |
---|---|---|
committer | Jens Axboe <axboe@kernel.dk> | 2018-09-28 11:40:17 -0400 |
commit | 15c206887603a452f13fbfde2db0f8830d37028c (patch) | |
tree | df1fe73c4682cf40b238cf72d2f966387799edc8 | |
parent | 587562d0c7cd6861f4f90a2eb811cccb1a376f5f (diff) |
Revert "xen/blkfront: When purging persistent grants, keep them in the buffer"
Fix didn't work for all cases, reverting to add a (hopefully)
better fix.
This reverts commit f151ba989d149bbdfc90e5405724bbea094f9b17.
Signed-off-by: Jens Axboe <axboe@kernel.dk>
-rw-r--r-- | drivers/block/xen-blkfront.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/block/xen-blkfront.c b/drivers/block/xen-blkfront.c index 3b441fe69c0d..a71d817e900d 100644 --- a/drivers/block/xen-blkfront.c +++ b/drivers/block/xen-blkfront.c | |||
@@ -2667,9 +2667,11 @@ static void purge_persistent_grants(struct blkfront_info *info) | |||
2667 | gnttab_query_foreign_access(gnt_list_entry->gref)) | 2667 | gnttab_query_foreign_access(gnt_list_entry->gref)) |
2668 | continue; | 2668 | continue; |
2669 | 2669 | ||
2670 | list_del(&gnt_list_entry->node); | ||
2670 | gnttab_end_foreign_access(gnt_list_entry->gref, 0, 0UL); | 2671 | gnttab_end_foreign_access(gnt_list_entry->gref, 0, 0UL); |
2671 | gnt_list_entry->gref = GRANT_INVALID_REF; | ||
2672 | rinfo->persistent_gnts_c--; | 2672 | rinfo->persistent_gnts_c--; |
2673 | __free_page(gnt_list_entry->page); | ||
2674 | kfree(gnt_list_entry); | ||
2673 | } | 2675 | } |
2674 | 2676 | ||
2675 | spin_unlock_irqrestore(&rinfo->ring_lock, flags); | 2677 | spin_unlock_irqrestore(&rinfo->ring_lock, flags); |