[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/3] target-i386: replace cpuid_*features fields
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 3/3] target-i386: replace cpuid_*features fields with a feature word array |
Date: |
Fri, 14 Dec 2012 10:27:34 -0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Fri, Dec 14, 2012 at 10:38:50AM +0100, Igor Mammedov wrote:
> On Wed, 12 Dec 2012 20:22:26 -0200
> Eduardo Habkost <address@hidden> wrote:
>
> > This replaces the feature-bit fields on both X86CPU and x86_def_t
> > structs with an array.
> >
> > With this, we will be able to simplify code that simply does the same
> > operation on all feature words (e.g. kvm_check_features_against_host(),
> > filter_features_for_kvm(), add_flagname_to_bitmaps(), and CPU
> > feature-bit property lookup/registration).
> >
>
> do you have a patch that simplifies kvm_check_features_against_host() using
> this?
I have a very old one, based on an older (and more complex) version of
this series:
https://github.com/ehabkost/qemu-hacks/commit/eb01d374baecf6df26fd6f0d0bb23f2e1547f499
It's in the work/cpuid-refactor-v0.22-2012-08-31 branch in my git
repository.
That branch also has some patches to merge kvm_check_features_against_host()
and filter_features_for_kvm() (because the purpose of
kvm_check_features_against_host() is simply to check if anything is
going to be filtered out by filter_features_for_kvm()).
If people are happy with the approach in this series, I plan to write
and submit cleanups for kvm_cpu_fill_host(),
kvm_check_features_against_host(), filter_features_for_kvm(),
add_flagname_to_bitmaps(), and the cpudef -> CPU feature copying code.
There's so much code that could be cleaned up using the array, that I am
afraid that it would cause too much conflicts in the CPU properties
work. So I can wait until the CPU properties series are submitted before
making the cleanups, if necessary.
--
Eduardo
Re: [Qemu-devel] [PATCH 3/3] target-i386: replace cpuid_*features fields with a feature word array, Andreas Färber, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] target-i386: replace cpuid_*features fields with a feature word array, Eduardo Habkost, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] target-i386: replace cpuid_*features fields with a feature word array, Andreas Färber, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] target-i386: replace cpuid_*features fields with a feature word array, Eduardo Habkost, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] target-i386: replace cpuid_*features fields with a feature word array, Igor Mammedov, 2012/12/14
- Re: [Qemu-devel] [PATCH 3/3] target-i386: replace cpuid_*features fields with a feature word array, Eduardo Habkost, 2012/12/14