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

Reply via email to