[libvirt] [PATCH] blkio: change the minimum weight from 100 to 10

Gao feng gaofeng at cn.fujitsu.com
Mon Aug 19 05:59:02 UTC 2013


kernel had changed the minimum weight of device blkio from
100 to 10 in commit df457f845e5449be2e7d96668791f789b3770ac7.

	commit df457f845e5449be2e7d96668791f789b3770ac7
	Author: Justin TerAvest <teravest at google.com>
	Date:   Tue Mar 8 19:45:00 2011 +0100

          blk-cgroup: Lower minimum weight from 100 to 10.

 	   We've found that we still get good, useful isolation at weights this
  	  low. I'd like to adjust the minimum so that any other changes can take
  	  these values into account.

 	  Signed-off-by: Justin TerAvest <teravest at google.com>
  	  Acked-by: Vivek Goyal <vgoyal at redhat.com>
	  Signed-off-by: Jens Axboe <jaxboe at fusionio.com>

libvirt should comport with kernel.

Signed-off-by: Gao feng <gaofeng at cn.fujitsu.com>
---
 docs/formatdomain.html.in |  4 ++--
 src/util/vircgroup.c      | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
index 83d551a..541acb3 100644
--- a/docs/formatdomain.html.in
+++ b/docs/formatdomain.html.in
@@ -766,7 +766,7 @@
         defaults. <span class="since">Since 0.8.8</span></dd>
       <dt><code>weight</code></dt>
       <dd> The optional <code>weight</code> element is the overall I/O
-        weight of the guest. The value should be in the range [100,
+        weight of the guest. The value should be in the range [10,
         1000].</dd>
       <dt><code>device</code></dt>
       <dd>The domain may have multiple <code>device</code> elements
@@ -783,7 +783,7 @@
         Each <code>device</code> element has two
         mandatory sub-elements, <code>path</code> describing the
         absolute path of the device, and <code>weight</code> giving
-        the relative weight of that device, in the range [100,
+        the relative weight of that device, in the range [10,
         1000].  <span class="since">Since 0.9.8</span></dd>
     </dl>
 
diff --git a/src/util/vircgroup.c b/src/util/vircgroup.c
index 16458a3..6e1fe6c 100644
--- a/src/util/vircgroup.c
+++ b/src/util/vircgroup.c
@@ -1784,9 +1784,9 @@ virCgroupPathOfController(virCgroupPtr group,
 int
 virCgroupSetBlkioWeight(virCgroupPtr group, unsigned int weight)
 {
-    if (weight > 1000 || weight < 100) {
+    if (weight > 1000 || weight < 10) {
         virReportError(VIR_ERR_INVALID_ARG,
-                       _("weight '%u' must be in range (100, 1000)"),
+                       _("weight '%u' must be in range (10, 1000)"),
                        weight);
         return -1;
     }
@@ -1825,7 +1825,7 @@ virCgroupGetBlkioWeight(virCgroupPtr group, unsigned int *weight)
  *
  * @group: The cgroup to change io device weight device for
  * @path: The device with a weight to alter
- * @weight: The new device weight (100-1000), or 0 to clear
+ * @weight: The new device weight (10-1000), or 0 to clear
  *
  * device_weight is treated as a write-only parameter, so
  * there isn't a getter counterpart.
@@ -1841,9 +1841,9 @@ virCgroupSetBlkioDeviceWeight(virCgroupPtr group,
     struct stat sb;
     int ret;
 
-    if (weight && (weight > 1000 || weight < 100)) {
+    if (weight && (weight > 1000 || weight < 10)) {
         virReportError(VIR_ERR_INVALID_ARG,
-                       _("weight '%u' must be in range (100, 1000)"),
+                       _("weight '%u' must be in range (10, 1000)"),
                        weight);
         return -1;
     }
-- 
1.8.3.1




More information about the libvir-list mailing list