[libvirt] [PATCH v2 1/4] news: Support for kvm dedicated performance hint in 5.7

Han Han posted 4 patches 6 years, 2 months ago
[libvirt] [PATCH v2 1/4] news: Support for kvm dedicated performance hint in 5.7
Posted by Han Han 6 years, 2 months ago
Signed-off-by: Han Han <hhan@redhat.com>
---
 docs/news.xml | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/docs/news.xml b/docs/news.xml
index c362bf3a..1c063b7e 100644
--- a/docs/news.xml
+++ b/docs/news.xml
@@ -269,6 +269,18 @@
           traditional libvirtd by default.
         </description>
       </change>
+      <change>
+        <summary>
+          qemu: Support kvm-hint-dedicated performance hint
+        </summary>
+        <description>
+          With <code>&lt;hint-dedicated state=&apos;on&apos;/&gt;</code> and
+          <code>&lt;cpu mode=&apos;host-passthrough&apos;/&gt;</code>, it
+          allows a guest to enable optimizations when running on dedicated
+          vCPUs. Qemu over <code>2.12</code> and kernel over <code>4.17</code>
+          are required.
+        </description>
+      </change>
     </section>
     <section title="Removed features">
       <change>
-- 
2.23.0

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

Re: [libvirt] [PATCH v2 1/4] news: Support for kvm dedicated performance hint in 5.7
Posted by Andrea Bolognani 6 years, 2 months ago
On Mon, 2019-11-11 at 09:58 +0800, Han Han wrote:
> +      <change>
> +        <summary>
> +          qemu: Support kvm-hint-dedicated performance hint
> +        </summary>
> +        <description>
> +          With <code>&lt;hint-dedicated state=&apos;on&apos;/&gt;</code> and
> +          <code>&lt;cpu mode=&apos;host-passthrough&apos;/&gt;</code>, it

No need to use "&apos;", a simple "'" will work just fine.

> +          allows a guest to enable optimizations when running on dedicated
> +          vCPUs. Qemu over <code>2.12</code> and kernel over <code>4.17</code>

s/Qemu/QEMU/

and the minimum version is 2.12.1 according to cb12c59dac04.


With the nits mentioned above addressed,

  Reviewed-by: Andrea Bolognani <abologna@redhat.com>

-- 
Andrea Bolognani / Red Hat / Virtualization

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

Re: [libvirt] [PATCH v2 1/4] news: Support for kvm dedicated performance hint in 5.7
Posted by Han Han 6 years, 2 months ago
On Thu, Dec 5, 2019 at 6:31 PM Andrea Bolognani <abologna@redhat.com> wrote:

> On Mon, 2019-11-11 at 09:58 +0800, Han Han wrote:
> > +      <change>
> > +        <summary>
> > +          qemu: Support kvm-hint-dedicated performance hint
> > +        </summary>
> > +        <description>
> > +          With <code>&lt;hint-dedicated
> state=&apos;on&apos;/&gt;</code> and
> > +          <code>&lt;cpu mode=&apos;host-passthrough&apos;/&gt;</code>,
> it
>
> No need to use "&apos;", a simple "'" will work just fine.
>
> > +          allows a guest to enable optimizations when running on
> dedicated
> > +          vCPUs. Qemu over <code>2.12</code> and kernel over
> <code>4.17</code>
>
> s/Qemu/QEMU/
>
> and the minimum version is 2.12.1 according to cb12c59dac04.
>
> Feature introduced version in commit msg cb12c59dac04 of  is not correct
here.
Check that in qemu code, for the commit be7773268d target-i386: add
KVM_HINTS_DEDICATED performance hint
# git tag --contains=be7773268d|sort -n |head -n2
s
v2.12.0

So the minimum version is v2.12.0 not v2.12.1

Fix it at
https://www.redhat.com/archives/libvir-list/2019-December/msg00319.html

>
> With the nits mentioned above addressed,
>
>   Reviewed-by: Andrea Bolognani <abologna@redhat.com>
>
> --
> Andrea Bolognani / Red Hat / Virtualization
>
>

-- 
Best regards,
-----------------------------------
Han Han
Quality Engineer
Redhat.

Email: hhan@redhat.com
Phone: +861065339333
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list