On Fri, Sep 04, 2015 at 05:22:56PM +0100, Peter Maydell wrote: > This patchset enables the TZ support in the GIC for the systems > where we enable TZ support in the CPU. In practice that means > just the "virt" and "vexpress" boards, since all the others > disable the CPU TZ support. > > There are no changes since v3, I've just rebased this all on > top of current target-arm.next (which will be in master early > next week as I've just sent out the pullreq). > > These have already been reviewed by Peter C, but when I spoke > to Edgar at KVM Forum he said he hadn't been able to sort out > which tree these would apply to, so I figured I'd better resend > them so there's a chance for further review/testing if desired. >
Thanks Peter, Reviewed-by: Edgar E. Iglesias <edgar.igles...@xilinx.com> Tested-by: Edgar E. Iglesias <edgar.igles...@xilinx.com> Cheers, Edgar > thanks > -- PMM > > Changes since v2: > * rebased onto target-arm.next > Changes since v1: > * New patch which switches the default for the 'virt' board from > "enable TZ" to "disable TZ". The UEFI blob can't handle TZ being > fully enabled, so I had a choice of breaking it or breaking code > which assumes TZ. As far as I know only the TZ test suite falls > in the latter category. -machine secure=on will give you the > old behaviour back. > * rather than the property on the GIC, we take the approach Peter C > suggested of defining an interface for devices to implement if > they need to do firmware-equivalent setup. The API is a little > different from Peter C's RFC patch, but the principle is the same. > > > > Peter Crosthwaite (1): > qom: Add recursive version of object_child_for_each > > Peter Maydell (5): > hw/arm: new interface for devices which need to behave differently for > kernel boot > hw/intc/arm_gic_common: Configure IRQs as NS if doing direct NS kernel > boot > hw/cpu/{a15mpcore,a9mpcore}: enable TrustZone in GIC if it is enabled > in CPUs > hw/arm/virt: Default to not providing TrustZone support > hw/arm/virt: Enable TZ extensions on the GIC if we are using them > > hw/arm/boot.c | 34 +++++++++++++++++++++++++++ > hw/arm/virt.c | 14 +++++++---- > hw/cpu/a15mpcore.c | 13 ++++++++++ > hw/cpu/a9mpcore.c | 11 +++++++++ > hw/intc/arm_gic_common.c | 51 > +++++++++++++++++++++++++++++++++++++--- > include/hw/arm/linux-boot-if.h | 43 +++++++++++++++++++++++++++++++++ > include/hw/intc/arm_gic_common.h | 1 + > include/qom/object.h | 15 ++++++++++++ > qom/object.c | 25 +++++++++++++++++--- > 9 files changed, 197 insertions(+), 10 deletions(-) > create mode 100644 include/hw/arm/linux-boot-if.h > > -- > 1.9.1 >