linux-kernel - Re: [PATCH 03/15] [BLACKFIN] 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 for Android: free password hash cracker in your pocket

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

Message-ID: <386072610804212027i58895642n5320b96376c2732@mail.gmail.com>
Date:	Tue, 22 Apr 2008 11:27:00 +0800
From:	"Bryan Wu" <cooloney@...nel.org>
To:	"Jeff Garzik" <jeff@...zik.org>
Cc:	"Andrew Morton" <akpm@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 03/15] [BLACKFIN] minor irq handler cleanups

On Sat, Apr 19, 2008 at 7:22 AM, Jeff Garzik <jeff@...zik.org> wrote:
> - time_sched_init() should use standard irq_handler_t
>
>  - sys_timer0_int_handler() should not take 'regs' third argument
>
>  - remove pointless cast from void*
>
>  This change's main purpose is to prepare for the patchset in
>  jgarzik/misc-2.6.git#irq-remove, that explores removal of the
>  never-used 'irq' argument in each interrupt handler.
>
>  Signed-off-by: Jeff Garzik <jgarzik@...hat.com>
>  ---
>   arch/blackfin/kernel/time.c        |    5 ++---
>   arch/blackfin/oprofile/timer_int.c |    5 ++---
>   2 files changed, 4 insertions(+), 6 deletions(-)
>
>  diff --git a/arch/blackfin/kernel/time.c b/arch/blackfin/kernel/time.c
>  index 9bdc8f9..715b394 100644
>  --- a/arch/blackfin/kernel/time.c
>  +++ b/arch/blackfin/kernel/time.c
>  @@ -39,8 +39,7 @@
>   /* This is an NTP setting */
>   #define        TICK_SIZE (tick_nsec / 1000)
>
>  -static void time_sched_init(irqreturn_t(*timer_routine)
>  -                       (int, void *));
>  +static void time_sched_init(irq_handler_t timer_routine);
>   static unsigned long gettimeoffset(void);
>
>   static struct irqaction bfin_timer_irq = {
>  @@ -64,7 +63,7 @@ static struct irqaction bfin_timer_irq = {
>   #define TIME_SCALE 1
>
>   static void
>  -time_sched_init(irqreturn_t(*timer_routine) (int, void *))
>  +time_sched_init(irq_handler_t timer_routine)
>   {
>         u32 tcount;
>
>  diff --git a/arch/blackfin/oprofile/timer_int.c b/arch/blackfin/oprofile/timer_int.c
>  index 6c6f860..e2c825a 100644
>  --- a/arch/blackfin/oprofile/timer_int.c
>  +++ b/arch/blackfin/oprofile/timer_int.c
>  @@ -40,10 +40,9 @@ static void disable_sys_timer0()
>   {
>   }
>
>  -static irqreturn_t sys_timer0_int_handler(int irq, void *dev_id,
>  -                                         struct pt_regs *regs)
>  +static irqreturn_t sys_timer0_int_handler(int irq, void *dev_id)
>   {
>  -       oprofile_add_sample(regs, 0);
>  +       oprofile_add_sample(get_irq_regs(), 0);
>         return IRQ_HANDLED;
>   }
>

Sorry for the delay, it is pretty good for Blackfin.

Acked-by: Bryan Wu <cooloney@...nel.org>

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