Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754201AbbETUQT (ORCPT ); Wed, 20 May 2015 16:16:19 -0400 Received: from mail3-relais-sop.national.inria.fr ([192.134.164.104]:17431 "EHLO mail3-relais-sop.national.inria.fr" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753727AbbETUQO (ORCPT ); Wed, 20 May 2015 16:16:14 -0400 X-IronPort-AV: E=Sophos;i="5.13,465,1427752800"; d="scan'208";a="125301670" Date: Wed, 20 May 2015 22:15:56 +0200 (CEST) From: Julia Lawall X-X-Sender: jll@hadrien To: Thomas Gleixner cc: Julia Lawall , Jiang Liu , Bjorn Helgaas , Benjamin Herrenschmidt , Ingo Molnar , "H. Peter Anvin" , "Rafael J. Wysocki" , Randy Dunlap , Yinghai Lu , Borislav Petkov , Jason Cooper , Kevin Cernekee , Arnd Bergmann , Marc Zyngier , Konrad Rzeszutek Wilk , Tony Luck , x86@kernel.org, LKML , linux-pci@vger.kernel.org, linux-acpi@vger.kernel.org Subject: Re: [RFC v1 14/25] genirq: Kill the first parameter 'irq' of irq_flow_handler_t In-Reply-To: Message-ID: References: <1432116013-25902-1-git-send-email-jiang.liu@linux.intel.com> <1432116013-25902-15-git-send-email-jiang.liu@linux.intel.com> <555CA834.7080807@linux.intel.com> User-Agent: Alpine 2.10 (DEB 1266 2009-07-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2318 Lines: 66 On Wed, 20 May 2015, Thomas Gleixner wrote: > On Wed, 20 May 2015, Julia Lawall wrote: > > On Wed, 20 May 2015, Thomas Gleixner wrote: > > > The purpose of the script is to: > > > > > > 1) Identify all handlers > > > > > > This can be done by analyzing the arguments of > > > > > > __irq_set_handler() > > > irq_set_handler() > > > irq_set_chained_handler() > > > irq_alloc_generic_chip() > > > irq_alloc_domain_generic_chips() > > > irq_set_chip_and_handler_name() > > > irq_set_chip_and_handler() > > > __irq_set_handler_locked() > > > __irq_set_chip_handler_name_locked() > > > __irq_set_preflow_handler() > > > > > > If the argument is a function local to the file step 2 can > > > take place. > > > > > > If it's a extern function, then the script should at least > > > print out the function names. Those are in most of the cases > > > the generic flow handler functions. > > > > Looks reasonable. > > Good. > > > > 2) Check the handlers whether they use the irq argument > > > > > > For those which do: > > > > > > Check whether they just use the argument instead of a local > > > variable, i.e. the first usage is an assignment. In that > > > case rename the argument to __irq and add a local variable > > > > > > unsigned int irq; > > > > > > If they use the irq argument for some computation or as > > > argument to another function, then you need to rename the > > > argument as well and add: > > > > > > unsigned int irq = irq_get_desc_irq(desc); > > > > Scripting this may be a little complex, because the variable can be used > > in one way in one execution path (eg if branch) and another way (or not at > > all) in another. Let me know if help is needed. > > I feared that, but at least identifying all functions, where the irq > argument is used inside the function itself is really key for such a > massive rework. It's not impossible, but I may need to think a bit how best to do it. julia -- 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/