[Qemu-devel] [PATCH] target-ppc kvm: save cr register
This adds a missing code to save CR (condition register) via kvm_arch_put_registers(). kvm_arch_get_registers() already has it. Signed-off-by: Alexey Kardashevskiy --- target-ppc/kvm.c |5 + 1 file changed, 5 insertions(+) diff --git a/target-ppc/kvm.c b/target-ppc/kvm.c index 2bbc3b8..c89dd58 100644 --- a/target-ppc/kvm.c +++ b/target-ppc/kvm.c @@ -791,6 +791,11 @@ int kvm_arch_put_registers(CPUState *cs, int level) for (i = 0;i < 32; i++) regs.gpr[i] = env->gpr[i]; +regs.cr = 0; +for (i = 0; i < 8; i++) { +regs.cr |= (env->crf[i] & 15) << (4 * (7 - i)); +} + ret = kvm_vcpu_ioctl(cs, KVM_SET_REGS, ®s); if (ret < 0) return ret; -- 1.7.10.4
Re: [Qemu-devel] [PATCH] target-ppc kvm: save cr register
On Thu, May 30, 2013 at 07:26:10PM +1000, Alexey Kardashevskiy wrote: > This adds a missing code to save CR (condition register) via > kvm_arch_put_registers(). kvm_arch_get_registers() already has it. > > Signed-off-by: Alexey Kardashevskiy Huh. I can't believe that's been broken for so long. I'll put that in my tree. I meant to post my pending patches at the end of last week, but was sidetracked by a minor injury :(. -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson pgpYrIz8Hb5kL.pgp Description: PGP signature
Re: [Qemu-devel] [PATCH] target-ppc kvm: save cr register
On 06/04/2013 10:11 PM, David Gibson wrote: > On Thu, May 30, 2013 at 07:26:10PM +1000, Alexey Kardashevskiy wrote: >> This adds a missing code to save CR (condition register) via >> kvm_arch_put_registers(). kvm_arch_get_registers() already has it. >> >> Signed-off-by: Alexey Kardashevskiy > > Huh. I can't believe that's been broken for so long. I'll put that > in my tree. We probably have another bug which compensates this one but just have not found it yet :-D > I meant to post my pending patches at the end of last week, but was > sidetracked by a minor injury :(. Sigh... What exactly are you going to post? PCI rework? -- Alexey
Re: [Qemu-devel] [PATCH] target-ppc kvm: save cr register
On Wed, Jun 05, 2013 at 12:36:04PM +1000, Alexey Kardashevskiy wrote: > On 06/04/2013 10:11 PM, David Gibson wrote: > > On Thu, May 30, 2013 at 07:26:10PM +1000, Alexey Kardashevskiy wrote: > >> This adds a missing code to save CR (condition register) via > >> kvm_arch_put_registers(). kvm_arch_get_registers() already has it. > >> > >> Signed-off-by: Alexey Kardashevskiy > > > > Huh. I can't believe that's been broken for so long. I'll put that > > in my tree. > > We probably have another bug which compensates this one but just have not > found it yet :-D Yay :/. > > I meant to post my pending patches at the end of last week, but was > > sidetracked by a minor injury :(. > > Sigh... > What exactly are you going to post? PCI rework? Well until now the only thing I had in my ppc-next tree was the change of default machine. I have the PCI stuff too, but that's no PCI related so I was going to post those separately, rather than as a PPC pull request. -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson pgpBtZnVifmI7.pgp Description: PGP signature
Re: [Qemu-devel] [PATCH] target-ppc kvm: save cr register
On 05.06.2013 19:22, David Gibson wrote: >>> I meant to post my pending patches at the end of last week, but was >>> sidetracked by a minor injury :(. >> >> Sigh... >> What exactly are you going to post? PCI rework? > > Well until now the only thing I had in my ppc-next tree was the change > of default machine. btw what is going on with this patch? > I have the PCI stuff too, but that's no PCI > related so I was going to post those separately, rather than as a PPC > pull request. I guess I still want this stuff for VFIO, some bits. -- With best regards Alexey Kardashevskiy -- icq: 52150396
Re: [Qemu-devel] [PATCH] target-ppc kvm: save cr register
On Wed, Jun 05, 2013 at 08:54:38PM +1000, Alexey Kardashevskiy wrote: > On 05.06.2013 19:22, David Gibson wrote: > > >>> I meant to post my pending patches at the end of last week, but was > >>> sidetracked by a minor injury :(. > >> > >> Sigh... > >> What exactly are you going to post? PCI rework? > > > > Well until now the only thing I had in my ppc-next tree was the change > > of default machine. > > btw what is going on with this patch? I've pulled it into my github tree at git://github.com/dgibson/qemu.git, ppc-next branch. I'm still trying to find time to push that out. -- David Gibson| I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson pgpb0rvx41TZT.pgp Description: PGP signature