* [patch] x86 BIOS interface for RTC on SGI UV
@ 2008-07-09 20:27 Russ Anderson
2008-07-09 20:58 ` Rene Herman
2008-07-10 1:44 ` H. Peter Anvin
0 siblings, 2 replies; 8+ messages in thread
From: Russ Anderson @ 2008-07-09 20:27 UTC (permalink / raw)
To: mingo, tglx; +Cc: linux-kernel, Jack Steiner, Russ Anderson
[patch] x86 BIOS interface for RTC on SGI UV
Real-time code needs to know the number of cycles per second
on SGI UV. The information is provided via a run time BIOS
call. This patch provides the linux side of that interface.
This is the first of several run time BIOS calls to be defined
in uv/bios.h and bios_uv.c.
Note that BIOS_CALL() is just a stub for now. The bios
side is being worked on.
Signed-off-by: Russ Anderson <rja@sgi.com>
---
arch/x86/kernel/Makefile | 1
arch/x86/kernel/bios_uv.c | 48 +++++++++++++++++++++++++++
arch/x86/kernel/genx2apic_uv_x.c | 23 +++++++++++++
include/asm-x86/uv/bios.h | 68 +++++++++++++++++++++++++++++++++++++++
4 files changed, 140 insertions(+)
Index: linux/include/asm-x86/uv/bios.h
===================================================================
--- /dev/null 1970-01-01 00:00:00.000000000 +0000
+++ linux/include/asm-x86/uv/bios.h 2008-07-09 14:30:38.000000000 -0500
@@ -0,0 +1,68 @@
+#ifndef _ASM_X86_BIOS_H
+#define _ASM_X86_BIOS_H
+
+/*
+ * BIOS layer definitions.
+ *
+ * Copyright (c) 2008 Silicon Graphics, Inc. All Rights Reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+#include <linux/rtc.h>
+
+#define BIOS_FREQ_BASE 0x01000001
+
+enum {
+ BIOS_FREQ_BASE_PLATFORM = 0,
+ BIOS_FREQ_BASE_INTERVAL_TIMER = 1,
+ BIOS_FREQ_BASE_REALTIME_CLOCK = 2
+};
+
+# define BIOS_CALL(result, a0, a1, a2, a3, a4, a5, a6, a7) \
+ do { \
+ /* XXX - the real call goes here */ \
+ result.status = BIOS_STATUS_UNIMPLEMENTED; \
+ isrv.v0 = 0; \
+ isrv.v1 = 0; \
+ } while (0)
+
+enum {
+ BIOS_STATUS_SUCCESS = 0,
+ BIOS_STATUS_UNIMPLEMENTED = -1,
+ BIOS_STATUS_EINVAL = -2,
+ BIOS_STATUS_ERROR = -3
+};
+
+struct uv_bios_retval {
+ /*
+ * A zero status value indicates call completed without error.
+ * A negative status value indicates reason of call failure.
+ * A positive status value indicates success but an
+ * informational value should be printed (e.g., "reboot for
+ * change to take effect").
+ */
+ s64 status;
+ u64 v0;
+ u64 v1;
+ u64 v2;
+};
+
+extern long
+x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second,
+ unsigned long *drift_info);
+extern const char *x86_bios_strerror(long status);
+
+#endif /* _ASM_X86_BIOS_H */
Index: linux/arch/x86/kernel/genx2apic_uv_x.c
===================================================================
--- linux.orig/arch/x86/kernel/genx2apic_uv_x.c 2008-07-09 14:30:15.000000000 -0500
+++ linux/arch/x86/kernel/genx2apic_uv_x.c 2008-07-09 14:30:38.000000000 -0500
@@ -22,6 +22,7 @@
#include <asm/genapic.h>
#include <asm/uv/uv_mmrs.h>
#include <asm/uv/uv_hub.h>
+#include <asm/uv/bios.h>
DEFINE_PER_CPU(struct uv_hub_info_s, __uv_hub_info);
EXPORT_PER_CPU_SYMBOL_GPL(__uv_hub_info);
@@ -38,6 +39,9 @@ EXPORT_SYMBOL_GPL(uv_cpu_to_blade);
short uv_possible_blades;
EXPORT_SYMBOL_GPL(uv_possible_blades);
+unsigned long sn_rtc_cycles_per_second;
+EXPORT_SYMBOL(sn_rtc_cycles_per_second);
+
/* Start with all IRQs pointing to boot CPU. IRQ balancing will shift them. */
static cpumask_t uv_target_cpus(void)
@@ -208,6 +212,23 @@ static __init void get_lowmem_redirect(u
BUG();
}
+static __init void uv_rtc_init(void)
+{
+ long status, ticks_per_sec, drift;
+
+ status =
+ x86_bios_freq_base(BIOS_FREQ_BASE_REALTIME_CLOCK, &ticks_per_sec,
+ &drift);
+ if (status != 0 || ticks_per_sec < 100000) {
+ printk(KERN_WARNING
+ "unable to determine platform RTC clock frequency, "
+ "guessing.\n");
+ /* BIOS gives wrong value for clock freq. so guess */
+ sn_rtc_cycles_per_second = 1000000000000UL / 30000UL;
+ } else
+ sn_rtc_cycles_per_second = ticks_per_sec;
+}
+
static __init void uv_system_init(void)
{
union uvh_si_addr_map_config_u m_n_config;
@@ -258,6 +279,8 @@ static __init void uv_system_init(void)
node_id.v = uv_read_local_mmr(UVH_NODE_ID);
gnode_upper = (((unsigned long)node_id.s.node_id) &
~((1 << n_val) - 1)) << m_val;
+
+ uv_rtc_init();
for_each_present_cpu(cpu) {
nid = cpu_to_node(cpu);
Index: linux/arch/x86/kernel/bios_uv.c
===================================================================
--- /dev/null 1970-01-01 00:00:00.000000000 +0000
+++ linux/arch/x86/kernel/bios_uv.c 2008-07-09 14:30:38.000000000 -0500
@@ -0,0 +1,48 @@
+/*
+ * BIOS run time interface routines.
+ *
+ * Copyright (c) 2008 Silicon Graphics, Inc. All Rights Reserved.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with this program; if not, write to the Free Software
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ */
+
+#include <asm/uv/bios.h>
+
+const char *
+x86_bios_strerror(long status)
+{
+ const char *str;
+ switch (status) {
+ case 0: str = "Call completed without error"; break;
+ case -1: str = "Not implemented"; break;
+ case -2: str = "Invalid argument"; break;
+ case -3: str = "Call completed with error"; break;
+ default: str = "Unknown BIOS status code"; break;
+ }
+ return str;
+}
+
+long
+x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second,
+ unsigned long *drift_info)
+{
+ struct uv_bios_retval isrv;
+
+ BIOS_CALL(isrv, BIOS_FREQ_BASE, which, 0, 0, 0, 0, 0, 0);
+ *ticks_per_second = isrv.v0;
+ *drift_info = isrv.v1;
+ return isrv.status;
+}
+EXPORT_SYMBOL_GPL(x86_bios_freq_base);
Index: linux/arch/x86/kernel/Makefile
===================================================================
--- linux.orig/arch/x86/kernel/Makefile 2008-07-09 14:30:14.000000000 -0500
+++ linux/arch/x86/kernel/Makefile 2008-07-09 14:30:38.000000000 -0500
@@ -103,6 +103,7 @@ obj-$(CONFIG_OLPC) += olpc.o
# 64 bit specific files
ifeq ($(CONFIG_X86_64),y)
obj-y += genapic_64.o genapic_flat_64.o genx2apic_uv_x.o tlb_uv.o
+ obj-y += bios_uv.o
obj-$(CONFIG_X86_PM_TIMER) += pmtimer_64.o
obj-$(CONFIG_AUDIT) += audit_64.o
--
Russ Anderson, OS RAS/Partitioning Project Lead
SGI - Silicon Graphics Inc rja@sgi.com
^ permalink raw reply [flat|nested] 8+ messages in thread* Re: [patch] x86 BIOS interface for RTC on SGI UV 2008-07-09 20:27 [patch] x86 BIOS interface for RTC on SGI UV Russ Anderson @ 2008-07-09 20:58 ` Rene Herman 2008-07-09 21:36 ` Russ Anderson 2008-07-10 1:44 ` H. Peter Anvin 1 sibling, 1 reply; 8+ messages in thread From: Rene Herman @ 2008-07-09 20:58 UTC (permalink / raw) To: Russ Anderson; +Cc: mingo, tglx, linux-kernel, Jack Steiner On 09-07-08 22:27, Russ Anderson wrote: > [patch] x86 BIOS interface for RTC on SGI UV [ ... ] > +# define BIOS_CALL(result, a0, a1, a2, a3, a4, a5, a6, a7) \ > + do { \ > + /* XXX - the real call goes here */ \ > + result.status = BIOS_STATUS_UNIMPLEMENTED; \ > + isrv.v0 = 0; \ > + isrv.v1 = 0; \ isrv? should be "result" I guess? [ ... ] > +extern long > +x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, > + unsigned long *drift_info); [ ... ] > +unsigned long sn_rtc_cycles_per_second; > +EXPORT_SYMBOL(sn_rtc_cycles_per_second); > + > +static __init void uv_rtc_init(void) > +{ > + long status, ticks_per_sec, drift; > + > + status = > + x86_bios_freq_base(BIOS_FREQ_BASE_REALTIME_CLOCK, &ticks_per_sec, > + &drift); > + if (status != 0 || ticks_per_sec < 100000) { > + printk(KERN_WARNING > + "unable to determine platform RTC clock frequency, " > + "guessing.\n"); > + /* BIOS gives wrong value for clock freq. so guess */ > + sn_rtc_cycles_per_second = 1000000000000UL / 30000UL; > + } else > + sn_rtc_cycles_per_second = ticks_per_sec; > +} It seems ticks_per_sec and drift should be "unsigned long"... > +const char * > +x86_bios_strerror(long status) [ ... ] > +long > +x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, > + unsigned long *drift_info) > +{ > + struct uv_bios_retval isrv; ... and wouldn't uv_bios_strerror/uv_bios_freq_base be better names? Rene. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [patch] x86 BIOS interface for RTC on SGI UV 2008-07-09 20:58 ` Rene Herman @ 2008-07-09 21:36 ` Russ Anderson 2008-07-09 21:47 ` Rene Herman 0 siblings, 1 reply; 8+ messages in thread From: Russ Anderson @ 2008-07-09 21:36 UTC (permalink / raw) To: Rene Herman; +Cc: mingo, tglx, linux-kernel, Jack Steiner On Wed, Jul 09, 2008 at 10:58:19PM +0200, Rene Herman wrote: > On 09-07-08 22:27, Russ Anderson wrote: > > >+# define BIOS_CALL(result, a0, a1, a2, a3, a4, a5, a6, a7) \ > >+ do { \ > >+ /* XXX - the real call goes here */ \ > >+ result.status = BIOS_STATUS_UNIMPLEMENTED; \ > >+ isrv.v0 = 0; \ > >+ isrv.v1 = 0; \ > > isrv? should be "result" I guess? Yes. > >+static __init void uv_rtc_init(void) > >+{ > >+ long status, ticks_per_sec, drift; > >+ > >+ status = > >+ x86_bios_freq_base(BIOS_FREQ_BASE_REALTIME_CLOCK, &ticks_per_sec, > >+ &drift); > >+ if (status != 0 || ticks_per_sec < 100000) { > >+ printk(KERN_WARNING > >+ "unable to determine platform RTC clock frequency, " > >+ "guessing.\n"); > >+ /* BIOS gives wrong value for clock freq. so guess */ > >+ sn_rtc_cycles_per_second = 1000000000000UL / 30000UL; > >+ } else > >+ sn_rtc_cycles_per_second = ticks_per_sec; > >+} > > It seems ticks_per_sec and drift should be "unsigned long"... Yes. > >+const char * > >+x86_bios_strerror(long status) > > [ ... ] > > >+long > >+x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, > >+ unsigned long *drift_info) > >+{ > >+ struct uv_bios_retval isrv; > > ... and wouldn't uv_bios_strerror/uv_bios_freq_base be better names? The plan is for both x86 and ia64 versions of uv, with this being the x86 version. The ia64 version has ia64_sal_strerror() and ia64_sal_freq_base(). Hence the x86_bios_* naming convention. I'm not strongly tied to that convention, it just seemed logical. Here is the updated patch. --------------------------------------------------------------------- [patch] x86 BIOS interface for RTC on SGI UV Real-time code needs to know the number of cycles per second on SGI UV. The information is provided via a run time BIOS call. This patch provides the linux side of that interface. This is the first of several run time BIOS calls to be defined in uv/bios.h and bios_uv.c. Note that BIOS_CALL() is just a stub for now. The bios side is being worked on. Signed-off-by: Russ Anderson <rja@sgi.com> --- arch/x86/kernel/Makefile | 1 arch/x86/kernel/bios_uv.c | 48 +++++++++++++++++++++++++++ arch/x86/kernel/genx2apic_uv_x.c | 23 +++++++++++++ include/asm-x86/uv/bios.h | 68 +++++++++++++++++++++++++++++++++++++++ 4 files changed, 140 insertions(+) Index: linux/include/asm-x86/uv/bios.h =================================================================== --- /dev/null 1970-01-01 00:00:00.000000000 +0000 +++ linux/include/asm-x86/uv/bios.h 2008-07-09 16:31:33.000000000 -0500 @@ -0,0 +1,68 @@ +#ifndef _ASM_X86_BIOS_H +#define _ASM_X86_BIOS_H + +/* + * BIOS layer definitions. + * + * Copyright (c) 2008 Silicon Graphics, Inc. All Rights Reserved. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + +#include <linux/rtc.h> + +#define BIOS_FREQ_BASE 0x01000001 + +enum { + BIOS_FREQ_BASE_PLATFORM = 0, + BIOS_FREQ_BASE_INTERVAL_TIMER = 1, + BIOS_FREQ_BASE_REALTIME_CLOCK = 2 +}; + +# define BIOS_CALL(result, a0, a1, a2, a3, a4, a5, a6, a7) \ + do { \ + /* XXX - the real call goes here */ \ + result.status = BIOS_STATUS_UNIMPLEMENTED; \ + result.v0 = 0; \ + result.v1 = 0; \ + } while (0) + +enum { + BIOS_STATUS_SUCCESS = 0, + BIOS_STATUS_UNIMPLEMENTED = -1, + BIOS_STATUS_EINVAL = -2, + BIOS_STATUS_ERROR = -3 +}; + +struct x86_bios_retval { + /* + * A zero status value indicates call completed without error. + * A negative status value indicates reason of call failure. + * A positive status value indicates success but an + * informational value should be printed (e.g., "reboot for + * change to take effect"). + */ + s64 status; + u64 v0; + u64 v1; + u64 v2; +}; + +extern long +x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, + unsigned long *drift_info); +extern const char *x86_bios_strerror(long status); + +#endif /* _ASM_X86_BIOS_H */ Index: linux/arch/x86/kernel/genx2apic_uv_x.c =================================================================== --- linux.orig/arch/x86/kernel/genx2apic_uv_x.c 2008-07-09 16:05:25.000000000 -0500 +++ linux/arch/x86/kernel/genx2apic_uv_x.c 2008-07-09 16:23:21.000000000 -0500 @@ -22,6 +22,7 @@ #include <asm/genapic.h> #include <asm/uv/uv_mmrs.h> #include <asm/uv/uv_hub.h> +#include <asm/uv/bios.h> DEFINE_PER_CPU(struct uv_hub_info_s, __uv_hub_info); EXPORT_PER_CPU_SYMBOL_GPL(__uv_hub_info); @@ -38,6 +39,9 @@ EXPORT_SYMBOL_GPL(uv_cpu_to_blade); short uv_possible_blades; EXPORT_SYMBOL_GPL(uv_possible_blades); +unsigned long sn_rtc_cycles_per_second; +EXPORT_SYMBOL(sn_rtc_cycles_per_second); + /* Start with all IRQs pointing to boot CPU. IRQ balancing will shift them. */ static cpumask_t uv_target_cpus(void) @@ -208,6 +212,23 @@ static __init void get_lowmem_redirect(u BUG(); } +static __init void uv_rtc_init(void) +{ + unsigned long status, ticks_per_sec, drift; + + status = + x86_bios_freq_base(BIOS_FREQ_BASE_REALTIME_CLOCK, &ticks_per_sec, + &drift); + if (status != 0 || ticks_per_sec < 100000) { + printk(KERN_WARNING + "unable to determine platform RTC clock frequency, " + "guessing.\n"); + /* BIOS gives wrong value for clock freq. so guess */ + sn_rtc_cycles_per_second = 1000000000000UL / 30000UL; + } else + sn_rtc_cycles_per_second = ticks_per_sec; +} + static __init void uv_system_init(void) { union uvh_si_addr_map_config_u m_n_config; @@ -258,6 +279,8 @@ static __init void uv_system_init(void) node_id.v = uv_read_local_mmr(UVH_NODE_ID); gnode_upper = (((unsigned long)node_id.s.node_id) & ~((1 << n_val) - 1)) << m_val; + + uv_rtc_init(); for_each_present_cpu(cpu) { nid = cpu_to_node(cpu); Index: linux/arch/x86/kernel/bios_uv.c =================================================================== --- /dev/null 1970-01-01 00:00:00.000000000 +0000 +++ linux/arch/x86/kernel/bios_uv.c 2008-07-09 16:31:08.000000000 -0500 @@ -0,0 +1,48 @@ +/* + * BIOS run time interface routines. + * + * Copyright (c) 2008 Silicon Graphics, Inc. All Rights Reserved. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + +#include <asm/uv/bios.h> + +const char * +x86_bios_strerror(long status) +{ + const char *str; + switch (status) { + case 0: str = "Call completed without error"; break; + case -1: str = "Not implemented"; break; + case -2: str = "Invalid argument"; break; + case -3: str = "Call completed with error"; break; + default: str = "Unknown BIOS status code"; break; + } + return str; +} + +long +x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, + unsigned long *drift_info) +{ + struct x86_bios_retval isrv; + + BIOS_CALL(isrv, BIOS_FREQ_BASE, which, 0, 0, 0, 0, 0, 0); + *ticks_per_second = isrv.v0; + *drift_info = isrv.v1; + return isrv.status; +} +EXPORT_SYMBOL_GPL(x86_bios_freq_base); Index: linux/arch/x86/kernel/Makefile =================================================================== --- linux.orig/arch/x86/kernel/Makefile 2008-07-09 16:05:25.000000000 -0500 +++ linux/arch/x86/kernel/Makefile 2008-07-09 16:05:58.000000000 -0500 @@ -103,6 +103,7 @@ obj-$(CONFIG_OLPC) += olpc.o # 64 bit specific files ifeq ($(CONFIG_X86_64),y) obj-y += genapic_64.o genapic_flat_64.o genx2apic_uv_x.o tlb_uv.o + obj-y += bios_uv.o obj-$(CONFIG_X86_PM_TIMER) += pmtimer_64.o obj-$(CONFIG_AUDIT) += audit_64.o -- Russ Anderson, OS RAS/Partitioning Project Lead SGI - Silicon Graphics Inc rja@sgi.com ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [patch] x86 BIOS interface for RTC on SGI UV 2008-07-09 21:36 ` Russ Anderson @ 2008-07-09 21:47 ` Rene Herman 2008-07-09 22:09 ` Russ Anderson 0 siblings, 1 reply; 8+ messages in thread From: Rene Herman @ 2008-07-09 21:47 UTC (permalink / raw) To: Russ Anderson; +Cc: mingo, tglx, linux-kernel, Jack Steiner On 09-07-08 23:36, Russ Anderson wrote: >> ... and wouldn't uv_bios_strerror/uv_bios_freq_base be better names? > > The plan is for both x86 and ia64 versions of uv, with this being > the x86 version. The ia64 version has ia64_sal_strerror() > and ia64_sal_freq_base(). Hence the x86_bios_* naming convention. > I'm not strongly tied to that convention, it just seemed logical. Ah, I see. Not much of an opinion then. x86_bios sounds a bit generic, but whatever... > Here is the updated patch. [ ... ] > +enum { > + BIOS_STATUS_SUCCESS = 0, > + BIOS_STATUS_UNIMPLEMENTED = -1, > + BIOS_STATUS_EINVAL = -2, > + BIOS_STATUS_ERROR = -3 > +}; [ .. ] > +extern long > +x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, > + unsigned long *drift_info); > +static __init void uv_rtc_init(void) > +{ > + unsigned long status, ticks_per_sec, drift; *status* should remain a long :-) Rene. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [patch] x86 BIOS interface for RTC on SGI UV 2008-07-09 21:47 ` Rene Herman @ 2008-07-09 22:09 ` Russ Anderson 2008-07-09 22:54 ` Rene Herman 0 siblings, 1 reply; 8+ messages in thread From: Russ Anderson @ 2008-07-09 22:09 UTC (permalink / raw) To: Rene Herman; +Cc: mingo, tglx, linux-kernel, Jack Steiner On Wed, Jul 09, 2008 at 11:47:57PM +0200, Rene Herman wrote: > On 09-07-08 23:36, Russ Anderson wrote: > > >>... and wouldn't uv_bios_strerror/uv_bios_freq_base be better names? > > > >The plan is for both x86 and ia64 versions of uv, with this being > >the x86 version. The ia64 version has ia64_sal_strerror() > >and ia64_sal_freq_base(). Hence the x86_bios_* naming convention. > >I'm not strongly tied to that convention, it just seemed logical. > > Ah, I see. Not much of an opinion then. x86_bios sounds a bit generic, > but whatever... On ia64, the sal interface is an Intel standard. On x86, the bios interface is SGI UV specific. I'll make it uv_bios_* . > >Here is the updated patch. > > [ ... ] > > >+enum { > >+ BIOS_STATUS_SUCCESS = 0, > >+ BIOS_STATUS_UNIMPLEMENTED = -1, > >+ BIOS_STATUS_EINVAL = -2, > >+ BIOS_STATUS_ERROR = -3 > >+}; > > [ .. ] > > >+extern long > >+x86_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, > >+ unsigned long *drift_info); > > >+static __init void uv_rtc_init(void) > >+{ > >+ unsigned long status, ticks_per_sec, drift; > > *status* should remain a long :-) Ouch. Thanks. --------------------------------------------------------------------- [patch] x86 BIOS interface for RTC on SGI UV Real-time code needs to know the number of cycles per second on SGI UV. The information is provided via a run time BIOS call. This patch provides the linux side of that interface. This is the first of several run time BIOS calls to be defined in uv/bios.h and bios_uv.c. Note that BIOS_CALL() is just a stub for now. The bios side is being worked on. Signed-off-by: Russ Anderson <rja@sgi.com> --- arch/x86/kernel/Makefile | 1 arch/x86/kernel/bios_uv.c | 48 +++++++++++++++++++++++++++ arch/x86/kernel/genx2apic_uv_x.c | 24 +++++++++++++ include/asm-x86/uv/bios.h | 68 +++++++++++++++++++++++++++++++++++++++ 4 files changed, 141 insertions(+) Index: linux/include/asm-x86/uv/bios.h =================================================================== --- /dev/null 1970-01-01 00:00:00.000000000 +0000 +++ linux/include/asm-x86/uv/bios.h 2008-07-09 17:01:42.000000000 -0500 @@ -0,0 +1,68 @@ +#ifndef _ASM_X86_BIOS_H +#define _ASM_X86_BIOS_H + +/* + * BIOS layer definitions. + * + * Copyright (c) 2008 Silicon Graphics, Inc. All Rights Reserved. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + +#include <linux/rtc.h> + +#define BIOS_FREQ_BASE 0x01000001 + +enum { + BIOS_FREQ_BASE_PLATFORM = 0, + BIOS_FREQ_BASE_INTERVAL_TIMER = 1, + BIOS_FREQ_BASE_REALTIME_CLOCK = 2 +}; + +# define BIOS_CALL(result, a0, a1, a2, a3, a4, a5, a6, a7) \ + do { \ + /* XXX - the real call goes here */ \ + result.status = BIOS_STATUS_UNIMPLEMENTED; \ + result.v0 = 0; \ + result.v1 = 0; \ + } while (0) + +enum { + BIOS_STATUS_SUCCESS = 0, + BIOS_STATUS_UNIMPLEMENTED = -1, + BIOS_STATUS_EINVAL = -2, + BIOS_STATUS_ERROR = -3 +}; + +struct uv_bios_retval { + /* + * A zero status value indicates call completed without error. + * A negative status value indicates reason of call failure. + * A positive status value indicates success but an + * informational value should be printed (e.g., "reboot for + * change to take effect"). + */ + s64 status; + u64 v0; + u64 v1; + u64 v2; +}; + +extern long +uv_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, + unsigned long *drift_info); +extern const char *uv_bios_strerror(long status); + +#endif /* _ASM_X86_BIOS_H */ Index: linux/arch/x86/kernel/genx2apic_uv_x.c =================================================================== --- linux.orig/arch/x86/kernel/genx2apic_uv_x.c 2008-07-09 17:01:25.000000000 -0500 +++ linux/arch/x86/kernel/genx2apic_uv_x.c 2008-07-09 17:01:42.000000000 -0500 @@ -22,6 +22,7 @@ #include <asm/genapic.h> #include <asm/uv/uv_mmrs.h> #include <asm/uv/uv_hub.h> +#include <asm/uv/bios.h> DEFINE_PER_CPU(struct uv_hub_info_s, __uv_hub_info); EXPORT_PER_CPU_SYMBOL_GPL(__uv_hub_info); @@ -38,6 +39,9 @@ EXPORT_SYMBOL_GPL(uv_cpu_to_blade); short uv_possible_blades; EXPORT_SYMBOL_GPL(uv_possible_blades); +unsigned long sn_rtc_cycles_per_second; +EXPORT_SYMBOL(sn_rtc_cycles_per_second); + /* Start with all IRQs pointing to boot CPU. IRQ balancing will shift them. */ static cpumask_t uv_target_cpus(void) @@ -208,6 +212,24 @@ static __init void get_lowmem_redirect(u BUG(); } +static __init void uv_rtc_init(void) +{ + unsigned long ticks_per_sec, drift; + long status; + + status = + uv_bios_freq_base(BIOS_FREQ_BASE_REALTIME_CLOCK, &ticks_per_sec, + &drift); + if (status != 0 || ticks_per_sec < 100000) { + printk(KERN_WARNING + "unable to determine platform RTC clock frequency, " + "guessing.\n"); + /* BIOS gives wrong value for clock freq. so guess */ + sn_rtc_cycles_per_second = 1000000000000UL / 30000UL; + } else + sn_rtc_cycles_per_second = ticks_per_sec; +} + static __init void uv_system_init(void) { union uvh_si_addr_map_config_u m_n_config; @@ -258,6 +280,8 @@ static __init void uv_system_init(void) node_id.v = uv_read_local_mmr(UVH_NODE_ID); gnode_upper = (((unsigned long)node_id.s.node_id) & ~((1 << n_val) - 1)) << m_val; + + uv_rtc_init(); for_each_present_cpu(cpu) { nid = cpu_to_node(cpu); Index: linux/arch/x86/kernel/bios_uv.c =================================================================== --- /dev/null 1970-01-01 00:00:00.000000000 +0000 +++ linux/arch/x86/kernel/bios_uv.c 2008-07-09 17:01:42.000000000 -0500 @@ -0,0 +1,48 @@ +/* + * BIOS run time interface routines. + * + * Copyright (c) 2008 Silicon Graphics, Inc. All Rights Reserved. + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program; if not, write to the Free Software + * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA + */ + +#include <asm/uv/bios.h> + +const char * +uv_bios_strerror(long status) +{ + const char *str; + switch (status) { + case 0: str = "Call completed without error"; break; + case -1: str = "Not implemented"; break; + case -2: str = "Invalid argument"; break; + case -3: str = "Call completed with error"; break; + default: str = "Unknown BIOS status code"; break; + } + return str; +} + +long +uv_bios_freq_base(unsigned long which, unsigned long *ticks_per_second, + unsigned long *drift_info) +{ + struct uv_bios_retval isrv; + + BIOS_CALL(isrv, BIOS_FREQ_BASE, which, 0, 0, 0, 0, 0, 0); + *ticks_per_second = isrv.v0; + *drift_info = isrv.v1; + return isrv.status; +} +EXPORT_SYMBOL_GPL(uv_bios_freq_base); Index: linux/arch/x86/kernel/Makefile =================================================================== --- linux.orig/arch/x86/kernel/Makefile 2008-07-09 17:01:25.000000000 -0500 +++ linux/arch/x86/kernel/Makefile 2008-07-09 17:01:43.000000000 -0500 @@ -103,6 +103,7 @@ obj-$(CONFIG_OLPC) += olpc.o # 64 bit specific files ifeq ($(CONFIG_X86_64),y) obj-y += genapic_64.o genapic_flat_64.o genx2apic_uv_x.o tlb_uv.o + obj-y += bios_uv.o obj-$(CONFIG_X86_PM_TIMER) += pmtimer_64.o obj-$(CONFIG_AUDIT) += audit_64.o -- Russ Anderson, OS RAS/Partitioning Project Lead SGI - Silicon Graphics Inc rja@sgi.com ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [patch] x86 BIOS interface for RTC on SGI UV 2008-07-09 22:09 ` Russ Anderson @ 2008-07-09 22:54 ` Rene Herman 0 siblings, 0 replies; 8+ messages in thread From: Rene Herman @ 2008-07-09 22:54 UTC (permalink / raw) To: Russ Anderson; +Cc: mingo, tglx, linux-kernel, Jack Steiner On 10-07-08 00:09, Russ Anderson wrote: > Ouch. Thanks. Yep, should do. Pestering you about the include guard maybe missing a _UV in there would be going overboard... > Index: linux/include/asm-x86/uv/bios.h > =================================================================== > --- /dev/null 1970-01-01 00:00:00.000000000 +0000 > +++ linux/include/asm-x86/uv/bios.h 2008-07-09 17:01:42.000000000 -0500 > @@ -0,0 +1,68 @@ > +#ifndef _ASM_X86_BIOS_H > +#define _ASM_X86_BIOS_H Rene. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [patch] x86 BIOS interface for RTC on SGI UV 2008-07-09 20:27 [patch] x86 BIOS interface for RTC on SGI UV Russ Anderson 2008-07-09 20:58 ` Rene Herman @ 2008-07-10 1:44 ` H. Peter Anvin 2008-07-10 14:54 ` Russ Anderson 1 sibling, 1 reply; 8+ messages in thread From: H. Peter Anvin @ 2008-07-10 1:44 UTC (permalink / raw) To: Russ Anderson; +Cc: mingo, tglx, linux-kernel, Jack Steiner Russ Anderson wrote: > [patch] x86 BIOS interface for RTC on SGI UV > > Real-time code needs to know the number of cycles per second > on SGI UV. The information is provided via a run time BIOS > call. This patch provides the linux side of that interface. > This is the first of several run time BIOS calls to be defined > in uv/bios.h and bios_uv.c. > > Note that BIOS_CALL() is just a stub for now. The bios > side is being worked on. > Do you have a proposed specification that could be reviewed before it is cast in stone? -hpa ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [patch] x86 BIOS interface for RTC on SGI UV 2008-07-10 1:44 ` H. Peter Anvin @ 2008-07-10 14:54 ` Russ Anderson 0 siblings, 0 replies; 8+ messages in thread From: Russ Anderson @ 2008-07-10 14:54 UTC (permalink / raw) To: H. Peter Anvin; +Cc: mingo, tglx, linux-kernel, Jack Steiner On Wed, Jul 09, 2008 at 09:44:26PM -0400, H. Peter Anvin wrote: > Russ Anderson wrote: > >[patch] x86 BIOS interface for RTC on SGI UV > > > >Real-time code needs to know the number of cycles per second > >on SGI UV. The information is provided via a run time BIOS > >call. This patch provides the linux side of that interface. > >This is the first of several run time BIOS calls to be defined > >in uv/bios.h and bios_uv.c. > > > >Note that BIOS_CALL() is just a stub for now. The bios > >side is being worked on. > > > > Do you have a proposed specification that could be reviewed before it is > cast in stone? I would like to follow the ia64 SAL spec (http://download.intel.com/design/itanium/specupdt/245359.pdf). There is no corresponding Intel spec for x86_64. For examples of what I would like, see the ia64 files: include/asm-ia64/sal.h, include/asm-ia64/sn/sn_sal.h, and arch/ia64/kernel/sal.c. -- Russ Anderson, OS RAS/Partitioning Project Lead SGI - Silicon Graphics Inc rja@sgi.com ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2008-07-10 14:54 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2008-07-09 20:27 [patch] x86 BIOS interface for RTC on SGI UV Russ Anderson 2008-07-09 20:58 ` Rene Herman 2008-07-09 21:36 ` Russ Anderson 2008-07-09 21:47 ` Rene Herman 2008-07-09 22:09 ` Russ Anderson 2008-07-09 22:54 ` Rene Herman 2008-07-10 1:44 ` H. Peter Anvin 2008-07-10 14:54 ` Russ Anderson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox