On Fri, Mar 16, 2018 at 05:25:04PM +0000, Peter Maydell wrote: > On 15 March 2018 at 04:18, David Gibson <[email protected]> wrote: > > 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/dgibson/qemu.git tags/ppc-for-2.12-20180315 > > > > for you to fetch changes up to a9ab8cc157054ea6941fb849c78d9e6c515a7730: > > > > target/ppc: fix tlbsync to check privilege level depending on GTSE > > (2018-03-15 11:18:31 +1100) > > > > ---------------------------------------------------------------- > > ppc patch queue for 2018-03-15 > > > > Here's the set of accumulated patches now that we're into soft freeze. > > I've split new functionality into a ppc-for-2.13 branch, so this only > > has bugfixes. Well.. and a couple of simple cleanups to make bugfixes > > easier, some test improvements and a trivial change to make command > > line options more obvious. I think those are all acceptable for soft > > freeze. > > > > Hi -- this looks like it provokes new runtime error warnings from the > clang sanitizer:
Hrm. What options do you need to trip these warnings? Just using
--cc=clang doesn't give them to me, and using --enable-sanitizers
gives my piles of unrelated warnings.
>
> TEST: tests/boot-serial-test... (pid=926)
> /ppc/boot-serial/ppce500: OK
> /ppc/boot-serial/prep: OK
> /ppc/boot-serial/40p: OK
> /ppc/boot-serial/g3beige: OK
> /ppc/boot-serial/mac99: OK
> /ppc/boot-serial/sam460ex:
> /home/petmay01/linaro/qemu-for-merges/target/ppc/translate.c:2979:15:
> runtime error: load of value 142, which is not a valid value for type
> 'bool'
> OK
>
> TEST: tests/boot-serial-test... (pid=1016)
> /ppc64/boot-serial/ppce500: OK
> /ppc64/boot-serial/prep: OK
> /ppc64/boot-serial/40p: OK
> /ppc64/boot-serial/mac99: OK
> /ppc64/boot-serial/pseries: OK
> /ppc64/boot-serial/powernv: OK
> /ppc64/boot-serial/sam460ex:
> /home/petmay01/linaro/qemu-for-merges/target/ppc/translate.c:2979:15:
> runtime error: load of value 85, which is not a valid value for type
> 'bool'
> OK
>
> Looks like you're not initializing ctx->lazy_tlb_flush for all configs:
> if (env->mmu_model == POWERPC_MMU_32B ||
> env->mmu_model == POWERPC_MMU_601 ||
> (env->mmu_model & POWERPC_MMU_64B))
> ctx->lazy_tlb_flush = true;
>
> should perhaps be
> ctx->lazy_tlb_flush =
> env->mmu_model == POWERPC_MMU_32B ||
> env->mmu_model == POWERPC_MMU_601 ||
> (env->mmu_model & POWERPC_MMU_64B);
>
> ?
Uh.. maybe.. except I don't see anything in the series that would be
likely to change that behaviour.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature
