From: Eric Paris <eparis@redhat.com>
Date: Wed, 13 Oct 2010 21:50:08 +0000 (-0400)
Subject: kernel: roundup should only reference arguments once
X-Git-Tag: upstream/snapshot3+hdmi~12760^2~6
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b28efd54d9d5c8005a29cd8782335beb9daaa32d;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git

kernel: roundup should only reference arguments once

Currently the roundup macro references it's arguments more than one time.
This patch changes it so it will only use its arguments once.

Suggested-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Eric Paris <eparis@redhat.com>
Signed-off-by: James Morris <jmorris@namei.org>
---

diff --git a/include/linux/kernel.h b/include/linux/kernel.h
index 6d6eea7..1759ba5 100644
--- a/include/linux/kernel.h
+++ b/include/linux/kernel.h
@@ -58,7 +58,12 @@ extern const char linux_proc_banner[];
 
 #define FIELD_SIZEOF(t, f) (sizeof(((t*)0)->f))
 #define DIV_ROUND_UP(n,d) (((n) + (d) - 1) / (d))
-#define roundup(x, y) ((((x) + ((y) - 1)) / (y)) * (y))
+#define roundup(x, y) (					\
+{							\
+	typeof(y) __y = y;				\
+	(((x) + (__y - 1)) / __y) * __y;		\
+}							\
+)
 #define rounddown(x, y) (				\
 {							\
 	typeof(x) __x = (x);				\