xen/privcmd: Mark pages as dirty

pages need to be marked as dirty before unpinned it in
unlock_pages() which was oversight. This is fixed now.

Signed-off-by: Souptick Joarder <jrdr.linux@gmail.com>
Suggested-by: John Hubbard <jhubbard@nvidia.com>
Reviewed-by: Juergen Gross <jgross@suse.com>
Reviewed-by: Paul Durrant <paul@xen.org>
Cc: John Hubbard <jhubbard@nvidia.com>
Cc: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Cc: Paul Durrant <xadimgnik@gmail.com>
Link: https://lore.kernel.org/r/1594525195-28345-3-git-send-email-jrdr.linux@gmail.com
Signed-off-by: Juergen Gross <jgross@suse.com>

diff --git a/drivers/xen/privcmd.c b/drivers/xen/privcmd.c
index a9b2baa2c7bef82453ca85060b7134d34ce8147f..aa8ca58e6efc551c263297991a95e20ebe070c49 100644 (file)
--- a/drivers/xen/privcmd.c
+++ b/drivers/xen/privcmd.c
@@ -612,8 +612,11 @@ static void unlock_pages(struct page *pages[], unsigned int nr_pages)
 {
        unsigned int i;
 
-       for (i = 0; i < nr_pages; i++)
+       for (i = 0; i < nr_pages; i++) {
+               if (!PageDirty(pages[i]))
+                       set_page_dirty_lock(pages[i]);
                put_page(pages[i]);
+       }
 }
 
 static long privcmd_ioctl_dm_op(struct file *file, void __user *udata)