linux-kernel - Re: [PATCH] IA64, PPC, SPARC: minor irq handler cleanups

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: <4721B817.7090602@garzik.org>
Date:	Fri, 26 Oct 2007 05:49:11 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	Kumar Gala <galak@...nel.crashing.org>
CC:	LKML <linux-kernel@...r.kernel.org>, akpm@...ux-foundation.org,
	davem@...emloft.net, tony.luck@...el.com,
	linux-ia64@...r.kernel.org, paulus@...ba.org,
	benh@...nel.crashing.org
Subject: Re: [PATCH] IA64, PPC, SPARC: minor irq handler cleanups

Kumar Gala wrote:
> 
> On Oct 26, 2007, at 4:40 AM, Jeff Garzik wrote:
> 
>> ia64/sn/kernel/huberror.c:
>>     - remove pointless void* cast
>>     - add KERN_xxx prefix
>>
>> ia64/sn/pci/tioce_provider.c: start functions at column zero
>>
>> ppc/8xx_io/fec.c: kill prototype, remove extra whitespace
>>
>> ppc/platforms/85xx/*
>>     'irq' argument in irq handler is used purely as a temporary
>>     variable.  Update the function to reflect this usage, changing
>>     the first arg's name from 'irq' to 'dummy'
>>
>> ppc/platforms/sbc82xx.c: ditto
>>
>> sparc/kernel/time.c: mark timer_interrupt() static (from DaveM)
>>
>> Signed-off-by: Jeff Garzik <jgarzik@...hat.com>
>> ---
>>  arch/ia64/sn/kernel/huberror.c               |    4 ++--
>>  arch/ia64/sn/pci/tioce_provider.c            |    6 ++++--
>>  arch/ppc/8xx_io/fec.c                        |    3 +--
>>  arch/ppc/platforms/85xx/mpc8560_ads.c        |    4 +++-
>>  arch/ppc/platforms/85xx/mpc85xx_cds_common.c |    4 +++-
>>  arch/ppc/platforms/85xx/stx_gp3.c            |    4 +++-
>>  arch/ppc/platforms/85xx/tqm85xx.c            |    4 +++-
>>  arch/ppc/platforms/sbc82xx.c                 |    4 +++-
>>  arch/sparc/kernel/time.c                     |    2 +-
>>  9 files changed, 23 insertions(+), 12 deletions(-)
> 
> I haven't been following this with detail.  Is the intent that this 
> would be for 2.6.24 or .25?

These patches are all cleanups and minor fixes I found while iterating 
through each interrupt handler in the kernel, in pursuit of a related 
project.

So they can stand by themselves, and can integrate into whatever flow 
maintainers find most comfortable.

I'm not in any rush, but neither would I like them to sit forever...

	Jeff


-
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/