Re: [Qemu-devel] [PATCH 3/5] Fix arm_semi_flen_cb for BE32 system mode.

[Paolo Bonzini]

On 03/11/2016 18:30, Julian Brown wrote:
> +#ifdef CONFIG_USER_ONLY
>      size = be32_to_cpu(size);
> +#else
> +    /* If we're running in BE32 system mode, we don't need to do an explicit
> +     * byte swap, because (I think) target memory is already stored in
> +     * byte-swapped format.

Isn't this true also of user-mode (both BE8 and BE32)?

Paolo

> +     */
> +    if (!arm_sctlr_b(env)) {
> +        size = be32_to_cpu(size);
> +    }
> +#endif