time: improve time datastructure
authorNathan Binkert <nate@binkert.org>
Sat, 15 Jan 2011 15:48:25 +0000 (07:48 -0800)
committerNathan Binkert <nate@binkert.org>
Sat, 15 Jan 2011 15:48:25 +0000 (07:48 -0800)
Use posix clock functions (and librt) if it is available.
Inline a bunch of functions and implement more operators.
* * *
time: more cleanup

SConstruct
src/base/time.cc
src/base/time.hh
src/sim/stat_control.cc

index 18ebea2afcaf214962c5a9e0bcbda90b75e21123..6127113b6706cd5fe1425c2679e280ccc552236e 100755 (executable)
@@ -703,6 +703,13 @@ if not conf.CheckLibWithHeader('z', 'zlib.h', 'C++','zlibVersion();'):
     print '       Please install zlib and try again.'
     Exit(1)
 
+# Check for librt.
+have_posix_clock = conf.CheckLib(None, 'clock_nanosleep', 'time.h') or \
+    conf.CheckLib('rt', 'clock_nanosleep', 'time.h')
+
+if not have_posix_clock:
+    print "Can't find library for POSIX clocks."
+
 # Check for <fenv.h> (C99 FP environment control)
 have_fenv = conf.CheckHeader('fenv.h', '<>')
 if not have_fenv:
@@ -819,6 +826,7 @@ sticky_vars.AddVariables(
                  'Compile for SSE2 (-msse2) to get IEEE FP on x86 hosts',
                  False),
     BoolVariable('USE_MYSQL', 'Use MySQL for stats output', have_mysql),
+    BoolVariable('USE_POSIX_CLOCK', 'Use POSIX Clocks', have_posix_clock),
     BoolVariable('USE_FENV', 'Use <fenv.h> IEEE mode control', have_fenv),
     BoolVariable('USE_CHECKER', 'Use checker for detailed CPU models', False),
     BoolVariable('CP_ANNOTATE', 'Enable critical path annotation capability', False),
@@ -828,7 +836,8 @@ sticky_vars.AddVariables(
 # These variables get exported to #defines in config/*.hh (see src/SConscript).
 export_vars += ['FULL_SYSTEM', 'USE_FENV', 'USE_MYSQL',
                 'NO_FAST_ALLOC', 'FAST_ALLOC_DEBUG', 'FAST_ALLOC_STATS',
-                'SS_COMPATIBLE_FP', 'USE_CHECKER', 'TARGET_ISA', 'CP_ANNOTATE']
+                'SS_COMPATIBLE_FP', 'USE_CHECKER', 'TARGET_ISA', 'CP_ANNOTATE',
+                'USE_POSIX_CLOCK' ]
 
 ###################################################
 #
index a1732773e026a0aa4bc066e1e0b788fde8598202..b9bbb083020621b8dfe0a0375e6003346b37d924 100644 (file)
  * Authors: Nathan Binkert
  */
 
-#include <cctype>
-#include <cstring>
-#include <ctime>
 #include <iostream>
-#include <string>
+#include <sstream>
 
 #include "base/time.hh"
+#include "config/use_posix_clock.hh"
 
 using namespace std;
 
-struct _timeval
-{
-    timeval tv;
-};
-
-double
-convert(const timeval &tv)
-{
-    return (double)tv.tv_sec + (double)tv.tv_usec / 1000000.0;
-}
-
-Time::Time(bool set_now)
-{
-    time = new _timeval;
-    if (set_now)
-        set();
-}
-
-Time::Time(const timeval &val)
-{
-    time = new _timeval;
-    set(val);
-}
-
-Time::Time(const Time &val)
-{
-    time = new _timeval;
-    set(val.get());
-}
-
-Time::~Time()
-{
-    delete time;
-}
-
-const timeval &
-Time::get() const
-{
-    return time->tv;
-}
-
 void
-Time::set()
+Time::_set(bool monotonic)
 {
-    ::gettimeofday(&time->tv, NULL);
-}
-
-void
-Time::set(const timeval &tv)
-{
-    memcpy(&time->tv, &tv, sizeof(timeval));
-}
-
-double
-Time::operator()() const
-{
-    return convert(get());
+#if USE_POSIX_CLOCK
+    ::clock_gettime(monotonic ? CLOCK_MONOTONIC : CLOCK_REALTIME, &_time);
+#else
+    timeval tv;
+    ::gettimeofday(&tv, NULL);
+    operator=(tv);
+#endif
 }
 
 string
