Re: [PATCH v3 1/4] irqchip: GICv3: Convert to EOImode == 1
On 26/08/15 09:54, Eric Auger wrote: Hi Marc, On 08/25/2015 01:53 PM, Marc Zyngier wrote: So far, GICv3 has been used in with EOImode == 0. The effect of this mode is to perform the priority drop and the deactivation of the interrupt at the same time. While this works perfectly for Linux (we only have a single priority), it causes issues when an interrupt is forwarded to a guest, and when we want the guest to perform the EOI itself. For this case, the GIC architecture provides EOImode == 1, where: - A write to ICC_EOIR1_EL1 drops the priority of the interrupt and leaves it active. Other interrupts at the same priority level can now be taken, but the active interrupt cannot be taken again - A write to ICC_DIR_EL1 marks the interrupt as inactive, meaning it can now be taken again. This patch converts the driver to be able to use this new mode, depending on whether or not the kernel can behave as a hypervisor. No feature change. Reviewed-by: Eric Auger eric.au...@linaro.org Signed-off-by: Marc Zyngier marc.zyng...@arm.com --- drivers/irqchip/irq-gic-v3.c | 39 ++ include/linux/irqchip/arm-gic-v3.h | 9 + 2 files changed, 44 insertions(+), 4 deletions(-) diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c index c52f7ba..addd2ee 100644 --- a/drivers/irqchip/irq-gic-v3.c +++ b/drivers/irqchip/irq-gic-v3.c @@ -30,6 +30,7 @@ #include asm/cputype.h #include asm/exception.h #include asm/smp_plat.h +#include asm/virt.h #include irq-gic-common.h #include irqchip.h @@ -50,6 +51,7 @@ struct gic_chip_data { }; static struct gic_chip_data gic_data __read_mostly; +static struct static_key supports_deactivate = STATIC_KEY_INIT_TRUE; in http://www.spinics.net/lists/arm-kernel/msg439813.html, Catalin reported the direct usage of static_key was deprecated. Sorry I learned that after my last review. see https://lkml.org/lkml/2015/8/3/539 Hi Eric, I'm aware of Peter's work to rework static keys, but that code is not merged yet (though I believe it is on its way to 4.3). I have an additional patch that repaints the code to use static_key_likely() all over the place, but I keep it separate for the time being. If this series makes it in for 4.3, I'll post the repainting patch after -rc1. Otherwise, I'll just update it in my tree. I'm about to post v4 anyway, and I'll mention this in the cover letter. Thanks, M. -- Jazz is not dead. It just smells funny... -- To unsubscribe from this list: send the line unsubscribe kvm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH v3 1/4] irqchip: GICv3: Convert to EOImode == 1
Hi Marc, On 08/25/2015 01:53 PM, Marc Zyngier wrote: So far, GICv3 has been used in with EOImode == 0. The effect of this mode is to perform the priority drop and the deactivation of the interrupt at the same time. While this works perfectly for Linux (we only have a single priority), it causes issues when an interrupt is forwarded to a guest, and when we want the guest to perform the EOI itself. For this case, the GIC architecture provides EOImode == 1, where: - A write to ICC_EOIR1_EL1 drops the priority of the interrupt and leaves it active. Other interrupts at the same priority level can now be taken, but the active interrupt cannot be taken again - A write to ICC_DIR_EL1 marks the interrupt as inactive, meaning it can now be taken again. This patch converts the driver to be able to use this new mode, depending on whether or not the kernel can behave as a hypervisor. No feature change. Reviewed-by: Eric Auger eric.au...@linaro.org Signed-off-by: Marc Zyngier marc.zyng...@arm.com --- drivers/irqchip/irq-gic-v3.c | 39 ++ include/linux/irqchip/arm-gic-v3.h | 9 + 2 files changed, 44 insertions(+), 4 deletions(-) diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c index c52f7ba..addd2ee 100644 --- a/drivers/irqchip/irq-gic-v3.c +++ b/drivers/irqchip/irq-gic-v3.c @@ -30,6 +30,7 @@ #include asm/cputype.h #include asm/exception.h #include asm/smp_plat.h +#include asm/virt.h #include irq-gic-common.h #include irqchip.h @@ -50,6 +51,7 @@ struct gic_chip_data { }; static struct gic_chip_data gic_data __read_mostly; +static struct static_key supports_deactivate = STATIC_KEY_INIT_TRUE; in http://www.spinics.net/lists/arm-kernel/msg439813.html, Catalin reported the direct usage of static_key was deprecated. Sorry I learned that after my last review. see https://lkml.org/lkml/2015/8/3/539 Best Regards Eric #define gic_data_rdist() (this_cpu_ptr(gic_data.rdists.rdist)) #define gic_data_rdist_rd_base() (gic_data_rdist()-rd_base) @@ -293,7 +295,14 @@ static int gic_irq_get_irqchip_state(struct irq_data *d, static void gic_eoi_irq(struct irq_data *d) { - gic_write_eoir(gic_irq(d)); + if (static_key_true(supports_deactivate)) { + /* No need to deactivate an LPI */ + if (gic_irq(d) = 8192) + return; + gic_write_dir(gic_irq(d)); + } else { + gic_write_eoir(gic_irq(d)); + } } static int gic_set_type(struct irq_data *d, unsigned int type) @@ -343,15 +352,26 @@ static asmlinkage void __exception_irq_entry gic_handle_irq(struct pt_regs *regs if (likely(irqnr 15 irqnr 1020) || irqnr = 8192) { int err; + + if (static_key_true(supports_deactivate)) + gic_write_eoir(irqnr); + err = handle_domain_irq(gic_data.domain, irqnr, regs); if (err) { WARN_ONCE(true, Unexpected interrupt received!\n); - gic_write_eoir(irqnr); + if (static_key_true(supports_deactivate)) { + if (irqnr 8192) + gic_write_dir(irqnr); + } else { + gic_write_eoir(irqnr); + } } continue; } if (irqnr 16) { gic_write_eoir(irqnr); + if (static_key_true(supports_deactivate)) + gic_write_dir(irqnr); #ifdef CONFIG_SMP handle_IPI(irqnr, regs); #else @@ -451,8 +471,13 @@ static void gic_cpu_sys_reg_init(void) /* Set priority mask register */ gic_write_pmr(DEFAULT_PMR_VALUE); - /* EOI deactivates interrupt too (mode 0) */ - gic_write_ctlr(ICC_CTLR_EL1_EOImode_drop_dir); + if (static_key_true(supports_deactivate)) { + /* EOI drops priority only (mode 1) */ + gic_write_ctlr(ICC_CTLR_EL1_EOImode_drop); + } else { + /* EOI deactivates interrupt too (mode 0) */ + gic_write_ctlr(ICC_CTLR_EL1_EOImode_drop_dir); + } /* ... and let's hit the road... */ gic_write_grpen1(1); @@ -820,6 +845,12 @@ static int __init gic_of_init(struct device_node *node, struct device_node *pare if (of_property_read_u64(node, redistributor-stride, redist_stride)) redist_stride = 0; + if (!is_hyp_mode_available()) + static_key_slow_dec(supports_deactivate); + + if (static_key_true(supports_deactivate)) + pr_info(GIC: Using split EOI/Deactivate mode\n); + gic_data.dist_base =
[PATCH v3 1/4] irqchip: GICv3: Convert to EOImode == 1
So far, GICv3 has been used in with EOImode == 0. The effect of this mode is to perform the priority drop and the deactivation of the interrupt at the same time. While this works perfectly for Linux (we only have a single priority), it causes issues when an interrupt is forwarded to a guest, and when we want the guest to perform the EOI itself. For this case, the GIC architecture provides EOImode == 1, where: - A write to ICC_EOIR1_EL1 drops the priority of the interrupt and leaves it active. Other interrupts at the same priority level can now be taken, but the active interrupt cannot be taken again - A write to ICC_DIR_EL1 marks the interrupt as inactive, meaning it can now be taken again. This patch converts the driver to be able to use this new mode, depending on whether or not the kernel can behave as a hypervisor. No feature change. Reviewed-by: Eric Auger eric.au...@linaro.org Signed-off-by: Marc Zyngier marc.zyng...@arm.com --- drivers/irqchip/irq-gic-v3.c | 39 ++ include/linux/irqchip/arm-gic-v3.h | 9 + 2 files changed, 44 insertions(+), 4 deletions(-) diff --git a/drivers/irqchip/irq-gic-v3.c b/drivers/irqchip/irq-gic-v3.c index c52f7ba..addd2ee 100644 --- a/drivers/irqchip/irq-gic-v3.c +++ b/drivers/irqchip/irq-gic-v3.c @@ -30,6 +30,7 @@ #include asm/cputype.h #include asm/exception.h #include asm/smp_plat.h +#include asm/virt.h #include irq-gic-common.h #include irqchip.h @@ -50,6 +51,7 @@ struct gic_chip_data { }; static struct gic_chip_data gic_data __read_mostly; +static struct static_key supports_deactivate = STATIC_KEY_INIT_TRUE; #define gic_data_rdist() (this_cpu_ptr(gic_data.rdists.rdist)) #define gic_data_rdist_rd_base() (gic_data_rdist()-rd_base) @@ -293,7 +295,14 @@ static int gic_irq_get_irqchip_state(struct irq_data *d, static void gic_eoi_irq(struct irq_data *d) { - gic_write_eoir(gic_irq(d)); + if (static_key_true(supports_deactivate)) { + /* No need to deactivate an LPI */ + if (gic_irq(d) = 8192) + return; + gic_write_dir(gic_irq(d)); + } else { + gic_write_eoir(gic_irq(d)); + } } static int gic_set_type(struct irq_data *d, unsigned int type) @@ -343,15 +352,26 @@ static asmlinkage void __exception_irq_entry gic_handle_irq(struct pt_regs *regs if (likely(irqnr 15 irqnr 1020) || irqnr = 8192) { int err; + + if (static_key_true(supports_deactivate)) + gic_write_eoir(irqnr); + err = handle_domain_irq(gic_data.domain, irqnr, regs); if (err) { WARN_ONCE(true, Unexpected interrupt received!\n); - gic_write_eoir(irqnr); + if (static_key_true(supports_deactivate)) { + if (irqnr 8192) + gic_write_dir(irqnr); + } else { + gic_write_eoir(irqnr); + } } continue; } if (irqnr 16) { gic_write_eoir(irqnr); + if (static_key_true(supports_deactivate)) + gic_write_dir(irqnr); #ifdef CONFIG_SMP handle_IPI(irqnr, regs); #else @@ -451,8 +471,13 @@ static void gic_cpu_sys_reg_init(void) /* Set priority mask register */ gic_write_pmr(DEFAULT_PMR_VALUE); - /* EOI deactivates interrupt too (mode 0) */ - gic_write_ctlr(ICC_CTLR_EL1_EOImode_drop_dir); + if (static_key_true(supports_deactivate)) { + /* EOI drops priority only (mode 1) */ + gic_write_ctlr(ICC_CTLR_EL1_EOImode_drop); + } else { + /* EOI deactivates interrupt too (mode 0) */ + gic_write_ctlr(ICC_CTLR_EL1_EOImode_drop_dir); + } /* ... and let's hit the road... */ gic_write_grpen1(1); @@ -820,6 +845,12 @@ static int __init gic_of_init(struct device_node *node, struct device_node *pare if (of_property_read_u64(node, redistributor-stride, redist_stride)) redist_stride = 0; + if (!is_hyp_mode_available()) + static_key_slow_dec(supports_deactivate); + + if (static_key_true(supports_deactivate)) + pr_info(GIC: Using split EOI/Deactivate mode\n); + gic_data.dist_base = dist_base; gic_data.redist_regions = rdist_regs; gic_data.nr_redist_regions = nr_redist_regions; diff --git a/include/linux/irqchip/arm-gic-v3.h b/include/linux/irqchip/arm-gic-v3.h index ffbc034..bc98832 100644 --- a/include/linux/irqchip/arm-gic-v3.h +++