Re: [Qemu-arm] [PATCH v4 0/8] Support disabling TCG on ARM

[Peter Maydell]

On Mon, 1 Jul 2019 at 20:49, Philippe Mathieu-Daudé <address@hidden> wrote:
>
> Paolo motived me to salvage this (other!) previous series fromi
> Samuel Ortiz (NEMU project).
>
> v1 cover from Samuel [1]:
>
>   This patchset allows for building and running ARM targets with TCG
>   disabled. It splits the target/arm/helper.c file into logical TCG and
>   non TCG dependent files so that one can build and run QEMU binaries with
>   or without TCG enabled.
>
>   The rationale behind this work comes from the NEMU project where we're
>   trying to only support x86 and ARM 64-bit architectures, without
>   including the TCG code base. We can only do so if we can build and run
>   ARM binaries with TCG disabled.
>
> v2: https://lists.gnu.org/archive/html/qemu-devel/2019-06/msg03271.html
>
> Most of the patches from v1 got adapted, except the "Move all
> interrupt handlers" patch, because Peter disagreed with it.
> See threads:
>  https://lists.gnu.org/archive/html/qemu-devel/2018-11/msg03908.html
>  https://lists.gnu.org/archive/html/qemu-devel/2019-05/msg07304.html
> Anyway this is not a blocking issue, and can be done once this series
> get merged.
>
> v3: http://mid.mail-archive.com/address@hidden
> - Addressed Alex's review comments from v2.
> - added RFC patches to remove (pre-v7, M-profile, R-profile) from KVM-only 
> build
>
> v4:
> - patches 1-12, 14-17, 19-21 accepted, rebased on target-arm.next
> - addressed Peter's comment (described in each patch).
> Based-on: target-arm.next
>
> $ git backport-diff -u v3 -r target-arm.next..v4
> Key:
> [----] : patches are identical
> [####] : number of functional differences between upstream/downstream patch
> [down] : patch is downstream-only
> The flags [FC] indicate (F)unctional and (C)ontextual differences, 
> respectively
>
> 001/8:[0048] [FC] 'target/arm: Move debug routines to debug_helper.c'
> 002/8:[0004] [FC] 'target/arm: Restrict semi-hosting to TCG'
> 003/8:[0027] [FC] 'target/arm/helper: Move M profile routines to m_helper.c'
> 004/8:[----] [--] 'RFC target/arm: Restrict pre-ARMv7 cpus to TCG'
> 005/8:[----] [--] 'RFC target/arm: Do not build pre-ARMv7 cpus when using KVM'
> 006/8:[----] [--] 'RFC target/arm: Restrict R and M profiles to TCG'
> 007/8:[----] [--] 'RFC target/arm: Do not build A/M-profile cpus when using 
> KVM'
> 008/8:[----] [--] 'target/arm: Do not build TCG objects when TCG is off'

I'm going to apply patches 1 and 2 to target-arm.next for rc0 (since
2 is helpful to redhat downstream and taking 1 now reduces the
amount of code-movement patches you'll need to rebase). Patch 3
doesn't compile and 4-8 are rfc.

thanks
-- PMM