On 13 March 2018 at 18:53, Max Filippov <jcmvb...@gmail.com> wrote: > Hi Peter, > > please pull the following series that adds linux-user support for > target/xtensa. > There's a trivial merge conflict with linux-user pull request from > Laurent Vivier. > > Changes since v6 patch series: > - rebase to the current master. > > The following changes since commit 026aaf47c02b79036feb830206cfebb2a726510d: > > Merge remote-tracking branch > 'remotes/ehabkost/tags/python-next-pull-request' into staging (2018-03-13 > 16:26:44 +0000) > > are available in the git repository at: > > git://github.com/OSLL/qemu-xtensa.git tags/20180313-xtensa > > for you to fetch changes up to be00d35eab124bb4e4e393ac2b0a451e065b7041: > > MAINTAINERS: fix W: address for xtensa (2018-03-13 11:30:23 -0700) >
Hi. This fails to build on 32-bit: In file included from /home/peter.maydell/qemu/include/exec/exec-all.h:41:0, from /home/peter.maydell/qemu/target/xtensa/helper.c:30: /home/peter.maydell/qemu/target/xtensa/helper.c: In function 'xtensa_cpu_handle_mmu_fault': /home/peter.maydell/qemu/target/xtensa/helper.c:331:32: error: format '%lx' expects argument of type 'long unsigned int', but argument 4 has type 'vaddr {aka long long unsigned int}' [-Werror=format=] qemu_log_mask(CPU_LOG_INT, "%s: rw = %d, address = 0x%08lx, size = %d\n", ^ /home/peter.maydell/qemu/include/qemu/log.h:84:22: note: in definition of macro 'qemu_log_mask' qemu_log(FMT, ## __VA_ARGS__); \ ^ 'vaddr' type is uint64_t, so you need PRIx64 here. thanks -- PMM