Re: [PATCH v2 4/4] qemu: allow PMU feature to be enabled for ppc64 guests

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

 



On Tue, 2020-03-24 at 14:33 -0300, Daniel Henrique Barboza wrote:
> The PMU feature is enabled by default in ppc64 guests and can't
> be disabled via Libvirt or QEMU [1]. The current PMU feature
> implementation does not allow PMU to enabled or disabled in the
> ppc64 guest. Declaring the PMU feature will make the 'pmu'
> property to be passed on to QEMU, but this property isn't
> available for ppc64:
> 
> qemu-kvm: can't apply global host-powerpc64-cpu.pmu=on: Property '.pmu' not found
> 
> A similar error is thrown when trying to disable the PMU.
> 
> This patch standardizes the PMU handling for ppc64 guests by:
> 
> - throwing an error if the user attempts to set the feature to
> 'off', given that this feature can't be turned off at all;
> 
> - allowing the feature to be declared as 'on' in the domain XML.
> This is done by skipping ppc64 guests when creating the command
> line for this feature.
> 
> [1] https://www.redhat.com/archives/libvir-list/2020-March/msg00874.html
> Signed-off-by: Daniel Henrique Barboza <danielhb413@xxxxxxxxx>
> ---
>  src/qemu/qemu_command.c |  4 +++-
>  src/qemu/qemu_domain.c  | 11 ++++++++++-
>  2 files changed, 13 insertions(+), 2 deletions(-)

Reviewed-by: Andrea Bolognani <abologna@xxxxxxxxxx>


I'll push the series shortly.

-- 
Andrea Bolognani / Red Hat / Virtualization




[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]

  Powered by Linux