[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 03/10] target/i386: Fix duplicated feature name in FEAT_KV
From: |
Igor Mammedov |
Subject: |
Re: [PATCH v2 03/10] target/i386: Fix duplicated feature name in FEAT_KVM |
Date: |
Fri, 8 Sep 2023 16:21:09 +0200 |
On Fri, 8 Sep 2023 14:45:27 +0200
Tim Wiederhake <twiederh@redhat.com> wrote:
> The mistake became apparent as there were two features with the same name
> in this cpuid leaf. The names are now in line with the documentation from
> https://kernel.org/doc/html/latest/virt/kvm/x86/cpuid.html
I'd describe what duplication breaks and it's effects.
and also why it's considered that it's safe to change names here.
> Fixes: 642258c6c7 ("kvm: add kvmclock to its second bit")
> Signed-off-by: Tim Wiederhake <twiederh@redhat.com>
> ---
> target/i386/cpu.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/i386/cpu.c b/target/i386/cpu.c
> index f10d343935..f0fedf4b88 100644
> --- a/target/i386/cpu.c
> +++ b/target/i386/cpu.c
> @@ -852,7 +852,7 @@ FeatureWordInfo feature_word_info[FEATURE_WORDS] = {
> [FEAT_KVM] = {
> .type = CPUID_FEATURE_WORD,
> .feat_names = {
> - "kvmclock", "kvm-nopiodelay", "kvm-mmu", "kvmclock",
> + "kvmclock", "kvm-nopiodelay", "kvm-mmu", "kvmclock2",
> "kvm-asyncpf", "kvm-steal-time", "kvm-pv-eoi", "kvm-pv-unhalt",
> NULL, "kvm-pv-tlb-flush", NULL, "kvm-pv-ipi",
> "kvm-poll-control", "kvm-pv-sched-yield", "kvm-asyncpf-int",
> "kvm-msi-ext-dest-id",
- [PATCH v2 00/10] Generate x86 cpu features, Tim Wiederhake, 2023/09/08
- [PATCH v2 01/10] target/i386: Add missing feature names in FEAT_VMX_EPT_VPID_CAPS, Tim Wiederhake, 2023/09/08
- [PATCH v2 06/10] target/i386: Format feature_word_info.c.inc: Remove comments, Tim Wiederhake, 2023/09/08
- [PATCH v2 03/10] target/i386: Fix duplicated feature name in FEAT_KVM, Tim Wiederhake, 2023/09/08
- Re: [PATCH v2 03/10] target/i386: Fix duplicated feature name in FEAT_KVM,
Igor Mammedov <=
- [PATCH v2 09/10] target/i386: Format feature_word_info.c.inc: Whitespaces and trailing commas, Tim Wiederhake, 2023/09/08
- [PATCH v2 07/10] target/i386: Format feature_word_info.c.inc: Fill out feat_names, Tim Wiederhake, 2023/09/08
- [PATCH v2 02/10] target/i386: Fix feature names in FEAT_VMX_EPT_VPID_CAPS, Tim Wiederhake, 2023/09/08
- [PATCH v2 04/10] target/i386: Split out feature_word_info, Tim Wiederhake, 2023/09/08
- [PATCH v2 10/10] target/i386: Autogenerate feature_word_info.c.inc, Tim Wiederhake, 2023/09/08
- [PATCH v2 05/10] target/i386: Translate feature_word_info to yaml, Tim Wiederhake, 2023/09/08