Re: [PATCH v3 00/25] risu cleanups and improvements

[Richard Henderson]

On 6/23/20 2:00 AM, Alex Bennée wrote:
> 
> Richard Henderson <richard.henderson@linaro.org> writes:
> 
>> Version 3 changes the --dump option to --fulldump and --diffdump,
>> after an off-hand suggestion by Alex.
>>
>> These are now mode options, similar to --master.  Which means that
>> dumping is an orthogonal apprentice type, which means that we can
>> dump from a socket.  I'm not sure that will be useful as such, but
>> I think it makes main be a bit cleaner.
> 
> Hmm recording traces I ran into a difference, need to track down if its
> a master or apprentice bug (both are native):
> 
>   ./builds/arm64/risu aarch64-all-v8dot0/insn_LDAPR__INC.risu.bin -t 
> aarch64-all-v8dot0/insn_LDAPR__INC.risu.bin.trace
> 
> fails with:
> 
>   loading test image aarch64-all-v8dot0/insn_LDAPR__INC.risu.bin...
>   starting apprentice image at 0xffff8548c000
>   starting image
>   Mismatch reg after 4 checkpoints
>   master reginfo:
>   faulting insn 38bfc1f4
>   .
>   .
>   .
>   mismatch detail (master : apprentice):
>     X15    : 0000ffff9eba41dc vs 0000ffff8548c1dc

Did this work before?  I can't see how.

Your v8dot0 string is suspicious, because LDAPR is a v8.3 instruction.  Are you
sure you're testing what you think you are?


r~