[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] tcg/tci: Remove unnecessary TODO() for INDE
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] tcg/tci: Remove unnecessary TODO() for INDEX_op_bswap16_i64 |
Date: |
Wed, 12 Jul 2017 15:49:12 -0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 |
Hi Jaroslaw,
Thank for reporting this, however this fix has been reviewed:
http://patchwork.ozlabs.org/patch/781982/
On 07/12/2017 10:52 AM, Jaroslaw Pelczar wrote:
Running Ubuntu with systemd on AArch64 Cortex-A53 and qemu is compiled
with
./configure --target-list=aarch64-softmmu --enable-tcg-interpreter
System gives the following log:
Welcome to Ubuntu 16.04.1 LTS!
systemd[1]: Set hostname to <localhost.localdomain>.
TODO (...)/qemu/tcg/tci.c:1049: tcg_qemu_tb_exec()
(...)/qemu/tcg/tci.c:1049: tcg fatal error
Aborted (core dumped)
After investigation it turns out that TCG generates bswap16_i64
opcode, which looks fine in the implementation.
Indeed while correctly implemented, "that code part was simply never
executed, and [Stefan] only removed the TODO() statements from tested
code." as explained Stefan.
Signed-off-by: Jaroslaw Pelczar <address@hidden>
I can add your Signed-off-by there if you mind
---
tcg/tci.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/tcg/tci.c b/tcg/tci.c
index 4bdc645..f39bfb9 100644
--- a/tcg/tci.c
+++ b/tcg/tci.c
@@ -1046,7 +1046,6 @@ uintptr_t tcg_qemu_tb_exec(CPUArchState *env, uint8_t
*tb_ptr)
break;
#if TCG_TARGET_HAS_bswap16_i64
case INDEX_op_bswap16_i64:
- TODO();
t0 = *tb_ptr++;
t1 = tci_read_r16(&tb_ptr);
tci_write_reg64(t0, bswap16(t1));
Regards,
Phil.