From: Jeremy Fitzhardinge <jeremy.fitzhardinge@citrix.com>
Date: Fri, 6 Mar 2009 17:56:26 +0000 (-0800)
Subject: xen/xenfs: set_page_dirty is supposed to return true if it dirties
X-Git-Tag: upstream/snapshot3+hdmi~12635^2~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=35f8c1c343f2918ea24f05282d14e711887d8278;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

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>
---

diff --git a/drivers/xen/xenfs/super.c b/drivers/xen/xenfs/super.c
index 23f1cca..afaa6ed 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 = {