* asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace
@ 2008-05-10 12:36 Mike Frysinger
2008-05-14 11:05 ` maximilian attems
0 siblings, 1 reply; 8+ messages in thread
From: Mike Frysinger @ 2008-05-10 12:36 UTC (permalink / raw)
To: LKML, Andrew Morton; +Cc: chris, rth, ysato
I noticed this because alpha was broken due to the recent bdc807871d58285737.
Most arches do something like this in their asm/param.h:
#ifdef __KERNEL__
# define HZ CONFIG_HZ
#else
# define HZ 100
#endif
A few arches though (namely alpha/h8300/um/v850/xtensa) either do no set HZ at
all for !__KERNEL__, or they set it wrongly. This should bring all arches in
line by setting up HZ for userspace.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
diff --git a/include/asm-alpha/param.h b/include/asm-alpha/param.h
index 0982f1d..d300950 100644
--- a/include/asm-alpha/param.h
+++ b/include/asm-alpha/param.h
@@ -5,8 +5,12 @@
hardware ignores reprogramming. We also need userland buy-in to the
change in HZ, since this is visible in the wait4 resources etc. */
+#ifdef __KERNEL__
#define HZ CONFIG_HZ
#define USER_HZ HZ
+#else
+#define HZ 100
+#endif
#define EXEC_PAGESIZE 8192
diff --git a/include/asm-h8300/param.h b/include/asm-h8300/param.h
index 04f64f1..1c72fb8 100644
--- a/include/asm-h8300/param.h
+++ b/include/asm-h8300/param.h
@@ -1,14 +1,12 @@
#ifndef _H8300_PARAM_H
#define _H8300_PARAM_H
-
-#ifndef HZ
-#define HZ CONFIG_HZ
-#endif
-
#ifdef __KERNEL__
+#define HZ CONFIG_HZ
#define USER_HZ HZ
#define CLOCKS_PER_SEC (USER_HZ)
+#else
+#define HZ 100
#endif
#define EXEC_PAGESIZE 4096
diff --git a/include/asm-um/param.h b/include/asm-um/param.h
index 4cd4a22..e44f4e6 100644
--- a/include/asm-um/param.h
+++ b/include/asm-um/param.h
@@ -13,6 +13,8 @@
#define HZ CONFIG_HZ
#define USER_HZ 100 /* .. some user interfaces are in "ticks" */
#define CLOCKS_PER_SEC (USER_HZ) /* frequency at which times() counts */
+#else
+#define HZ 100
#endif
#endif
diff --git a/include/asm-v850/param.h b/include/asm-v850/param.h
index 2818326..4391f5f 100644
--- a/include/asm-v850/param.h
+++ b/include/asm-v850/param.h
@@ -26,6 +26,8 @@
# define HZ CONFIG_HZ
# define USER_HZ 100
# define CLOCKS_PER_SEC USER_HZ
+#else
+# define HZ 100
#endif
#endif /* __V850_PARAM_H__ */
diff --git a/include/asm-xtensa/param.h b/include/asm-xtensa/param.h
index 82ad34d..ba03d5a 100644
--- a/include/asm-xtensa/param.h
+++ b/include/asm-xtensa/param.h
@@ -15,6 +15,8 @@
# define HZ CONFIG_HZ /* internal timer frequency */
# define USER_HZ 100 /* for user interfaces in "ticks" */
# define CLOCKS_PER_SEC (USER_HZ) /* frequnzy at which times() counts */
+#else
+# define HZ 100
#endif
#define EXEC_PAGESIZE 4096
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace
2008-05-10 12:36 asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace Mike Frysinger
@ 2008-05-14 11:05 ` maximilian attems
2008-05-14 11:29 ` maximilian attems
` (2 more replies)
0 siblings, 3 replies; 8+ messages in thread
From: maximilian attems @ 2008-05-14 11:05 UTC (permalink / raw)
To: linux-kernel
Cc: H. Peter Anvin, akpm, Ivan Kokshaysky, Richard Henderson,
Mike Frysinger, debian-kernel
could we please get an ack or nack on that patch.
checked linux-next and it doesn't appear there.
without it currently perl 5.10 doesn't build on alpha:
perl.c: In function 'perl_construct':
perl.c:388: error: 'CONFIG_HZ' undeclared (first use in this function)
-> http://buildd.debian.org/fetch.cgi?pkg=perl;ver=5.10.0-10;arch=alpha;stamp=1210252894
--
maks
I noticed this because alpha was broken due to the recent bdc807871d58285737.
Most arches do something like this in their asm/param.h:
#ifdef __KERNEL__
# define HZ CONFIG_HZ
#else
# define HZ 100
#endif
A few arches though (namely alpha/h8300/um/v850/xtensa) either do no set HZ at
all for !__KERNEL__, or they set it wrongly. This should bring all arches in
line by setting up HZ for userspace.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
---
diff --git a/include/asm-alpha/param.h b/include/asm-alpha/param.h
index 0982f1d..d300950 100644
--- a/include/asm-alpha/param.h
+++ b/include/asm-alpha/param.h
@@ -5,8 +5,12 @@
hardware ignores reprogramming. We also need userland buy-in to the
change in HZ, since this is visible in the wait4 resources etc. */
+#ifdef __KERNEL__
#define HZ CONFIG_HZ
#define USER_HZ HZ
+#else
+#define HZ 100
+#endif
#define EXEC_PAGESIZE 8192
diff --git a/include/asm-h8300/param.h b/include/asm-h8300/param.h
index 04f64f1..1c72fb8 100644
--- a/include/asm-h8300/param.h
+++ b/include/asm-h8300/param.h
@@ -1,14 +1,12 @@
#ifndef _H8300_PARAM_H
#define _H8300_PARAM_H
-
-#ifndef HZ
-#define HZ CONFIG_HZ
-#endif
-
#ifdef __KERNEL__
+#define HZ CONFIG_HZ
#define USER_HZ HZ
#define CLOCKS_PER_SEC (USER_HZ)
+#else
+#define HZ 100
#endif
#define EXEC_PAGESIZE 4096
diff --git a/include/asm-um/param.h b/include/asm-um/param.h
index 4cd4a22..e44f4e6 100644
--- a/include/asm-um/param.h
+++ b/include/asm-um/param.h
@@ -13,6 +13,8 @@
#define HZ CONFIG_HZ
#define USER_HZ 100 /* .. some user interfaces are in "ticks" */
#define CLOCKS_PER_SEC (USER_HZ) /* frequency at which times() counts */
+#else
+#define HZ 100
#endif
#endif
diff --git a/include/asm-v850/param.h b/include/asm-v850/param.h
index 2818326..4391f5f 100644
--- a/include/asm-v850/param.h
+++ b/include/asm-v850/param.h
@@ -26,6 +26,8 @@
# define HZ CONFIG_HZ
# define USER_HZ 100
# define CLOCKS_PER_SEC USER_HZ
+#else
+# define HZ 100
#endif
#endif /* __V850_PARAM_H__ */
diff --git a/include/asm-xtensa/param.h b/include/asm-xtensa/param.h
index 82ad34d..ba03d5a 100644
--- a/include/asm-xtensa/param.h
+++ b/include/asm-xtensa/param.h
@@ -15,6 +15,8 @@
# define HZ CONFIG_HZ /* internal timer frequency */
# define USER_HZ 100 /* for user interfaces in "ticks" */
# define CLOCKS_PER_SEC (USER_HZ) /* frequnzy at which times() counts */
+#else
+# define HZ 100
#endif
#define EXEC_PAGESIZE 4096
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace
2008-05-14 11:05 ` maximilian attems
@ 2008-05-14 11:29 ` maximilian attems
2008-05-14 12:12 ` Mike Frysinger
2008-05-14 12:00 ` Mike Frysinger
2008-05-14 17:32 ` Andrew Morton
2 siblings, 1 reply; 8+ messages in thread
From: maximilian attems @ 2008-05-14 11:29 UTC (permalink / raw)
To: linux-kernel, H. Peter Anvin, akpm, Ivan Kokshaysky,
Richard Henderson, Mike Frysinger, debian-kernel
On Wed, May 14, 2008 at 01:05:31PM +0200, maximilian attems wrote:
> could we please get an ack or nack on that patch.
> checked linux-next and it doesn't appear there.
> without it currently perl 5.10 doesn't build on alpha:
> perl.c: In function 'perl_construct':
> perl.c:388: error: 'CONFIG_HZ' undeclared (first use in this function)
> -> http://buildd.debian.org/fetch.cgi?pkg=perl;ver=5.10.0-10;arch=alpha;stamp=1210252894
hmm it seems more userspace expects CONFIG_HZ, so it's not only a perl
build breakage..
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace
2008-05-14 11:05 ` maximilian attems
2008-05-14 11:29 ` maximilian attems
@ 2008-05-14 12:00 ` Mike Frysinger
2008-05-14 17:32 ` Andrew Morton
2 siblings, 0 replies; 8+ messages in thread
From: Mike Frysinger @ 2008-05-14 12:00 UTC (permalink / raw)
To: maximilian attems
Cc: linux-kernel, H. Peter Anvin, akpm, Ivan Kokshaysky,
Richard Henderson, debian-kernel
[-- Attachment #1: Type: text/plain, Size: 146 bytes --]
On Wednesday 14 May 2008, maximilian attems wrote:
> could we please get an ack or nack on that patch.
it's already been added to linux-mm
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 827 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace
2008-05-14 11:29 ` maximilian attems
@ 2008-05-14 12:12 ` Mike Frysinger
2008-05-14 12:32 ` maximilian attems
0 siblings, 1 reply; 8+ messages in thread
From: Mike Frysinger @ 2008-05-14 12:12 UTC (permalink / raw)
To: maximilian attems
Cc: linux-kernel, H. Peter Anvin, akpm, Ivan Kokshaysky,
Richard Henderson, debian-kernel
[-- Attachment #1: Type: text/plain, Size: 802 bytes --]
On Wednesday 14 May 2008, maximilian attems wrote:
> On Wed, May 14, 2008 at 01:05:31PM +0200, maximilian attems wrote:
> > could we please get an ack or nack on that patch.
> > checked linux-next and it doesn't appear there.
> > without it currently perl 5.10 doesn't build on alpha:
> > perl.c: In function 'perl_construct':
> > perl.c:388: error: 'CONFIG_HZ' undeclared (first use in this function)
> > ->
> > http://buildd.debian.org/fetch.cgi?pkg=perl;ver=5.10.0-10;arch=alpha;stam
> >p=1210252894
>
> hmm it seems more userspace expects CONFIG_HZ, so it's not only a perl
> build breakage..
anything using CONFIG_HZ is absolutely broken. go fix the package.
anything using HZ should be converted to sysconf(), but i'm not about to go on
that crusade to fix things up
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 827 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace
2008-05-14 12:12 ` Mike Frysinger
@ 2008-05-14 12:32 ` maximilian attems
2008-05-14 12:49 ` Mike Frysinger
0 siblings, 1 reply; 8+ messages in thread
From: maximilian attems @ 2008-05-14 12:32 UTC (permalink / raw)
To: Mike Frysinger
Cc: linux-kernel, H. Peter Anvin, akpm, Ivan Kokshaysky,
Richard Henderson, debian-kernel
On Wed, May 14, 2008 at 08:12:42AM -0400, Mike Frysinger wrote:
> On Wednesday 14 May 2008, maximilian attems wrote:
> > On Wed, May 14, 2008 at 01:05:31PM +0200, maximilian attems wrote:
> > > could we please get an ack or nack on that patch.
> > > checked linux-next and it doesn't appear there.
> > > without it currently perl 5.10 doesn't build on alpha:
> > > perl.c: In function 'perl_construct':
> > > perl.c:388: error: 'CONFIG_HZ' undeclared (first use in this function)
> > > ->
> > > http://buildd.debian.org/fetch.cgi?pkg=perl;ver=5.10.0-10;arch=alpha;stam
> > >p=1210252894
> >
> > hmm it seems more userspace expects CONFIG_HZ, so it's not only a perl
> > build breakage..
>
> anything using CONFIG_HZ is absolutely broken. go fix the package.
>
> anything using HZ should be converted to sysconf(), but i'm not about to go on
> that crusade to fix things up
news at 11
okay great that the patch went into mm.
--
maks
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace
2008-05-14 12:32 ` maximilian attems
@ 2008-05-14 12:49 ` Mike Frysinger
0 siblings, 0 replies; 8+ messages in thread
From: Mike Frysinger @ 2008-05-14 12:49 UTC (permalink / raw)
To: maximilian attems
Cc: linux-kernel, H. Peter Anvin, akpm, Ivan Kokshaysky,
Richard Henderson, debian-kernel
[-- Attachment #1: Type: text/plain, Size: 1297 bytes --]
On Wednesday 14 May 2008, maximilian attems wrote:
> On Wed, May 14, 2008 at 08:12:42AM -0400, Mike Frysinger wrote:
> > On Wednesday 14 May 2008, maximilian attems wrote:
> > > On Wed, May 14, 2008 at 01:05:31PM +0200, maximilian attems wrote:
> > > > could we please get an ack or nack on that patch.
> > > > checked linux-next and it doesn't appear there.
> > > > without it currently perl 5.10 doesn't build on alpha:
> > > > perl.c: In function 'perl_construct':
> > > > perl.c:388: error: 'CONFIG_HZ' undeclared (first use in this
> > > > function) ->
> > > > http://buildd.debian.org/fetch.cgi?pkg=perl;ver=5.10.0-10;arch=alpha;
> > > >stam p=1210252894
> > >
> > > hmm it seems more userspace expects CONFIG_HZ, so it's not only a perl
> > > build breakage..
> >
> > anything using CONFIG_HZ is absolutely broken. go fix the package.
> >
> > anything using HZ should be converted to sysconf(), but i'm not about to
> > go on that crusade to fix things up
>
> news at 11
>
> okay great that the patch went into mm.
i'm guessing you're thinking packages are using CONFIG_HZ when in reality they
arent. the asm-alpha/param.h in 2.6.25 does:
#define HZ CONFIG_HZ
so anything using HZ would end up with an error message about CONFIG_HZ being
undefined
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 827 bytes --]
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace
2008-05-14 11:05 ` maximilian attems
2008-05-14 11:29 ` maximilian attems
2008-05-14 12:00 ` Mike Frysinger
@ 2008-05-14 17:32 ` Andrew Morton
2 siblings, 0 replies; 8+ messages in thread
From: Andrew Morton @ 2008-05-14 17:32 UTC (permalink / raw)
To: maximilian attems
Cc: linux-kernel, H. Peter Anvin, Ivan Kokshaysky, Richard Henderson,
Mike Frysinger, debian-kernel
On Wed, 14 May 2008 13:05:31 +0200 maximilian attems <max@stro.at> wrote:
> without it currently perl 5.10 doesn't build on alpha:
> perl.c: In function 'perl_construct':
> perl.c:388: error: 'CONFIG_HZ' undeclared (first use in this function)
> -> http://buildd.debian.org/fetch.cgi?pkg=perl;ver=5.10.0-10;arch=alpha;stamp=1210252894
um, that was a fairly important piece of information which was cruelly
hidden from us - thanks.
I happened to have already planned on sending this fix in for 2.6.26
but it wasn't an obvious call.
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2008-05-14 17:34 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-05-10 12:36 asm-{alpha,h8300,um,v850,xtensa}/param.h: unbreak HZ for userspace Mike Frysinger
2008-05-14 11:05 ` maximilian attems
2008-05-14 11:29 ` maximilian attems
2008-05-14 12:12 ` Mike Frysinger
2008-05-14 12:32 ` maximilian attems
2008-05-14 12:49 ` Mike Frysinger
2008-05-14 12:00 ` Mike Frysinger
2008-05-14 17:32 ` Andrew Morton
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).