[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 11/12] kvm: x86: Pass KVMState to kvm_arch_get_s
From: |
Eduardo Habkost |
Subject: |
Re: [Qemu-devel] [PATCH 11/12] kvm: x86: Pass KVMState to kvm_arch_get_supported_cpuid |
Date: |
Thu, 9 Jun 2011 15:08:29 -0300 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Thu, Jun 09, 2011 at 07:41:03PM +0200, Jan Kiszka wrote:
<snip>
> > I'm wondering if it wouldn't be simpler to keep the existing interface
> > but just initialize CPUState->kvm_state earlier (today it is initialized
> > only on kvm_init_vcpu(), although the kvm_state global is initialized
> > much earlier).
>
> If there was more need for a slit up, I would agree. But I do not see it.
I guess it won't hurt to make the functions require only what they
really need.
> >
> > Even with this new KVMState-based interface, code that needs to use
> > these functions before kvm_init_vcpu() (e.g.
> > check_features_against_host()) will have to use the 'kvm_state' global
> > (and I would like to avoid that).
>
> That is a different topic that needs to be resolved for other use cases
> as well (e.g. for kvm devices). In many cases you should be able to find
> a way to pass kvm_state from functions that already holds a reference.
> Is that not true for your use case?
My use case is cpu_x86_register()/check_features_against_host(), that
hold only a CPUState reference, and I wouldn't like to add a new
reference to the kvm_state global variable there.
--
Eduardo
- Re: [Qemu-devel] [PATCH 03/12] Switch build system to accompanied kernel headers, (continued)
- Re: [Qemu-devel] [PATCH 03/12] Switch build system to accompanied kernel headers, Stefan Weil, 2011/06/23
- Re: [Qemu-devel] [PATCH 03/12] Switch build system to accompanied kernel headers, Jan Kiszka, 2011/06/23
- [Qemu-devel] [PATCH] Fix fallouts from Linux header inclusion, Jan Kiszka, 2011/06/23
- Re: [Qemu-devel] [PATCH] Fix fallouts from Linux header inclusion, Andreas Färber, 2011/06/23
- Re: [Qemu-devel] [PATCH] Fix fallouts from Linux header inclusion, Stefan Weil, 2011/06/23
- Re: [Qemu-devel] [PATCH] Fix fallouts from Linux header inclusion, Blue Swirl, 2011/06/26
[Qemu-devel] [PATCH 06/12] kvm: Drop useless zero-initializations, Jan Kiszka, 2011/06/08
[Qemu-devel] [PATCH 11/12] kvm: x86: Pass KVMState to kvm_arch_get_supported_cpuid, Jan Kiszka, 2011/06/08
[Qemu-devel] [PATCH 02/12] Import kernel headers, Jan Kiszka, 2011/06/08
Re: [Qemu-devel] [PATCH 00/12] [uq/master] Import linux headers and some cleanups, Marcelo Tosatti, 2011/06/20