linux-kernel - Re: [PATCH 2/2 v2] kernel: irq: rename irq_set_default_host to irq_domain_set

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150216095350.GH7119@twins.programming.kicks-ass.net>
Date:	Mon, 16 Feb 2015 10:53:50 +0100
From:	Peter Zijlstra <peterz@...radead.org>
To:	Linus Walleij <linus.walleij@...aro.org>
Cc:	linux-kernel@...r.kernel.org,
	Grant Likely <grant.likely@...retlab.ca>,
	Benjamin Herrenschmidt <benh@...nel.crashing.org>,
	Thomas Gleixner <tglx@...utronix.de>
Subject: Re: [PATCH 2/2 v2] kernel: irq: rename irq_set_default_host to
 irq_domain_set_default()

On Fri, Feb 13, 2015 at 08:02:17AM +0800, Linus Walleij wrote:
> This function does not conform to the irq_domain* namespace and
> makes it a disturbing artifact dating back to a time when irq
> domains were referred to as "hosts". Rename it.
> 
> Signed-off-by: Linus Walleij <linus.walleij@...aro.org>
> ---
> ChangeLog v1->v2:
> - Might as well commit changes I have in the working tree so the
>   thing compiles like it did in my tests, d'oh.
> ---
>  arch/arc/kernel/irq.c                            |  2 +-
>  arch/arm/mach-pxa/irq.c                          |  2 +-
>  arch/c6x/kernel/irq.c                            |  2 +-
>  arch/microblaze/kernel/intc.c                    |  2 +-
>  arch/mips/cavium-octeon/octeon-irq.c             |  4 ++--
>  arch/nios2/kernel/irq.c                          |  2 +-
>  arch/powerpc/platforms/52xx/mpc52xx_pic.c        |  2 +-
>  arch/powerpc/platforms/amigaone/setup.c          |  2 +-
>  arch/powerpc/platforms/cell/beat_interrupt.c     |  2 +-
>  arch/powerpc/platforms/cell/interrupt.c          |  2 +-
>  arch/powerpc/platforms/chrp/setup.c              |  2 +-
>  arch/powerpc/platforms/embedded6xx/flipper-pic.c |  2 +-
>  arch/powerpc/platforms/powermac/pic.c            |  2 +-
>  arch/powerpc/platforms/ps3/interrupt.c           |  2 +-
>  arch/powerpc/sysdev/ehv_pic.c                    |  2 +-
>  arch/powerpc/sysdev/ipic.c                       |  2 +-
>  arch/powerpc/sysdev/mpic.c                       |  2 +-
>  arch/powerpc/sysdev/uic.c                        |  2 +-
>  arch/powerpc/sysdev/xics/xics-common.c           |  2 +-
>  arch/powerpc/sysdev/xilinx_intc.c                |  2 +-
>  arch/x86/kernel/apic/io_apic.c                   |  2 +-
>  drivers/irqchip/irq-armada-370-xp.c              |  2 +-
>  drivers/irqchip/irq-clps711x.c                   |  2 +-
>  drivers/irqchip/irq-mmp.c                        |  8 ++++----
>  drivers/irqchip/irq-xtensa-mx.c                  |  4 ++--
>  drivers/irqchip/irq-xtensa-pic.c                 |  4 ++--
>  include/linux/irqdomain.h                        |  2 +-
>  kernel/irq/irqdomain.c                           | 14 +++++++-------
>  28 files changed, 40 insertions(+), 40 deletions(-)

Might it make sense to preserve both functions for one release to avoid
flag day changes and broken compiles?

Mark one __deprecated and point a comment to the new one, then make sure
nobody is still using it by the next release and remove the old
interface?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/