Re: [kvm-devel] [kvm-ppc-devel] [PATCH] Split kvm_vcpu to support new archs.

2007-10-19 Thread Hollis Blanchard
On Thu, 2007-10-18 at 17:04 -0500, Anthony Liguori wrote: > Hollis Blanchard wrote: > > > > I must be misunderstanding, because this seems completely backwards to > > me. With your nesting, any time architecture code wants to access > > architecture state (which is almost all the time), you'd *need

Re: [kvm-devel] [kvm-ppc-devel] [PATCH] Split kvm_vcpu to support new archs.

2007-10-19 Thread Carsten Otte
Hollis Blanchard wrote: > I must be misunderstanding, because this seems completely backwards to > me. With your nesting, any time architecture code wants to access > architecture state (which is almost all the time), you'd *need* > container_of: > > void arch_func(struct kvm_vcpu *vcpu) {

Re: [kvm-devel] [kvm-ppc-devel] [PATCH] Split kvm_vcpu to support new archs.

2007-10-18 Thread Anthony Liguori
Hollis Blanchard wrote: > On Thu, 2007-10-18 at 16:31 -0500, Anthony Liguori wrote: > >> Hollis Blanchard wrote: >> >>> On Thu, 2007-10-18 at 16:04 -0500, Anthony Liguori wrote: >>> >>> Hollis Blanchard wrote: > What is the problem with > embed

Re: [kvm-devel] [kvm-ppc-devel] [PATCH] Split kvm_vcpu to support new archs.

2007-10-18 Thread Hollis Blanchard
On Thu, 2007-10-18 at 16:31 -0500, Anthony Liguori wrote: > Hollis Blanchard wrote: > > On Thu, 2007-10-18 at 16:04 -0500, Anthony Liguori wrote: > > > >> Hollis Blanchard wrote: > >> > >>> What is the problem with > >>> embedding an architecture-specific sub-structure, i.e. > >>> s