[libvirt] [PATCH] docs: domain: document blkiotune {read, write}_{bytes, iops}_sec
Pavel Hrdina
phrdina at redhat.com
Thu Apr 14 14:39:14 UTC 2016
On Thu, Apr 14, 2016 at 03:38:20PM +0200, Andrea Bolognani wrote:
> On Thu, 2016-04-14 at 15:25 +0200, Pavel Hrdina wrote:
> > On Wed, Apr 13, 2016 at 04:37:17PM -0400, Cole Robinson wrote:
> > >
> > > Added with commit 3b431929 in v1.2.2 but never documented
> > >
> > > https://bugzilla.redhat.com/show_bug.cgi?id=1313613
> > > ---
> > > docs/formatdomain.html.in | 18 +++++++++++++++++-
> > > 1 file changed, 17 insertions(+), 1 deletion(-)
> > ACK, Pavel
>
> I was looking over the same patch and I'd like to propose an
> alternative diff, which IMHO fits better with the rest of the
> page.
>
> Cheers.
>
>
> diff --git a/docs/formatdomain.html.in b/docs/formatdomain.html.in
> index c2955eb..fef2f25 100644
> --- a/docs/formatdomain.html.in
> +++ b/docs/formatdomain.html.in
> @@ -991,6 +991,10 @@
> <device>
> <path>/dev/sdb</path>
> <weight>500</weight>
> + <read_bytes_sec>10000</read_bytes_sec>
> + <write_bytes_sec>10000</write_bytes_sec>
> + <read_iops_sec>20000</read_iops_sec>
> + <write_iops_sec>20000</write_iops_sec>
> </device>
> </blkiotune>
> ...
> @@ -1025,8 +1029,22 @@
> absolute path of the device, and <code>weight</code> giving
> the relative weight of that device, in the range [100,
> 1000]. After kernel 2.6.39, the value could be in the
> - range [10, 1000].<span class="since">Since 0.9.8</span></dd>
> - </dl>
> + range [10, 1000]. <span class="since">Since 0.9.8</span><br/>
> + Additionally, the following optional sub-elements can be used:
> + <dl>
> + <dt><code>read_bytes_sec</code></dt>
> + <dd>Read throughput limit in bytes per second.
> + <span class="since">Since 1.2.2</span></dd>
> + <dt><code>write_bytes_sec</code></dt>
> + <dd>Write throughput limit in bytes per second.
> + <span class="since">Since 1.2.2</span></dd>
> + <dt><code>read_iops_sec</code></dt>
> + <dd>Read I/O operations per second limit.
> + <span class="since">Since 1.2.2</span></dd>
> + <dt><code>write_iops_sec</code></dt>
> + <dd>Write I/O operations per second limit.
> + <span class="since">Since 1.2.2</span></dd>
> + </dl></dd></dl>
Right, this is a better approach. ACK with this modification.
More information about the libvir-list
mailing list