From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758623AbaGAQg6 (ORCPT ); Tue, 1 Jul 2014 12:36:58 -0400 Received: from smtp02.citrix.com ([66.165.176.63]:5814 "EHLO SMTP02.CITRIX.COM" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758522AbaGAQfH (ORCPT ); Tue, 1 Jul 2014 12:35:07 -0400 X-IronPort-AV: E=Sophos;i="5.01,582,1400025600"; d="scan'208";a="149125982" Date: Tue, 1 Jul 2014 17:34:28 +0100 From: Stefano Stabellini X-X-Sender: sstabellini@kaball.uk.xensource.com To: Marc Zyngier CC: Stefano Stabellini , Anup Patel , "kvmarm@lists.cs.columbia.edu" , linux-arm-kernel , "linux-kernel@vger.kernel.org List" , Catalin Marinas , Thomas Gleixner , Will Deacon , Christoffer Dall , Eric Auger , Ian Campbell Subject: Re: [RFC PATCH 3/9] irqchip: GIC: Convert to EOImode == 1 In-Reply-To: <53B27049.10302@arm.com> Message-ID: References: <1403688530-23273-1-git-send-email-marc.zyngier@arm.com> <1403688530-23273-4-git-send-email-marc.zyngier@arm.com> <53B27049.10302@arm.com> User-Agent: Alpine 2.02 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: text/plain; charset="US-ASCII" X-DLP: MIA1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, 1 Jul 2014, Marc Zyngier wrote: > Hi Stefano, > > On 30/06/14 20:09, Stefano Stabellini wrote: > > On Wed, 25 Jun 2014, Anup Patel wrote: > >> Hi Marc, > >> > >> On Wed, Jun 25, 2014 at 2:58 PM, Marc Zyngier wrote: > >>> So far, GICv2 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 the EOI register 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 the DIR marks the interrupt as inactive, meaning it can > >>> now be taken again. > >>> > >>> We only enable this feature when booted in HYP mode. Also, as most device > >>> trees are broken (they report the CPU interface size to be 4kB, while > >>> the GICv2 CPU interface size is 8kB), output a warning if we're booted > >>> in HYP mode, and disable the feature. > >>> > >>> Signed-off-by: Marc Zyngier > >>> --- > >>> drivers/irqchip/irq-gic.c | 61 +++++++++++++++++++++++++++++++++++++---- > >>> include/linux/irqchip/arm-gic.h | 4 +++ > >>> 2 files changed, 59 insertions(+), 6 deletions(-) > >>> > >>> diff --git a/drivers/irqchip/irq-gic.c b/drivers/irqchip/irq-gic.c > >>> index 508b815..9295bf2 100644 > >>> --- a/drivers/irqchip/irq-gic.c > >>> +++ b/drivers/irqchip/irq-gic.c > >>> @@ -45,6 +45,7 @@ > >>> #include > >>> #include > >>> #include > >>> +#include > >>> > >>> #include "irq-gic-common.h" > >>> #include "irqchip.h" > >>> @@ -94,6 +95,10 @@ struct irq_chip gic_arch_extn = { > >>> .irq_set_wake = NULL, > >>> }; > >>> > >>> +static struct irq_chip *gic_chip; > >>> + > >>> +static bool supports_deactivate = false; > >>> + > >>> #ifndef MAX_GIC_NR > >>> #define MAX_GIC_NR 1 > >>> #endif > >>> @@ -185,6 +190,12 @@ static void gic_eoi_irq(struct irq_data *d) > >>> writel_relaxed(gic_irq(d), gic_cpu_base(d) + GIC_CPU_EOI); > >>> } > >>> > >>> +static void gic_eoi_dir_irq(struct irq_data *d) > >>> +{ > >>> + gic_eoi_irq(d); > >>> + writel_relaxed(gic_irq(d), gic_cpu_base(d) + GIC_CPU_DEACTIVATE); > >>> +} > > > > Would it be better if you moved the gic_eoi_irq call earlier? Maybe > > somewhere in gic_handle_irq? > > I'm not sure I see what we'd gain by doing so. Can you elaborate? You would be dropping the priority earlier. It would be beneficial if Linux was running the interrupt handlers with interrupts enabled, but I realize now that it is not the case.