aboutsummaryrefslogtreecommitdiffstats
path: root/mm
diff options
context:
space:
mode:
authorMark Rustad <mark.d.rustad@intel.com>2014-10-09 18:28:15 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2014-10-09 22:25:57 -0400
commit7c809968ffa92d41baaa9054e897436480179b20 (patch)
treeed398fc20903aa201f57aa508f1e03189802a776 /mm
parent7ade3c997208566c5bf50ece8fc319a8caf0d41a (diff)
mm/page-writeback.c: use min3/max3 macros to avoid shadow warnings
Nested calls to min/max functions result in shadow warnings in W=2 builds. Avoid the warning by using the min3 and max3 macros to get the min/max of 3 values instead of nested calls. Signed-off-by: Mark Rustad <mark.d.rustad@intel.com> Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'mm')
-rw-r--r--mm/page-writeback.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/mm/page-writeback.c b/mm/page-writeback.c
index 91d73ef1744d..35ca7102d421 100644
--- a/mm/page-writeback.c
+++ b/mm/page-writeback.c
@@ -1075,13 +1075,13 @@ static void bdi_update_dirty_ratelimit(struct backing_dev_info *bdi,
1075 } 1075 }
1076 1076
1077 if (dirty < setpoint) { 1077 if (dirty < setpoint) {
1078 x = min(bdi->balanced_dirty_ratelimit, 1078 x = min3(bdi->balanced_dirty_ratelimit,
1079 min(balanced_dirty_ratelimit, task_ratelimit)); 1079 balanced_dirty_ratelimit, task_ratelimit);
1080 if (dirty_ratelimit < x) 1080 if (dirty_ratelimit < x)
1081 step = x - dirty_ratelimit; 1081 step = x - dirty_ratelimit;
1082 } else { 1082 } else {
1083 x = max(bdi->balanced_dirty_ratelimit, 1083 x = max3(bdi->balanced_dirty_ratelimit,
1084 max(balanced_dirty_ratelimit, task_ratelimit)); 1084 balanced_dirty_ratelimit, task_ratelimit);
1085 if (dirty_ratelimit > x) 1085 if (dirty_ratelimit > x)
1086 step = dirty_ratelimit - x; 1086 step = dirty_ratelimit - x;
1087 } 1087 }