[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/7] target/i386/kvm: introduce 'pmu-cap-disabled' to set KVM
From: |
dongli . zhang |
Subject: |
Re: [PATCH 2/7] target/i386/kvm: introduce 'pmu-cap-disabled' to set KVM_PMU_CAP_DISABLE |
Date: |
Wed, 13 Nov 2024 16:13:21 -0800 |
User-agent: |
Mozilla Thunderbird |
On 11/13/24 9:15 AM, Zhao Liu wrote:
>>> Further, considering that this is currently the only case that needs to
>>> to set the VM level's capability in the CPU context, there is no need to
>>> introduce a new kvm interface (in your previous patch), which can instead
>>> be set in kvm_cpu_realizefn(), like:
>>>
>
> Now your case is not the only user of kvm_arch_pre_create_vcpu(), and
> TDX also needs this [*]. So, this is the support for bringing back your
> previous solution (preferably with comments, as I suggested earlier,
> explaining why it's necessary to handle VM-level cap in the CPU
> context). :-)
>
> [*]:
> 20241105062408.3533704-8-xiaoyao.li@intel.com/__;!!ACWV5N9M2RV99hQ!PGJQ9Kv-1mo_4YX1fb4N9YvZZqHInebtCNcSniRi3qJNPIfG5zZnDp1b1gVmO-DdpYxMvO1hlH9owAHV5UMT$">https://urldefense.com/v3/__https://lore.kernel.org/qemu-devel/20241105062408.3533704-8-xiaoyao.li@intel.com/__;!!ACWV5N9M2RV99hQ!PGJQ9Kv-1mo_4YX1fb4N9YvZZqHInebtCNcSniRi3qJNPIfG5zZnDp1b1gVmO-DdpYxMvO1hlH9owAHV5UMT$
>
>
Thank you very much for the reminder!
Dongli Zhang
[PATCH 3/7] target/i386/kvm: init PMU information only once, Dongli Zhang, 2024/11/04
[PATCH 1/7] target/i386: disable PerfMonV2 when PERFCORE unavailable, Dongli Zhang, 2024/11/04
[PATCH 4/7] target/i386/kvm: rename architectural PMU variables, Dongli Zhang, 2024/11/04