xen/xenfs: set_page_dirty is supposed to return true if it dirties

I don't think it matters at all in this case (there's only one caller
which checks the return value), but may as well be strictly correct.

[ Impact: cleanup ]

Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>

1 files changed, 1 insertions, 3 deletions

diff --git a/drivers/xen/xenfs/super.c b/drivers/xen/xenfs/super.c
index 23f1cca5a2e9..afaa6ede0168 100644
--- a/drivers/xen/xenfs/super.c
+++ b/drivers/xen/xenfs/super.c
@@ -26,9 +26,7 @@ MODULE_LICENSE("GPL");
 
 static int xenfs_set_page_dirty(struct page *page)
 {
-        if (!PageDirty(page))
-                SetPageDirty(page);
-        return 0;
+        return !TestSetPageDirty(page);
 }
 
 static const struct address_space_operations xenfs_aops = {