lkml.org 
[lkml]   [2008]   [Apr]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[PATCH 24/24] Use U64_C() instead of casts in kernel/time.c
Use the new U64_C() constructor macro instead of using casts for
constants in kernel/time.c. This avoids warnings with some gcc
versions.

Note that since pasting has priority over macro resolution, the header
file generation is modified to have suffixed versions of all the
generated values.

This resolves Bugzilla 10153.

Signed-off-by: H. Peter Anvin <hpa@zytor.com>
---
kernel/time.c | 8 ++++----
kernel/timeconst.pl | 21 ++++++++++++++++-----
2 files changed, 20 insertions(+), 9 deletions(-)

This is the corrected version of this patch. My apologies.
The git repository has also been corrected:

git://git.kernel.org/pub/scm/linux/kernel/git/hpa/linux-2.6-inttypes.git

diff --git a/kernel/time.c b/kernel/time.c
index 35d373a..652fd4f 100644
--- a/kernel/time.c
+++ b/kernel/time.c
@@ -244,7 +244,7 @@ unsigned int inline jiffies_to_msecs(const unsigned long j)
return (j + (HZ / MSEC_PER_SEC) - 1)/(HZ / MSEC_PER_SEC);
#else
# if BITS_PER_LONG == 32
- return ((u64)HZ_TO_MSEC_MUL32 * j) >> HZ_TO_MSEC_SHR32;
+ return (U64_C(HZ_TO_MSEC_MUL32) * j) >> HZ_TO_MSEC_SHR32;
# else
return (j * HZ_TO_MSEC_NUM) / HZ_TO_MSEC_DEN;
# endif
@@ -260,7 +260,7 @@ unsigned int inline jiffies_to_usecs(const unsigned long j)
return (j + (HZ / USEC_PER_SEC) - 1)/(HZ / USEC_PER_SEC);
#else
# if BITS_PER_LONG == 32
- return ((u64)HZ_TO_USEC_MUL32 * j) >> HZ_TO_USEC_SHR32;
+ return (U64_C(HZ_TO_USEC_MUL32) * j) >> HZ_TO_USEC_SHR32;
# else
return (j * HZ_TO_USEC_NUM) / HZ_TO_USEC_DEN;
# endif
@@ -470,7 +470,7 @@ unsigned long msecs_to_jiffies(const unsigned int m)
if (HZ > MSEC_PER_SEC && m > jiffies_to_msecs(MAX_JIFFY_OFFSET))
return MAX_JIFFY_OFFSET;

- return ((u64)MSEC_TO_HZ_MUL32 * m + MSEC_TO_HZ_ADJ32)
+ return (U64_C(MSEC_TO_HZ_MUL32) * m + U64_C(MSEC_TO_HZ_ADJ32))
>> MSEC_TO_HZ_SHR32;
#endif
}
@@ -485,7 +485,7 @@ unsigned long usecs_to_jiffies(const unsigned int u)
#elif HZ > USEC_PER_SEC && !(HZ % USEC_PER_SEC)
return u * (HZ / USEC_PER_SEC);
#else
- return ((u64)USEC_TO_HZ_MUL32 * u + USEC_TO_HZ_ADJ32)
+ return (U64_C(USEC_TO_HZ_MUL32) * u + U64_C(USEC_TO_HZ_ADJ32))
>> USEC_TO_HZ_SHR32;
#endif
}
diff --git a/kernel/timeconst.pl b/kernel/timeconst.pl
index 4146803..6212498 100644
--- a/kernel/timeconst.pl
+++ b/kernel/timeconst.pl
@@ -1,7 +1,7 @@
#!/usr/bin/perl
# -----------------------------------------------------------------------
#
-# Copyright 2007 rPath, Inc. - All Rights Reserved
+# Copyright 2007-2008 rPath, Inc. - All Rights Reserved
#
# This file is part of the Linux kernel, and is made available under
# the terms of the GNU General Public License version 2 or (at your
@@ -318,6 +318,19 @@ sub compute_values($) {
return @val;
}

+# Create defines for a value, including variants with all appropriate
+# suffixes. The suffixed versions also allow pasted constructs like
+# U64_C(HZ_TO_MSEC_MUL32) to work correctly.
+sub outputval($$)
+{
+ my($name, $val) = @_;
+ my $csuf;
+
+ foreach $csuf ('', 'U', 'L', 'UL', 'LL', 'ULL') {
+ printf "#define %-23s %s\n", $name.$csuf, $val.$csuf;
+ }
+}
+
sub output($@)
{
my($hz, @val) = @_;
@@ -342,13 +355,11 @@ sub output($@)
'HZ_TO_USEC','USEC_TO_HZ') {
foreach $bit (32, 64) {
foreach $suf ('MUL', 'ADJ', 'SHR') {
- printf "#define %-23s %s\n",
- "${pfx}_$suf$bit", shift(@val);
+ outputval("${pfx}_$suf$bit", shift(@val));
}
}
foreach $suf ('NUM', 'DEN') {
- printf "#define %-23s %s\n",
- "${pfx}_$suf", shift(@val);
+ outputval("${pfx}_$suf", shift(@val));
}
}

--
1.5.4.1


\
 
 \ /
  Last update: 2008-04-25 06:13    [W:0.706 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site