[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me...
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me... |
Date: |
Sun, 23 Sep 2007 10:44:12 +0200 |
Am 23.09.2007 um 01:11 schrieb Aurelien Jarno:
Andreas Färber a écrit :
Am 22.09.2007 um 23:41 schrieb Aurelien Jarno:
Aurelien Jarno a écrit :
Thanks, that's helpful, it looks like the "lduba" instructions
is not
working correctly in your case. Now I have to find why.
I don't really understand, because that is the part of the code (at
least from the op point of view) that is almost unchanged.
Looking at
the code again and again I can't see any problem.
Could you please try to find the values of T1 at the end of
helper_ld_asi (adding a printf or using gdb). That may help us to
understand what is the problem.
With:
printf("helper_ld_asi: T1 = %llx\n", T1); // not sure if that's
correct?
Yes that is correct.
I get these two:
helper_ld_asi: T1 = 5171200000
The correct value here is 51. I don't really understand why this value
is not a 8-bit one, as the memory access is done through ldub_phys().
Could you please verify that helper_ld_asi() is called with the
correct
parameters. This should be 0x20 0x1 0x0.
PS: I don't know if I will be able to get internet access in the
next 4
days.
Take your time, I've already reverted my Q back to the last working
state now that I've figured out that cool -D option.
Actually my printf wasn't correct: For sparc32 T1 is a uint32_t, with
%llx raising a warning I didn't notice before; with %x it does show
the correct value of 51.
printf("helper_ld_asi(%d, %d, %d)\n", asi, size, sign);
...
printf("helper_ld_asi: T1 = %x\n", T1);
starting qemu...
helper_ld_asi(32, 1, 0)
helper_ld_asi: T1 = 51
helper_ld_asi(47, 1, 0)
helper_ld_asi: T1 = 0
So with 0x20==32 that part looks correct.
Andreas
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., (continued)
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Andreas Färber, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Aurelien Jarno, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Andreas Färber, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Aurelien Jarno, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Andreas Färber, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Aurelien Jarno, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Andreas Färber, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Aurelien Jarno, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Andreas Färber, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Aurelien Jarno, 2007/09/22
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me...,
Andreas Färber <=
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Andreas Färber, 2007/09/25
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Aurelien Jarno, 2007/09/26
- Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me..., Andreas Färber, 2007/09/28