| 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
| ||
|
Message-ID: <4D4B354E.9050705@snapgear.com>
Date: Fri, 4 Feb 2011 09:07:58 +1000
From: Greg Ungerer <gerg@...pgear.com>
To: <mingo@...hat.com>, <hpa@...or.com>,
<linux-kernel@...r.kernel.org>, <sammy@...my.net>,
<peterz@...radead.org>, <torbenh@....de>, <geert@...ux-m68k.org>,
<gerg@...inux.org>, <zippel@...ux-m68k.org>, <tglx@...utronix.de>
CC: <linux-tip-commits@...r.kernel.org>
Subject: Re: [tip:timers/core] m68k: Switch do_timer() to xtime_update()
On 01/02/11 00:09, tip-bot for Torben Hohn wrote:
> Commit-ID: e53f276beb655c711a5d1f25f800b61aa976e34f
> Gitweb: http://git.kernel.org/tip/e53f276beb655c711a5d1f25f800b61aa976e34f
> Author: Torben Hohn<torbenh@....de>
> AuthorDate: Thu, 27 Jan 2011 16:00:06 +0100
> Committer: Thomas Gleixner<tglx@...utronix.de>
> CommitDate: Mon, 31 Jan 2011 14:55:46 +0100
>
> m68k: Switch do_timer() to xtime_update()
>
> xtime_update() properly takes the xtime_lock
>
> Signed-off-by: Torben Hohn<torbenh@....de>
> Cc: Sam Creasey<sammy@...my.net>
> Cc: Peter Zijlstra<peterz@...radead.org>
> Cc: johnstul@...ibm.com
> Cc: Roman Zippel<zippel@...ux-m68k.org>
> Cc: hch@...radead.org
> Cc: yong.zhang0@...il.com
> Cc: Geert Uytterhoeven<geert@...ux-m68k.org>
> Cc: Greg Ungerer<gerg@...inux.org>
I have no problem with the m68knommu part:
Ack-by: Greg Ungerer <gerg@...inux.org>
> LKML-Reference:<20110127150006.23248.71790.stgit@...alhost>
> Signed-off-by: Thomas Gleixner<tglx@...utronix.de>
> ---
> arch/m68k/bvme6000/config.c | 4 ++--
> arch/m68k/kernel/time.c | 4 ++--
> arch/m68k/mvme147/config.c | 4 ++--
> arch/m68k/mvme16x/config.c | 4 ++--
> arch/m68k/sun3/sun3ints.c | 2 +-
> arch/m68knommu/kernel/time.c | 8 ++------
> 6 files changed, 11 insertions(+), 15 deletions(-)
>
> diff --git a/arch/m68k/bvme6000/config.c b/arch/m68k/bvme6000/config.c
> index 9fe6fef..1edd950 100644
> --- a/arch/m68k/bvme6000/config.c
> +++ b/arch/m68k/bvme6000/config.c
> @@ -45,8 +45,8 @@ extern int bvme6000_set_clock_mmss (unsigned long);
> extern void bvme6000_reset (void);
> void bvme6000_set_vectors (void);
>
> -/* Save tick handler routine pointer, will point to do_timer() in
> - * kernel/sched.c, called via bvme6000_process_int() */
> +/* Save tick handler routine pointer, will point to xtime_update() in
> + * kernel/timer/timekeeping.c, called via bvme6000_process_int() */
>
> static irq_handler_t tick_handler;
>
> diff --git a/arch/m68k/kernel/time.c b/arch/m68k/kernel/time.c
> index 06438da..18b34ee 100644
> --- a/arch/m68k/kernel/time.c
> +++ b/arch/m68k/kernel/time.c
> @@ -37,11 +37,11 @@ static inline int set_rtc_mmss(unsigned long nowtime)
>
> /*
> * timer_interrupt() needs to keep up the real-time clock,
> - * as well as call the "do_timer()" routine every clocktick
> + * as well as call the "xtime_update()" routine every clocktick
> */
> static irqreturn_t timer_interrupt(int irq, void *dummy)
> {
> - do_timer(1);
> + xtime_update(1);
> update_process_times(user_mode(get_irq_regs()));
> profile_tick(CPU_PROFILING);
>
> diff --git a/arch/m68k/mvme147/config.c b/arch/m68k/mvme147/config.c
> index 100baaa..6cb9c3a 100644
> --- a/arch/m68k/mvme147/config.c
> +++ b/arch/m68k/mvme147/config.c
> @@ -46,8 +46,8 @@ extern void mvme147_reset (void);
>
> static int bcd2int (unsigned char b);
>
> -/* Save tick handler routine pointer, will point to do_timer() in
> - * kernel/sched.c, called via mvme147_process_int() */
> +/* Save tick handler routine pointer, will point to xtime_update() in
> + * kernel/time/timekeeping.c, called via mvme147_process_int() */
>
> irq_handler_t tick_handler;
>
> diff --git a/arch/m68k/mvme16x/config.c b/arch/m68k/mvme16x/config.c
> index 11edf61..0b28e26 100644
> --- a/arch/m68k/mvme16x/config.c
> +++ b/arch/m68k/mvme16x/config.c
> @@ -51,8 +51,8 @@ extern void mvme16x_reset (void);
>
> int bcd2int (unsigned char b);
>
> -/* Save tick handler routine pointer, will point to do_timer() in
> - * kernel/sched.c, called via mvme16x_process_int() */
> +/* Save tick handler routine pointer, will point to xtime_update() in
> + * kernel/time/timekeeping.c, called via mvme16x_process_int() */
>
> static irq_handler_t tick_handler;
>
> diff --git a/arch/m68k/sun3/sun3ints.c b/arch/m68k/sun3/sun3ints.c
> index 2d9e21b..6464ad3 100644
> --- a/arch/m68k/sun3/sun3ints.c
> +++ b/arch/m68k/sun3/sun3ints.c
> @@ -66,7 +66,7 @@ static irqreturn_t sun3_int5(int irq, void *dev_id)
> #ifdef CONFIG_SUN3
> intersil_clear();
> #endif
> - do_timer(1);
> + xtime_update(1);
> update_process_times(user_mode(get_irq_regs()));
> if (!(kstat_cpu(0).irqs[irq] % 20))
> sun3_leds(led_pattern[(kstat_cpu(0).irqs[irq] % 160) / 20]);
> diff --git a/arch/m68knommu/kernel/time.c b/arch/m68knommu/kernel/time.c
> index d6ac2a4..6623909 100644
> --- a/arch/m68knommu/kernel/time.c
> +++ b/arch/m68knommu/kernel/time.c
> @@ -36,7 +36,7 @@ static inline int set_rtc_mmss(unsigned long nowtime)
> #ifndef CONFIG_GENERIC_CLOCKEVENTS
> /*
> * timer_interrupt() needs to keep up the real-time clock,
> - * as well as call the "do_timer()" routine every clocktick
> + * as well as call the "xtime_update()" routine every clocktick
> */
> irqreturn_t arch_timer_interrupt(int irq, void *dummy)
> {
> @@ -44,11 +44,7 @@ irqreturn_t arch_timer_interrupt(int irq, void *dummy)
> if (current->pid)
> profile_tick(CPU_PROFILING);
>
> - write_seqlock(&xtime_lock);
> -
> - do_timer(1);
> -
> - write_sequnlock(&xtime_lock);
> + xtime_update(1);
>
> update_process_times(user_mode(get_irq_regs()));
>
>
--
------------------------------------------------------------------------
Greg Ungerer -- Principal Engineer EMAIL: gerg@...pgear.com
SnapGear Group, McAfee PHONE: +61 7 3435 2888
8 Gardner Close FAX: +61 7 3217 5323
Milton, QLD, 4064, Australia WEB: http://www.SnapGear.com
--
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/
Powered by blists - more mailing lists