Re: [Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-20 Thread Jan Kiszka
On 2011-06-18 01:28, Alexander Graf wrote: On 17.06.2011, at 22:39, Scott Wood wrote: Share the TLB array with KVM. This allows us to set the initial TLB both on initial boot and reset, is useful for debugging, and could eventually be used to support migration. Signed-off-by: Scott Wood

Re: [Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-20 Thread Avi Kivity
On 06/20/2011 10:41 AM, Jan Kiszka wrote: Those hopefully shouldn't be required anymore soon - when Jan's patches make it upstream. Jan, how's progress on that front? I can only forward this question: Avi, what are the plans for http://thread.gmane.org/gmane.comp.emulators.kvm.devel/73917?

Re: [Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-20 Thread Jan Kiszka
On 2011-06-20 10:03, Avi Kivity wrote: On 06/20/2011 10:41 AM, Jan Kiszka wrote: Those hopefully shouldn't be required anymore soon - when Jan's patches make it upstream. Jan, how's progress on that front? I can only forward this question: Avi, what are the plans for

Re: [Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-20 Thread Avi Kivity
On 06/20/2011 11:47 AM, Jan Kiszka wrote: On 2011-06-20 10:03, Avi Kivity wrote: On 06/20/2011 10:41 AM, Jan Kiszka wrote: Those hopefully shouldn't be required anymore soon - when Jan's patches make it upstream. Jan, how's progress on that front? I can only forward this question: Avi,

Re: [Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-18 Thread Richard Henderson
On 06/17/2011 04:28 PM, Alexander Graf wrote: +struct kvm_book3e_206_tlb_params params = {}; Hrm - I'm not familiar with that initialization. What exactly does it do? Set the struct contents to 0? Is this properly standardized? Yes and yes. r~

Re: [Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-18 Thread Alexander Graf
On 18.06.2011, at 18:13, Richard Henderson wrote: On 06/17/2011 04:28 PM, Alexander Graf wrote: +struct kvm_book3e_206_tlb_params params = {}; Hrm - I'm not familiar with that initialization. What exactly does it do? Set the struct contents to 0? Is this properly standardized? Yes and

[Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-17 Thread Scott Wood
Share the TLB array with KVM. This allows us to set the initial TLB both on initial boot and reset, is useful for debugging, and could eventually be used to support migration. Signed-off-by: Scott Wood scottw...@freescale.com --- hw/ppce500_mpc8544ds.c |2 + target-ppc/cpu.h |2 +

Re: [Qemu-devel] [PATCH 1/3] kvm: ppc: booke206: use MMU API

2011-06-17 Thread Alexander Graf
On 17.06.2011, at 22:39, Scott Wood wrote: Share the TLB array with KVM. This allows us to set the initial TLB both on initial boot and reset, is useful for debugging, and could eventually be used to support migration. Signed-off-by: Scott Wood scottw...@freescale.com ---