[Top] [All Lists]

Re: [PATCH] OCTEON: workaround linking failures with gcc-4.4.x 32-bits t

To: Florian Fainelli <>,
Subject: Re: [PATCH] OCTEON: workaround linking failures with gcc-4.4.x 32-bits toolchains
From: David Daney <>
Date: Wed, 28 Jul 2010 15:15:40 -0700
In-reply-to: <>
Original-recipient: rfc822;
References: <>
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv: Gecko/20100430 Fedora/3.0.4-2.fc12 Thunderbird/3.0.4
On 07/28/2010 03:13 PM, Florian Fainelli wrote:
When building with a gcc-4.4.x toolchain that is configured to produce 32-bits
executables by default, we will produce __lshrti3 in sched_clock() which is
never resolved so the kernel fails to link. Unconditionally use the inline
assembly version as suggested by David Daney, which works around the issue.

CC: David Daney<>
Signed-off-by: Florian Fainelli<>

Acked-by: David Daney <>

diff --git a/arch/mips/cavium-octeon/csrc-octeon.c 
index 0bf4bbe..36400d2 100644
--- a/arch/mips/cavium-octeon/csrc-octeon.c
+++ b/arch/mips/cavium-octeon/csrc-octeon.c
@@ -53,7 +53,6 @@ static struct clocksource clocksource_mips = {
  unsigned long long notrace sched_clock(void)
        /* 64-bit arithmatic can overflow, so use 128-bit.  */
-#if (__GNUC__<  4) || ((__GNUC__ == 4)&&  (__GNUC_MINOR__<= 3))
        u64 t1, t2, t3;
        unsigned long long rv;
        u64 mult = clocksource_mips.mult;
@@ -73,13 +72,6 @@ unsigned long long notrace sched_clock(void)
                : [cnt] "r" (cnt), [mult] "r" (mult), [shift] "r" (shift)
                : "hi", "lo");
        return rv;
-       /* GCC>  4.3 do it the easy way.  */
-       unsigned int __attribute__((mode(TI))) t;
-       t = read_c0_cvmcount();
-       t = t * clocksource_mips.mult;
-       return (unsigned long long)(t>>  clocksource_mips.shift);

  void __init plat_time_init(void)

<Prev in Thread] Current Thread [Next in Thread>