Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754595AbbGCHA1 (ORCPT ); Fri, 3 Jul 2015 03:00:27 -0400 Received: from mga11.intel.com ([192.55.52.93]:2791 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752283AbbGCHAW convert rfc822-to-8bit (ORCPT ); Fri, 3 Jul 2015 03:00:22 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,398,1432623600"; d="scan'208";a="518242068" From: "Wu, Feng" To: Paolo Bonzini , Eric Auger , "eric.auger@st.com" , "linux-arm-kernel@lists.infradead.org" , "kvmarm@lists.cs.columbia.edu" , "kvm@vger.kernel.org" , "christoffer.dall@linaro.org" , "marc.zyngier@arm.com" , "alex.williamson@redhat.com" , "avi.kivity@gmail.com" , "mtosatti@redhat.com" , "joro@8bytes.org" , "b.reynal@virtualopensystems.com" CC: "linux-kernel@vger.kernel.org" , "patches@linaro.org" , "Wu, Feng" Subject: RE: [RFC 12/17] irq: bypass: Extend skeleton for ARM forwarding control Thread-Topic: [RFC 12/17] irq: bypass: Extend skeleton for ARM forwarding control Thread-Index: AQHQtVzZf/miMN978EWsA+0sAcy8/p3JUE3w Date: Fri, 3 Jul 2015 07:00:14 +0000 Message-ID: References: <1435843047-6327-1-git-send-email-eric.auger@linaro.org> <1435843047-6327-13-git-send-email-eric.auger@linaro.org> <5596311A.3010703@redhat.com> In-Reply-To: <5596311A.3010703@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2019 Lines: 53 > -----Original Message----- > From: Paolo Bonzini [mailto:paolo.bonzini@gmail.com] On Behalf Of Paolo > Bonzini > Sent: Friday, July 03, 2015 2:52 PM > To: Wu, Feng; Eric Auger; eric.auger@st.com; > linux-arm-kernel@lists.infradead.org; kvmarm@lists.cs.columbia.edu; > kvm@vger.kernel.org; christoffer.dall@linaro.org; marc.zyngier@arm.com; > alex.williamson@redhat.com; avi.kivity@gmail.com; mtosatti@redhat.com; > joro@8bytes.org; b.reynal@virtualopensystems.com > Cc: linux-kernel@vger.kernel.org; patches@linaro.org > Subject: Re: [RFC 12/17] irq: bypass: Extend skeleton for ARM forwarding > control > > > > On 03/07/2015 04:43, Wu, Feng wrote: > > > > struct irq_bypass_consumer { > > struct list_head node; > > void *token; > > + unsigned irq; /*got from producer when registered*/ > > void (*add_producer)(struct irq_bypass_producer *, > > struct irq_bypass_consumer *); > > void (*del_producer)(struct irq_bypass_producer *, > > struct irq_bypass_consumer *); > > + void (*update)(struct irq_bypass_consumer *); > > }; > > > > 'update' is used to update the IRTE, while irq is initialized when > > registered, which is used to find the right IRTE. > > Feel free to add "update" in your PI patches. I am not sure if "irq" > belongs here or in the containing struct. You can play with both and > submit the version that looks better to you. Thanks for your review, Paolo. In my understanding, irq comes from the producer side, while gsi belongs to the consumer, so we need to get the irq from the producer somewhere. I am not sure adding irq here is the good way, but what I need is in the 'update' function, I have irq, gsi in hand. :) Thanks, Feng > > Paolo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/