File 0001-Reorder-limit-checks-in-grow-quest.patch of Package vdsm

File 0001-Reorder-limit-checks-in-grow-quest.patch of Package vdsm (Revision b8a0370be37a497fe883af63aa06d311)

Currently displaying revision b8a0370be37a497fe883af63aa06d311, show latest

From 6791ec2bd94195c711cd719c8021692720fb02fb Mon Sep 17 00:00:00 2001
From: Martin Sivak <msivak@redhat.com>
Date: Tue, 26 Nov 2013 16:53:36 +0100
Subject: [PATCH 01/11] Reorder limit checks in grow quest

balloon_min is a computed value that can (and will) go over
balloon_max. When the controller tries to use it VDSM reports
exception.

Reorder the checks so the balloon_max constant check is the latest
one to limit the value.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1034969
Change-Id: I2a8d19f7fe66bb98d97718da34ec2bad6f7987f2
Signed-off-by: Martin Sivak <msivak@redhat.com>
Reviewed-on: http://gerrit.ovirt.org/21765
Reviewed-by: Yaniv Bronhaim <ybronhei@redhat.com>
Tested-by: Yaniv Bronhaim <ybronhei@redhat.com>
---
 vdsm/mom.d/02-balloon.policy | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/vdsm/mom.d/02-balloon.policy b/vdsm/mom.d/02-balloon.policy
index 26c30e2..c98a231 100644
--- a/vdsm/mom.d/02-balloon.policy
+++ b/vdsm/mom.d/02-balloon.policy
@@ -87,10 +87,10 @@
 
         # Determine the new target for the BalloonController.  Only set
         # if the value is a large enough for the change to be worth it.
-        (if (> balloon_size guest.balloon_max)
-            (set balloon_size guest.balloon_max) 0)
         (if (< balloon_size balloon_min)
             (set balloon_size balloon_min) 0)
+        (if (> balloon_size guest.balloon_max)
+            (set balloon_size guest.balloon_max) 0)
         (if (change_big_enough guest balloon_size)
             (guest.Control "balloon_target" balloon_size) 0)
     } 0)
-- 
1.8.4.2