Re: [PATCH] docs: domain: document blkiotune {read, write}_{bytes, iops}_sec

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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.

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]