From mboxrd@z Thu Jan 1 00:00:00 1970 From: ebiederm@xmission.com (Eric W. Biederman) Subject: Re: [PATCH] irq: Start the transition of irq_chip methods taking a desc Date: Sun, 21 Mar 2010 17:32:41 -0700 Message-ID: References: <1267697339-5491-1-git-send-email-yinghai@kernel.org> <1267697339-5491-7-git-send-email-yinghai@kernel.org> <4B900250.8020100@kernel.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from out01.mta.xmission.com ([166.70.13.231]:56964 "EHLO out01.mta.xmission.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753177Ab0CVAcu (ORCPT ); Sun, 21 Mar 2010 20:32:50 -0400 In-Reply-To: (Thomas Gleixner's message of "Sun\, 21 Mar 2010 15\:43\:47 +0100 \(CET\)") Sender: linux-arch-owner@vger.kernel.org List-ID: To: Thomas Gleixner Cc: Yinghai Lu , Ingo Molnar , "H. Peter Anvin" , Andrew Morton , Julia Lawall , Suresh Siddha , LKML , linux-arch@vger.kernel.org Thomas Gleixner writes: > Eric, > > On Sun, 21 Mar 2010, Eric W. Biederman wrote: >> >> With SPARSE_IRQ irq_to_desc becomes an unnecessary lookup operation on >> the fast path of dispatching irqs to their handlers. We can avoid >> this cost by passing an irq_desc pointer instead of using an integer >> irq token to the irq_chip methods. >> >> A single patch to convert all of the architectures is an unreviewable >> 2000+ line patch. A gradual transition scenario with two sets of >> irq_chip methods in irq_chip is an unmanageable mess in kernel/irq. >> >> So instead I define some macros so the generic irq code in kernel/irq/ >> can compile either way based on a boolean Kconfig variable >> CONFIG_CHIP_PARAM_DESC. This allows us to convert one architecture at >> a time, reducing the follow on patches to manageable proportions. It >> is a little bit ugly but it is much better than the alternatives, and >> as soon as we finish the transition we can kill the macros. >> >> I introduce the macros CHIP_ARG, CHIP_VAR, and CHIP_PARAM where >> appropriate. I change a few declarations of irq as int to unsigned >> int. I normalize the variables names in the functions that call >> chip methods to ensure that I have the variables irq and desc present >> allowing CHIP_ARG to work properly. Most importantly none of the irq >> logic changes with this patch. > > I like that approach very much. Is the output binary equivivalent? Depends on what you mean. - If you are concerned about ABI changes, non exist. - If the question is will the compiler generate the same instruction sequences, I don't expect it will. In particular the bug fixes to use unsigned int instead of int should affect the tests used in the for_each_irq_desc. Of less avoid-ability is the change to ack_bad to use desc->irq instead of cached irq value. Since this is the only patch in the entire series that could possibly be binary equivalent I don't find it a particularly interesting test. Eric