This BUG_ON() in blkif_free() is incorrect, because indirect page can be added
to list info->indirect_pages in blkif_completion() no matter feature_persistent
is true or false.
Signed-off-by: Bob Liu <bob....@oracle.com>
---
drivers/block/xen-blkfront.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/block/xen-blkfront.c b/drivers/block/xen-blkfront.c
index e266d17..c98fcd0 100644
--- a/drivers/block/xen-blkfront.c
+++ b/drivers/block/xen-blkfront.c
@@ -986,7 +986,6 @@ static void blkif_free(struct blkfront_info *info, int
suspend)
if (!list_empty(&info->indirect_pages)) {
struct page *indirect_page, *n;
- BUG_ON(info->feature_persistent);
list_for_each_entry_safe(indirect_page, n,
&info->indirect_pages, lru) {
list_del(&indirect_page->lru);
__free_page(indirect_page);
--
1.7.10.4
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
