Re: [Qemu-devel] qemu/target-sparc exec.h op.c op_helper.c op_me...

[Andreas Färber]

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