Ticket #1894: pr1894.diff

cpukit/score/src/coretodmsecstoticks.c

@@ -1 @@
-/*  COPYRIGHT (c) 1989-2008.
+/*  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -21 +21 @@
   uint32_t milliseconds
 )
 {
-  return (milliseconds / rtems_configuration_get_milliseconds_per_tick());
+  uint32_t ticks;
+  uint32_t milliseconds_per_tick;
+
+  /**
+   *  We should ensure the ticks not be truncated by integer division.  We
+   *  need to have it be greater than or equal to the requested time.  It
+   *  should not be shorter.
+   */
+  milliseconds_per_tick = rtems_configuration_get_milliseconds_per_tick();
+  ticks                 = milliseconds / milliseconds_per_tick;
+  if ( (milliseconds % milliseconds_per_tick) != 0 )
+    ticks += 1;
+
+  return ticks;
 }

cpukit/score/src/coretodusectoticks.c

@@ -1 @@
-/*  COPYRIGHT (c) 1989-2008.
+/*  COPYRIGHT (c) 1989-2011.
  *  On-Line Applications Research Corporation (OAR).
  *
  *  The license and distribution terms for this file may be
@@ -21 +21 @@
   uint32_t microseconds
 )
 {
-  return (microseconds / rtems_configuration_get_microseconds_per_tick());
+  uint32_t ticks;
+  uint32_t microseconds_per_tick;
+
+  /**
+   *  We should ensure the ticks not be truncated by integer division.  We
+   *  need to have it be greater than or equal to the requested time.  It
+   *  should not be shorter.
+   */
+  microseconds_per_tick = rtems_configuration_get_microseconds_per_tick();
+  ticks                 = microseconds / microseconds_per_tick;
+  if ( (microseconds % microseconds_per_tick) != 0 )
+    ticks += 1;
+
+  return ticks;
 }

cpukit/score/src/timespectoticks.c

@@ -35 +35 @@
 )
 {
   uint32_t  ticks;
+  uint32_t  nanoseconds_per_tick;
 
   if ( (time->tv_sec == 0) && (time->tv_nsec == 0) )
     return 0;
 
-  ticks  = time->tv_sec * TOD_TICKS_PER_SECOND;
+  /**
+   *  We should ensure the ticks not be truncated by integer division.  We
+   *  need to have it be greater than or equal to the requested time.  It
+   *  should not be shorter.
+   */
+  ticks                 = time->tv_sec * TOD_TICKS_PER_SECOND;
+  nanoseconds_per_tick  = rtems_configuration_get_nanoseconds_per_tick();
+  ticks                += time->tv_nsec / nanoseconds_per_tick;
 
-  ticks += time->tv_nsec / rtems_configuration_get_nanoseconds_per_tick();
+  if ( (time->tv_nsec % nanoseconds_per_tick) != 0 )
+    ticks += 1;
 
-  if (ticks)
-    return ticks;
-
-  return 1;
+  return ticks;
 }