From: Ingo Molnar <mingo@elte.hu>
To: john stultz <johnstul@us.ibm.com>
Cc: lkml <linux-kernel@vger.kernel.org>,
Darren Hart <dvhltc@us.ibm.com>,
Nishanth Aravamudan <nacc@us.ibm.com>,
Frank Sorenson <frank@tuxrocks.com>,
George Anzinger <george@mvista.com>,
Roman Zippel <zippel@linux-m68k.org>,
Ulrich Windl <ulrich.windl@rz.uni-regensburg.de>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH 5/13] Time: i386 Conversion - part 1: Move timer_pit.c to i8253.c
Date: Sat, 26 Nov 2005 15:54:25 +0100 [thread overview]
Message-ID: <20051126145425.GG12999@elte.hu> (raw)
In-Reply-To: <20051122013548.18537.97737.sendpatchset@cog.beaverton.ibm.com>
- clean up timeofday-arch-i386-part1.patch
Signed-off-by: Ingo Molnar <mingo@elte.hu>
arch/i386/kernel/i8253.c | 29 +++++++++++++++++------------
1 files changed, 17 insertions(+), 12 deletions(-)
Index: linux/arch/i386/kernel/i8253.c
===================================================================
--- linux.orig/arch/i386/kernel/i8253.c
+++ linux/arch/i386/kernel/i8253.c
@@ -2,12 +2,12 @@
* i8253.c 8253/PIT functions
*
*/
-#include <linux/init.h>
-#include <linux/jiffies.h>
+#include <linux/clocksource.h>
#include <linux/spinlock.h>
+#include <linux/jiffies.h>
#include <linux/sysdev.h>
-#include <linux/clocksource.h>
#include <linux/module.h>
+#include <linux/init.h>
#include <asm/delay.h>
#include <asm/i8253.h>
@@ -34,6 +34,7 @@ void setup_pit_timer(void)
static int timer_resume(struct sys_device *dev)
{
setup_pit_timer();
+
return 0;
}
@@ -50,19 +51,20 @@ static struct sys_device device_timer =
static int __init init_timer_sysfs(void)
{
int error = sysdev_class_register(&timer_sysclass);
+
if (!error)
error = sysdev_register(&device_timer);
+
return error;
}
device_initcall(init_timer_sysfs);
-
-/* Since the PIT overflows every tick, its not very useful
+/*
+ * Since the PIT overflows every tick, its not very useful
* to just read by itself. So use jiffies to emulate a free
- * running counter.
+ * running counter:
*/
-
static cycle_t pit_read(void)
{
unsigned long flags, seq;
@@ -85,6 +87,7 @@ static cycle_t pit_read(void)
count = LATCH - 1;
}
spin_unlock_irqrestore(&i8253_lock, flags);
+
jifs = jiffies_64;
} while (read_seqretry(&xtime_lock, seq));
@@ -95,21 +98,23 @@ static cycle_t pit_read(void)
}
static struct clocksource clocksource_pit = {
- .name = "pit",
+ .name = "pit",
.rating = 110,
- .read = pit_read,
- .mask = (cycle_t)-1,
- .mult = 0,
- .shift = 20,
+ .read = pit_read,
+ .mask = (cycle_t)-1,
+ .mult = 0,
+ .shift = 20,
};
static int __init init_pit_clocksource(void)
{
+ /* TODO: bogus limit of 4 CPUs? --mingo */
if (num_possible_cpus() > 4) /* PIT does not scale! */
return 0;
clocksource_pit.mult = clocksource_hz2mult(CLOCK_TICK_RATE, 20);
register_clocksource(&clocksource_pit);
+
return 0;
}
module_init(init_pit_clocksource);
next prev parent reply other threads:[~2005-11-26 14:54 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-11-22 1:35 [PATCH 0/13] Time: Generic Timeofday Subsystem (v B11) john stultz
2005-11-22 1:35 ` [PATCH 1/13] Time: Reduced NTP rework (part 1) john stultz
2005-11-26 14:52 ` [patch] warn-on-once.patch Ingo Molnar
2005-11-26 15:03 ` Tim Schmielau
2005-11-26 15:03 ` Michal Schmidt
2005-11-26 15:17 ` Ingo Molnar
2005-11-26 15:21 ` Ingo Molnar
2005-11-26 14:52 ` [PATCH 1/13] Time: Reduced NTP rework (part 1) Ingo Molnar
2005-11-22 1:35 ` [PATCH 2/13] Time: Reduced NTP Rework (part 2) john stultz
2005-11-26 14:53 ` Ingo Molnar
2005-11-22 1:35 ` [PATCH 3/13] Time: Clocksource Infrastructure john stultz
2005-11-26 14:53 ` Ingo Molnar
2005-11-22 1:35 ` [PATCH 4/13] Time: Generic Timekeeping Infrastructure john stultz
2005-11-26 14:54 ` Ingo Molnar
2005-11-22 1:35 ` [PATCH 5/13] Time: i386 Conversion - part 1: Move timer_pit.c to i8253.c john stultz
2005-11-26 14:54 ` Ingo Molnar [this message]
2005-11-22 1:35 ` [PATCH 6/13] Time: i386 Conversion - part 2: Move timer_tsc.c to tsc.c john stultz
2005-11-26 14:54 ` Ingo Molnar
2005-11-22 1:36 ` [PATCH 7/13] Time: i386 Conversion - part 3: Rework TSC Support john stultz
2005-11-22 1:36 ` [PATCH 8/13] Time: i386 Conversion - part 4: ACPI PM variable renaming john stultz
2005-11-22 1:36 ` [PATCH 9/13] Time: i386 Conversion - part 5: Enable Generic Timekeeping john stultz
2005-11-26 14:55 ` Ingo Molnar
2005-11-22 1:36 ` [PATCH 10/13] Time: i386 Conversion - part 6: Remove Old Code john stultz
2005-11-22 1:36 ` [PATCH 11/13] Time: x86-64 Conversion to Generic Timekeeping john stultz
2005-11-26 14:55 ` Ingo Molnar
2005-11-26 15:11 ` Ingo Molnar
2005-11-22 1:36 ` [PATCH 12/13] Time: i386/x86-64 Clocksource Drivers john stultz
2005-11-26 14:55 ` Ingo Molnar
2005-11-22 1:36 ` [PATCH 13/13] Time: Generic Timekeeping Paraniod Debug Patch john stultz
2005-11-22 2:03 ` [PATCH 0/13] Time: Generic Timeofday Subsystem (v B11) john stultz
2005-11-26 14:50 ` Ingo Molnar
2005-11-26 14:58 ` Ingo Molnar
-- strict thread matches above, loose matches on Subject: below --
2005-12-15 2:00 [PATCH 0/13] Time: Generic Timeofday Subsystem (v B14) john stultz
2005-12-15 2:00 ` [PATCH 5/13] Time: i386 Conversion - part 1: Move timer_pit.c to i8253.c john stultz
2005-12-06 4:13 [PATCH 0/13] Time: Generic Timeofday Subsystem (v B13) john stultz
2005-12-06 4:14 ` [PATCH 5/13] Time: i386 Conversion - part 1: Move timer_pit.c to i8253.c john stultz
2005-12-02 3:25 [PATCH 0/13] Time: Generic Timeofday Subsystem (v B12) john stultz
2005-12-02 3:26 ` [PATCH 5/13] Time: i386 Conversion - part 1: Move timer_pit.c to i8253.c john stultz
2005-11-12 4:48 [PATCH 0/13] Time: Generic Timeofday Subsystem (v B10) john stultz
2005-11-12 4:49 ` [PATCH 5/13] Time: i386 Conversion - part 1: Move timer_pit.c to i8253.c john stultz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20051126145425.GG12999@elte.hu \
--to=mingo@elte.hu \
--cc=dvhltc@us.ibm.com \
--cc=frank@tuxrocks.com \
--cc=george@mvista.com \
--cc=johnstul@us.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=nacc@us.ibm.com \
--cc=tglx@linutronix.de \
--cc=ulrich.windl@rz.uni-regensburg.de \
--cc=zippel@linux-m68k.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®