[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH] tci: Fix qemu-alpha on 32 bit ho
Re: [Qemu-trivial] [Qemu-devel] [PATCH] tci: Fix qemu-alpha on 32 bit hosts (wrong assertions)
Thu, 12 Sep 2013 20:57:57 +0200
Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130803 Thunderbird/17.0.8
Am 12.09.2013 20:39, schrieb Richard Henderson:
> On 09/12/2013 11:17 AM, Stefan Weil wrote:
>> @@ -1093,7 +1093,6 @@ uintptr_t tcg_qemu_tb_exec(CPUArchState *env, uint8_t
>> tmp8 = helper_ldb_mmu(env, taddr, tci_read_i(&tb_ptr));
>> host_addr = (tcg_target_ulong)taddr;
>> - assert(taddr == host_addr);
>> tmp8 = *(uint8_t *)(host_addr + GUEST_BASE);
> I noticed first that g2h would be better than fiddling GUEST_BASE
> by hand. But then I noticed failure to handle endianness and
> failure to handle unaligned accesses too.
> You should be using
> tmp8 = ldub(taddr);
> et al. See include/exec/cpu-all.h, beginning line 253.
Thanks for your hint. Yes, as you can see from tcg/tci/README,the test
matrix of TCI
did not include big endian hosts up to now. Testing on an emulated big
system is terribly slow, and I have no access to real big endian
But I think that such changes are independent of this patchwhich can be
applied as it is.