From: Oleksandr Tyshchenko <oleksandr_tyshchenko@epam.com>
Date: Thu, 9 Dec 2021 20:05:31 +0000 (+0200)
Subject: xen/unpopulated-alloc: Drop check for virt_addr_valid() in fill_list()
X-Git-Tag: v5.17-rc1~96^2~5
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=fbf3a5c301685f7e9224bf5de9c426cf8f11ae15;p=thirdparty%2Fkernel%2Flinux.git

xen/unpopulated-alloc: Drop check for virt_addr_valid() in fill_list()

If memremap_pages() succeeds the range is guaranteed to have proper page
table, there is no need for an additional virt_addr_valid() check.

Signed-off-by: Oleksandr Tyshchenko <oleksandr_tyshchenko@epam.com>
Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Link: https://lore.kernel.org/r/1639080336-26573-2-git-send-email-olekstysh@gmail.com
Signed-off-by: Juergen Gross <jgross@suse.com>
---

diff --git a/drivers/xen/unpopulated-alloc.c b/drivers/xen/unpopulated-alloc.c
index 87e6b7db892f5..a03dc5b1f37bd 100644
--- a/drivers/xen/unpopulated-alloc.c
+++ b/drivers/xen/unpopulated-alloc.c
@@ -85,7 +85,6 @@ static int fill_list(unsigned int nr_pages)
 	for (i = 0; i < alloc_pages; i++) {
 		struct page *pg = virt_to_page(vaddr + PAGE_SIZE * i);
 
-		BUG_ON(!virt_addr_valid(vaddr + PAGE_SIZE * i));
 		pg->zone_device_data = page_list;
 		page_list = pg;
 		list_count++;