From patchwork Fri Jan 24 16:43:38 2014 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Julien Grall X-Patchwork-Id: 23684 Return-Path: X-Original-To: linaro@patches.linaro.org Delivered-To: linaro@patches.linaro.org Received: from mail-vb0-f72.google.com (mail-vb0-f72.google.com [209.85.212.72]) by ip-10-151-82-157.ec2.internal (Postfix) with ESMTPS id DA132203C5 for ; Fri, 24 Jan 2014 16:43:55 +0000 (UTC) Received: by mail-vb0-f72.google.com with SMTP id w20sf5841733vbb.3 for ; Fri, 24 Jan 2014 08:43:55 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:delivered-to:from:to:cc:subject :date:message-id:in-reply-to:references:x-original-sender :x-original-authentication-results:precedence:mailing-list:list-id :list-post:list-help:list-archive:list-unsubscribe; bh=Vd6cA+aHxOLodG7CNsn9ayjeDt8YPiuxhzQ8NJnjpJ8=; b=YO8dc0IYFDlKDeZKwUF+hoB0ViXu4/fPTIUCBI2bvqbkPeHlINFnHOR2Q8vyC3uQyb LCF8Y1W1C54V85hyUE7kyzTdUryd6PSoLkeJ5M6UXiKAXjZvRvuodcLNzISfvVR5At4z KuODbhILD2QCXBDMQ54BeShCC5pDMSP5xnXXIMJYXVuvxyhwMgv3Pu8gLjiEklFTHtgQ VllPGk/PEyJmL49C6bT9QpybzpTzp766KrbJJNYIqP73IYG31K7Ty8vQXU4sawmSOAZv gnLyS0uuHFiFMp0fbZde3LWw4ZinfD01lgN8gUBkFjnqo0IQmCEt7dd5J0PLxJs0LuGt li7w== X-Gm-Message-State: ALoCoQkM1DH1iX1PDjKJJcIIz7lhuIGFUZKPCzXW1gzXvVB6MqEpNv6WF7NYypABONTGgb7tc3Uu X-Received: by 10.236.26.209 with SMTP id c57mr4881666yha.45.1390581834968; Fri, 24 Jan 2014 08:43:54 -0800 (PST) MIME-Version: 1.0 X-BeenThere: patchwork-forward@linaro.org Received: by 10.140.83.73 with SMTP id i67ls677241qgd.97.gmail; Fri, 24 Jan 2014 08:43:54 -0800 (PST) X-Received: by 10.221.66.132 with SMTP id xq4mr80912vcb.57.1390581834836; Fri, 24 Jan 2014 08:43:54 -0800 (PST) Received: from mail-ve0-f172.google.com (mail-ve0-f172.google.com [209.85.128.172]) by mx.google.com with ESMTPS id i1si828963vcl.6.2014.01.24.08.43.54 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 24 Jan 2014 08:43:54 -0800 (PST) Received-SPF: neutral (google.com: 209.85.128.172 is neither permitted nor denied by best guess record for domain of patch+caf_=patchwork-forward=linaro.org@linaro.org) client-ip=209.85.128.172; Received: by mail-ve0-f172.google.com with SMTP id c14so2108503vea.31 for ; Fri, 24 Jan 2014 08:43:54 -0800 (PST) X-Received: by 10.220.225.71 with SMTP id ir7mr93242vcb.50.1390581834739; Fri, 24 Jan 2014 08:43:54 -0800 (PST) X-Forwarded-To: patchwork-forward@linaro.org X-Forwarded-For: patch@linaro.org patchwork-forward@linaro.org Delivered-To: patches@linaro.org Received: by 10.220.174.196 with SMTP id u4csp115783vcz; Fri, 24 Jan 2014 08:43:54 -0800 (PST) X-Received: by 10.14.48.1 with SMTP id u1mr8950251eeb.6.1390581833606; Fri, 24 Jan 2014 08:43:53 -0800 (PST) Received: from mail-ea0-f176.google.com (mail-ea0-f176.google.com [209.85.215.176]) by mx.google.com with ESMTPS id u4si3241245eeo.44.2014.01.24.08.43.53 for (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 24 Jan 2014 08:43:53 -0800 (PST) Received-SPF: neutral (google.com: 209.85.215.176 is neither permitted nor denied by best guess record for domain of julien.grall@linaro.org) client-ip=209.85.215.176; Received: by mail-ea0-f176.google.com with SMTP id h14so1102743eaj.7 for ; Fri, 24 Jan 2014 08:43:53 -0800 (PST) X-Received: by 10.15.41.140 with SMTP id s12mr13562920eev.50.1390581832995; Fri, 24 Jan 2014 08:43:52 -0800 (PST) Received: from belegaer.uk.xensource.com. ([185.25.64.249]) by mx.google.com with ESMTPSA id k41sm5521285eey.0.2014.01.24.08.43.51 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 24 Jan 2014 08:43:52 -0800 (PST) From: Julien Grall To: xen-devel@lists.xenproject.org Cc: patches@linaro.org, ian.campbell@citrix.com, tim@xen.org, stefano.stabellini@citrix.com, Julien Grall Subject: [PATCH for-4.5 4/8] xen/arm: irq: Don't need to have a specific function to route IRQ to Xen Date: Fri, 24 Jan 2014 16:43:38 +0000 Message-Id: <1390581822-32624-5-git-send-email-julien.grall@linaro.org> X-Mailer: git-send-email 1.7.10.4 In-Reply-To: <1390581822-32624-1-git-send-email-julien.grall@linaro.org> References: <1390581822-32624-1-git-send-email-julien.grall@linaro.org> X-Removed-Original-Auth: Dkim didn't pass. X-Original-Sender: julien.grall@linaro.org X-Original-Authentication-Results: mx.google.com; spf=neutral (google.com: 209.85.128.172 is neither permitted nor denied by best guess record for domain of patch+caf_=patchwork-forward=linaro.org@linaro.org) smtp.mail=patch+caf_=patchwork-forward=linaro.org@linaro.org Precedence: list Mailing-list: list patchwork-forward@linaro.org; contact patchwork-forward+owners@linaro.org List-ID: X-Google-Group-Id: 836684582541 List-Post: , List-Help: , List-Archive: List-Unsubscribe: , Actually, when the IRQ is handling by Xen, the setup is done in 2 steps: - Route the IRQ to the current CPU and set priorities - Set up the handler For PPIs, these step are called on every cpus. For SPIs, it's called only on the boot CPU. Divided the setup in two step complicated the code when a new driver is added by Xen (for instance a SMMU driver). Xen can safely route the IRQ when the driver setup the interrupt handler. Signed-off-by: Julien Grall --- xen/arch/arm/gic.c | 67 +++++++++++++++----------------------------- xen/arch/arm/setup.c | 3 -- xen/arch/arm/smpboot.c | 2 -- xen/arch/arm/time.c | 11 -------- xen/include/asm-arm/gic.h | 7 ----- xen/include/asm-arm/time.h | 3 -- 6 files changed, 23 insertions(+), 70 deletions(-) diff --git a/xen/arch/arm/gic.c b/xen/arch/arm/gic.c index d68bde3..58bcba3 100644 --- a/xen/arch/arm/gic.c +++ b/xen/arch/arm/gic.c @@ -254,43 +254,25 @@ static void gic_set_irq_properties(unsigned int irq, bool_t level, spin_unlock(&gic.lock); } -/* Program the GIC to route an interrupt */ +/* Program the GIC to route an interrupt to the host (eg Xen) + * - needs to be called with desc.lock held + */ static int gic_route_irq(unsigned int irq, bool_t level, const cpumask_t *cpu_mask, unsigned int priority) { struct irq_desc *desc = irq_to_desc(irq); - unsigned long flags; ASSERT(priority <= 0xff); /* Only 8 bits of priority */ ASSERT(irq < gic.lines); /* Can't route interrupts that don't exist */ - - if ( desc->action != NULL ) - return -EBUSY; - - /* Disable interrupt */ - desc->handler->shutdown(desc); - - spin_lock_irqsave(&desc->lock, flags); + ASSERT(desc->status & IRQ_DISABLED); desc->handler = &gic_host_irq_type; gic_set_irq_properties(irq, level, cpu_mask, priority); - spin_unlock_irqrestore(&desc->lock, flags); return 0; } -/* Program the GIC to route an interrupt with a dt_irq */ -void gic_route_dt_irq(const struct dt_irq *irq, const cpumask_t *cpu_mask, - unsigned int priority) -{ - bool_t level; - - level = dt_irq_is_level_triggered(irq); - - gic_route_irq(irq->irq, level, cpu_mask, priority); -} - static void __init gic_dist_init(void) { uint32_t type; @@ -538,28 +520,6 @@ void gic_disable_cpu(void) spin_unlock(&gic.lock); } -void gic_route_ppis(void) -{ - /* GIC maintenance */ - gic_route_dt_irq(&gic.maintenance, cpumask_of(smp_processor_id()), 0xa0); - /* Route timer interrupt */ - route_timer_interrupt(); -} - -void gic_route_spis(void) -{ - int seridx; - const struct dt_irq *irq; - - for ( seridx = 0; seridx <= SERHND_IDX; seridx++ ) - { - if ( (irq = serial_dt_irq(seridx)) == NULL ) - continue; - - gic_route_dt_irq(irq, cpumask_of(smp_processor_id()), 0xa0); - } -} - void __init release_irq(unsigned int irq) { struct irq_desc *desc; @@ -601,6 +561,7 @@ int __init setup_dt_irq(const struct dt_irq *irq, struct irqaction *new) int rc; unsigned long flags; struct irq_desc *desc; + bool_t disabled = 0; desc = irq_to_desc(irq->irq); @@ -620,6 +581,24 @@ int __init setup_dt_irq(const struct dt_irq *irq, struct irqaction *new) return -EADDRINUSE; } + disabled = (desc->action == NULL); + + /* First time the IRQ is setup */ + if ( disabled ) + { + bool_t level; + + level = dt_irq_is_level_triggered(irq); + /* It's fine to use smp_processor_id() because: + * For PPI: irq_desc is banked + * For SGI: we don't care for now which CPU will receive the + * interrupt + * TODO: Handle case where SGI is setup on different CPU than + * the targeted CPU and the priority. + */ + gic_route_irq(irq->irq, level, cpumask_of(smp_processor_id()), 0xa0); + } + rc = __setup_irq(desc, irq->irq, new); spin_unlock_irqrestore(&desc->lock, flags); diff --git a/xen/arch/arm/setup.c b/xen/arch/arm/setup.c index 5434784..1f6d713 100644 --- a/xen/arch/arm/setup.c +++ b/xen/arch/arm/setup.c @@ -711,9 +711,6 @@ void __init start_xen(unsigned long boot_phys_offset, init_IRQ(); - gic_route_ppis(); - gic_route_spis(); - init_maintenance_interrupt(); init_timer_interrupt(); diff --git a/xen/arch/arm/smpboot.c b/xen/arch/arm/smpboot.c index c53c765..807e7d3 100644 --- a/xen/arch/arm/smpboot.c +++ b/xen/arch/arm/smpboot.c @@ -287,8 +287,6 @@ void __cpuinit start_secondary(unsigned long boot_phys_offset, init_secondary_IRQ(); - gic_route_ppis(); - init_maintenance_interrupt(); init_timer_interrupt(); diff --git a/xen/arch/arm/time.c b/xen/arch/arm/time.c index 81e3e28..cd16318 100644 --- a/xen/arch/arm/time.c +++ b/xen/arch/arm/time.c @@ -218,17 +218,6 @@ static void vtimer_interrupt(int irq, void *dev_id, struct cpu_user_regs *regs) vgic_vcpu_inject_irq(current, current->arch.virt_timer.irq, 1); } -/* Route timer's IRQ on this CPU */ -void __cpuinit route_timer_interrupt(void) -{ - gic_route_dt_irq(&timer_irq[TIMER_PHYS_NONSECURE_PPI], - cpumask_of(smp_processor_id()), 0xa0); - gic_route_dt_irq(&timer_irq[TIMER_HYP_PPI], - cpumask_of(smp_processor_id()), 0xa0); - gic_route_dt_irq(&timer_irq[TIMER_VIRT_PPI], - cpumask_of(smp_processor_id()), 0xa0); -} - /* Set up the timer interrupt on this CPU */ void __cpuinit init_timer_interrupt(void) { diff --git a/xen/include/asm-arm/gic.h b/xen/include/asm-arm/gic.h index 87f4298..3fd1024 100644 --- a/xen/include/asm-arm/gic.h +++ b/xen/include/asm-arm/gic.h @@ -144,13 +144,6 @@ extern void vgic_vcpu_inject_irq(struct vcpu *v, unsigned int irq,int virtual); extern void vgic_clear_pending_irqs(struct vcpu *v); extern struct pending_irq *irq_to_pending(struct vcpu *v, unsigned int irq); -/* Program the GIC to route an interrupt with a dt_irq */ -extern void gic_route_dt_irq(const struct dt_irq *irq, - const cpumask_t *cpu_mask, - unsigned int priority); -extern void gic_route_ppis(void); -extern void gic_route_spis(void); - extern void gic_inject(void); extern void gic_clear_pending_irqs(struct vcpu *v); extern int gic_events_need_delivery(void); diff --git a/xen/include/asm-arm/time.h b/xen/include/asm-arm/time.h index 9d302d3..eaa96bc 100644 --- a/xen/include/asm-arm/time.h +++ b/xen/include/asm-arm/time.h @@ -28,9 +28,6 @@ enum timer_ppi /* Get one of the timer IRQ description */ const struct dt_irq* timer_dt_irq(enum timer_ppi ppi); -/* Route timer's IRQ on this CPU */ -extern void __cpuinit route_timer_interrupt(void); - /* Set up the timer interrupt on this CPU */ extern void __cpuinit init_timer_interrupt(void);