-Time::date(string format) const
+Time::date(const string &format) const
 {
-    const timeval &tv = get();
-    time_t sec = tv.tv_sec;
+    time_t sec = this->sec();
     char buf[256];
 
     if (format.empty()) {
 #ifdef __SUNPRO_CC
-        ctime_r(&sec, buf, 256);
+        ctime_r(&sec, buf, sizeof(buf));
 #else
         ctime_r(&sec, buf);
 #endif
@@ -119,19 +69,44 @@ Time::date(string format) const
     return buf;
 }
 
-ostream &
-operator<<(ostream &out, const Time &start)
+string
+Time::time() const
 {
-    out << start.date();
-    return out;
+    double time = double(*this);
+    double secs = fmod(time, 60.0);
+    double all_mins = floor(time / 60.0);
+    double mins = fmod(all_mins, 60.0);
+    double hours = floor(all_mins / 60.0);
+
+    stringstream str;
+
+    if (hours > 0.0) {
+        if (hours < 10.0)
+            str << '0';
+        str << hours << ':';
+    }
+
+    if (mins > 0.0) {
+        if (mins < 10.0)
+            str << '0';
+        str << mins << ':';
+    }
+
+    if (secs < 10.0 && !str.str().empty())
+        str << '0';
+    str << secs;
+
+    return str.str();
 }
 
-Time
-operator-(const Time &l, const Time &r)
+void
+sleep(const Time &time)
 {
-    timeval tv;
-    timersub(&l.get(), &r.get(), &tv);
-    return tv;
-}
+    timespec ts = time;
 
-const Time Time::start(true);
+#if USE_POSIX_CLOCK
+    clock_nanosleep(CLOCK_MONOTONIC, 0, &ts, NULL);
+#else
+    nanosleep(&ts, NULL);
+#endif
+}
index 565ea0aac63c11f38c7f3fb731498dafaac4b386..2c54f2675476523619b7d80122f4397599da1905 100644 (file)
  *          Nathan Binkert
  */
 
-#ifndef __SIM_TIME_HH__
-#define __SIM_TIME_HH__
+#ifndef __BASE_TIME_HH__
+#define __BASE_TIME_HH__
 
 #include <sys/time.h>
 
+#include <inttypes.h>
+
+#include <cmath>
+#include <cstring>
+#include <ctime>
 #include <iosfwd>
 #include <string>
 
-struct _timeval;
-
 class Time
 {
   protected:
-    mutable _timeval *time;
+    timespec _time;
+
+    /**
+     * Internal time set function
+     */
+    void _set(bool monotonic);
 
   public:
-    explicit Time(bool set_now = false);
-    Time(const timeval &val);
-    Time(const Time &val);
-    ~Time();
+    static const long NSEC_PER_SEC  = 1000 * 1000 * 1000;
+    static const long NSEC_PER_MSEC = 1000 * 1000;
+    static const long NSEC_PER_USEC = 1000;
 
-    void set();
-    const timeval &get() const;
-    void set(const timeval &val);
+  public:
+    explicit Time() { clear(); }
+    explicit Time(double sec) { operator=(sec); }
+    Time(const Time &val) : _time(val._time) { }
+    Time(uint64_t sec, uint64_t nsec) { set(sec, nsec); }
+    Time(const timeval &tv) { operator=(tv); }
+    Time(const timespec &ts) { operator=(ts); }
 
-    double operator()() const;
-    std::string date(std::string format = "") const;
+    /**
+     * Accessors for getting and setting the current clock
+     */
+    time_t sec() const { return _time.tv_sec; }
+    long msec() const { return _time.tv_nsec / NSEC_PER_MSEC; }
+    long usec() const { return _time.tv_nsec / NSEC_PER_USEC; }
+    long nsec() const { return _time.tv_nsec; }
 
-  public:
-    static const Time start;
+    void sec(time_t sec) { _time.tv_sec = sec; }
+    void msec(long msec) { _time.tv_nsec = msec * NSEC_PER_MSEC; }
+    void usec(long usec) { _time.tv_nsec = usec * NSEC_PER_USEC; }
+    void nsec(long nsec) { _time.tv_nsec = nsec; }
+
+    /**
+     * Clear the time
+     */
+    void clear() { memset(&_time, 0, sizeof(_time)); }
+
+    /**
+     * Use this to set time for the purposes of time measurement (use
+     * a monotonic clock if it is available
+     */
+    void setTimer() { _set(true); }
+
+    /**
+     * Use this to set the time to the actual current time
+     */
+    void setWallclock() { _set(false); }
+
+    /**
+     * Set the current time
+     */
+    void set(time_t _sec, long _nsec) { sec(_sec); nsec(_nsec); }
+
+    const Time &
+    operator=(const Time &other)
+    {
+        sec(other.sec());
+        nsec(other.nsec());
+        return *this;
+    }
+
+    const Time &
+    operator=(double new_time)
+    {
+        double seconds = floor(new_time);
+        sec((time_t)seconds);
+        nsec((long)((seconds - new_time) * 1e9));
+        return *this;
+    }
+
+    const Time &
+    operator=(const timeval &tv)
+    {
+        sec(tv.tv_sec);
+        nsec(tv.tv_usec * 1000);
+        return *this;
+    }
+
+    const Time &
+    operator=(const timespec &ts)
+    {
+        sec(ts.tv_sec);
+        nsec(ts.tv_nsec);
+        return *this;
+    }
+
+    /**
+     * Get the time in floating point seconds
+     */
+    operator double() const
+    {
+        return (double)sec() + ((double)nsec()) * 1e-9;
+    }
+
+    /**
+     * operators for time conversion
+     */
+    operator timespec() const { return _time; }
+    operator timeval() const
+    {
+        timeval tv;
+        tv.tv_sec = sec();
+        tv.tv_usec = usec();
+        return tv;
+    }
+
+    const Time &
+    operator+=(const Time &other)
+    {
+
+        _time.tv_sec += other.sec();
+        _time.tv_nsec += other.nsec();
+        if (_time.tv_nsec > NSEC_PER_SEC) {
+            _time.tv_sec++;
+            _time.tv_nsec -= NSEC_PER_SEC;
+        }
+
+        return *this;
+    }
+
+    const Time &
+    operator-=(const Time &other)
+    {
+        _time.tv_sec -= other.sec();
+        _time.tv_nsec -= other.nsec();
+        if (_time.tv_nsec < 0) {
+            _time.tv_sec--;
+            _time.tv_nsec += NSEC_PER_SEC;
+        }
+
+        return *this;
+    }
+
+    std::string date(const std::string &format = "") const;
+    std::string time() const;
 };
 
