[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Minios-devel] [UNIKRAFT PATCHv5 3/6] plat/common: Implement gic-v2 library for Arm



Hello,

On 7/10/19 6:52 AM, Jianyong Wu (Arm Technology China) wrote:
Hi julien,

-----Original Message-----
From: Julien Grall <julien.grall@xxxxxxx>
Sent: Wednesday, July 10, 2019 1:46 AM
To: Justin He (Arm Technology China) <Justin.He@xxxxxxx>; minios-
devel@xxxxxxxxxxxxxxxxxxxx; Simon Kuenzer <simon.kuenzer@xxxxxxxxx>;
Sharan.Santhanam@xxxxxxxxx
Cc: Florian Schmidt <florian.schmidt@xxxxxxxxx>; Felipe Huici
<felipe.huici@xxxxxxxxx>; yuri.volchkov@xxxxxxxxx; Kaly Xin (Arm
Technology China) <Kaly.Xin@xxxxxxx>; Jianyong Wu (Arm Technology
China) <Jianyong.Wu@xxxxxxx>; Wei Chen (Arm Technology China)
<Wei.Chen@xxxxxxx>
Subject: Re: [UNIKRAFT PATCHv5 3/6] plat/common: Implement gic-v2 library
for Arm

Hi,

On 6/28/19 8:09 AM, Jia He wrote:
+static void gic_init_dist(void)
+{
+uint32_t val, cpuif_number, irq_number;
+uint32_t i;
+
+/* Turn down distributor */
+gic_disable_dist();
+
+/* Get GIC CPU interface */
+val = read_gicd32(GICD_TYPER);
+cpuif_number = GICD_TYPER_CPUI_NUM(val);
+if (cpuif_number > GIC_MAX_CPUIF)
+cpuif_number = GIC_MAX_CPUIF;
+uk_pr_info("GICv2 Max CPU interface:%d\n", cpuif_number);
+
+/* Get the maximum number of interrupts that the GIC supports */
+irq_number = GICD_TYPER_LINE_NUM(val);
+if (irq_number > GIC_MAX_IRQ)
+irq_number = GIC_MAX_IRQ;
+uk_pr_info("GICv2 Max interrupt lines:%d\n", irq_number);
+/*
+ * Set all SPI interrupts targets to all CPU.
+ */
+for (i = GIC_SPI_BASE; i < irq_number; i += GICD_I_PER_ITARGETSRn)
+write_gicd32(GICD_ITARGETSR(i), GICD_ITARGETSR_DEF);
+
+/*
+ * Set all SPI interrupts type to be level triggered
+ */
+for (i = GIC_SPI_BASE; i < irq_number; i += GICD_I_PER_ICFGRn)
+write_gicd32(GICD_ICFGR(i), GICD_ICFGR_DEF_TYPE);

Why do you initialize SPI interrupts type to level-triggered? What if they are
edge?

I get this from xen, you can refer to [1] line 367 to get it.
[1] https://fossies.org/linux/xen/xen/arch/arm/gic-v2.c

Seems like you are using GPL code. Careful taking code from elsewhere especially with core libraries.

Thanks
Jianyong wu

+
+/*
+ * Set all SPI priority to a default value.
+ */
+for (i = GIC_SPI_BASE; i < irq_number; i += GICD_I_PER_IPRIORITYn)
+write_gicd32(GICD_IPRIORITYR(i), GICD_IPRIORITY_DEF);
+
+/*
+ * Deactivate and disable all SPIs.
+ */
+for (i = GIC_SPI_BASE; i < irq_number; i += GICD_I_PER_ICACTIVERn)
{
+write_gicd32(GICD_ICACTIVER(i), GICD_DEF_ICACTIVERn);
+write_gicd32(GICD_ICENABLER(i), GICD_DEF_ICENABLERn);
+}
+
+/* turn on distributor */
+gic_enable_dist();
+}
+
+static void gic_init_cpuif(void)
+{
+uint32_t i;
+/*
+ * set priority mask to the lowest priority to let all irq
+ * visible to cpu interface
+ */
+gic_set_threshold_priority(GICC_PMR_PRIO_MAX);
+
+/* set PPI and SGI to level triggered */
+for (i = 0; i < GIC_SPI_BASE; i += GICD_I_PER_ICFGRn)
+write_gicd32(GICD_ICFGR(i), GICD_ICFGR_DEF_TYPE);

Ditto.

Cheers,

--
Julien Grall
IMPORTANT NOTICE: The contents of this email and any attachments are 
confidential and may also be privileged. If you are not the intended recipient, 
please notify the sender immediately and do not disclose the contents to any 
other person, use it for any purpose, or store or copy the information in any 
medium. Thank you.


_______________________________________________
Minios-devel mailing list
Minios-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/minios-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.