Re: [Qemu-devel] State of ARM FIQ in Qemu

[Tim Sander <tim@krieglstein.org>]

Hi Greg

Thanks for your fast reply.
> I am still in the process of getting the security extension portion of the
> GIC patches fully up and running.  By the sounds of your use, it sounds
> like you just want FIQ support not necessarily secure GIC support.  Would
> this be correct?
Yes. More elaborate i am working on a modified cortexa9 versatile express, 
where i added my virtual test hardware.

> I recently sent out an updated set of patches for review that contain GIC
> interrupt grouping and FIQ enablement along with secure extension
> infrastructure.  If interested, you can find the patches here:
> 
> http://lists.nongnu.org/archive/html/qemu-devel/2014-10/msg03921.html
> 
> Alternatively, it sounds like you have access to the Linaro GIT repos, in
> which case you can use the following repo/branch that contains the same
> patches.  It is based on fairly recent upstream bits.
> 
> repo: git://git.linaro.org/people/greg.bellows/qemu.git
> branch: tzqemu_gic_v2
> 
> If you don't need the security extensions, then you shouldn't need to do
> anything to the code to get FIQ support on vexpress-a9/15 or virt machines.
Ok but i think i see a RAZ codepath in qemu when accessing the gic registers 
configuring the interrupt group.

> Please let me know if you have any further questions or issues.
I have the problem that the secure_extn property is not set and i have not 
figured out a way to set these.  The corresponding code is a slighly modified 
vexpress_common_init in hw/arm/vexpress.c.:519.

I guess setting the property would be done by 
qdev_prop_set_bool(dev,"security_extn",TRUE); 
but i fail to find the "dev" from the GIC i could use as argument.

Attached is also a snipped from a debugger run verifing that its indeed 
s->security-extn which is missing.

Best regards
Tim

Breakpoint 3, gic_dist_writeb (opaque=0x555556368a80, offset=136, value=0) at 
hw/intc/arm_gic.c:820
820             } else if (offset >= 0x80) {
(gdb) list
815                     s->enabled = (value & 0x1);
816                     DPRINTF("Distribution %sabled\n", s->enabled ? "En" : 
"Dis");
817                 }
818             } else if (offset < 4) {
819                 /* ignored.  */
820             } else if (offset >= 0x80) {
821                 /* Interrupt Group Registers
822                  *
823                  * For GIC with Security Extn and Non-secure access RAZ/WI
824                  * For GICv1 without Security Extn RAZ/WI
(gdb) n
826                 if (!(s->security_extn && ns_access()) &&
(gdb) n
828                                 || s->revision == 2)) {
(gdb) n
999         gic_update(s);
(gdb) print s->security_extn
$2 = false