-Time operator-(const Time &l, const Time &r);
+void sleep(const Time &time);
 
-std::ostream &operator<<(std::ostream &out, const Time &time);
+inline bool
+operator==(const Time &l, const Time &r)
+{
+    return l.sec() == r.sec() && l.nsec() == r.nsec();
+}
 
+inline bool
+operator!=(const Time &l, const Time &r)
+{
+    return l.sec() != r.sec() || l.nsec() != r.nsec();
+}
 
-/*
- * Copyright (c) 1982, 1986, 1993
- *      The Regents of the University of California.  All rights reserved.
- *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- * 3. Neither the name of the University nor the names of its contributors
- *    may be used to endorse or promote products derived from this software
- *    without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE REGENTS AND CONTRIBUTORS ``AS IS'' AND
- * ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
- * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
- * ARE DISCLAIMED.  IN NO EVENT SHALL THE REGENTS OR CONTRIBUTORS BE LIABLE
- * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
- * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
- * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
- * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT
- * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY
- * OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF
- * SUCH DAMAGE.
- *
- *      @(#)time.h      8.2 (Berkeley) 7/10/94
- */
+inline bool
+operator<(const Time &l, const Time &r)
+{
+    return (l.sec() < r.sec()) ||
+        (l.sec() == r.sec() && l.nsec() < r.nsec());
+}
+
+inline bool
+operator<=(const Time &l, const Time &r)
+{
+    return (l.sec() < r.sec()) ||
+        (l.sec() == r.sec() && l.nsec() <= r.nsec());
+}
+
+inline bool
+operator>(const Time &l, const Time &r)
+{
+    return (l.sec() > r.sec()) ||
+        (l.sec() == r.sec() && l.nsec() > r.nsec());
+}
+
+inline bool
+operator>=(const Time &l, const Time &r)
+{
+    return (l.sec() > r.sec()) ||
+        (l.sec() == r.sec() && l.nsec() >= r.nsec());
+}
+
+inline Time
+operator+(const Time &l, const Time &r)
+{
+    Time time(l);
+    time += r;
+    return time;
+}
+
+inline Time
+operator-(const Time &l, const Time &r)
+{
+    Time time(l);
+    time -= r;
+    return time;
+}
+
+inline std::ostream &
+operator<<(std::ostream &out, const Time &time)
+{
+    out << time.date();
+    return out;
+}
 
-#if defined(__sun)
-#define timersub(tvp, uvp, vvp)                                         \
-    do {                                                            \
-            (vvp)->tv_sec = (tvp)->tv_sec - (uvp)->tv_sec; \
-            (vvp)->tv_usec = (tvp)->tv_usec - (uvp)->tv_usec;       \
-            if ((vvp)->tv_usec < 0) { \
-                (vvp)->tv_sec--; \
-                (vvp)->tv_usec += 1000000; \
-            } \
-    } while (0)
-#endif
-
-#endif // __SIM_TIME_HH__
+#endif // __BASE_TIME_HH__
index f2c7c8a2e37f384030ab93431db48937f7d5a916..ad1d4a9e412a904b8ea3032ed73bf2fef6d22eae 100644 (file)
@@ -65,7 +65,7 @@ struct SimTicksReset : public Callback
 {
     void process()
     {
-        statTime.set();
+        statTime.setTimer();
         startTick = curTick();
     }
 };
@@ -73,9 +73,11 @@ struct SimTicksReset : public Callback
 double
 statElapsedTime()
 {
-    Time now(true);
+    Time now;
+    now.setTimer();
+
     Time elapsed = now - statTime;
-    return elapsed();
+    return elapsed;
 }
 
 Tick