Commit 57a800d4 authored by Andreas Henriksson's avatar Andreas Henriksson Committed by Stephen Hemminger

Switch helpers tc_core_{time2ktime,ktime2time} from long to unsigned as well.

Follow up patch to "Fix overflow in time2tick / tick2time." which switches
the remaining two helper functions from long to unsigned as well.
These functions are only used in "tc/q_hfsc.c" where both the passed argument
and the place the return value is stored are unsigned/u32 variables, so this
change should be safe to make but hasn't been tested as extensively as the
time2tick patch.
Signed-off-by: default avatarAndreas Henriksson <andreas@fatal.se>
Signed-off-by: default avatarStephen Hemminger <shemminger@linux-foundation.org>
parent ca78b0e7
...@@ -45,12 +45,12 @@ unsigned tc_core_tick2time(unsigned tick) ...@@ -45,12 +45,12 @@ unsigned tc_core_tick2time(unsigned tick)
return tick/tick_in_usec; return tick/tick_in_usec;
} }
long tc_core_time2ktime(long time) unsigned tc_core_time2ktime(unsigned time)
{ {
return time * clock_factor; return time * clock_factor;
} }
long tc_core_ktime2time(long ktime) unsigned tc_core_ktime2time(unsigned ktime)
{ {
return ktime / clock_factor; return ktime / clock_factor;
} }
......
...@@ -9,8 +9,8 @@ ...@@ -9,8 +9,8 @@
int tc_core_time2big(long time); int tc_core_time2big(long time);
unsigned tc_core_time2tick(unsigned time); unsigned tc_core_time2tick(unsigned time);
unsigned tc_core_tick2time(unsigned tick); unsigned tc_core_tick2time(unsigned tick);
long tc_core_time2ktime(long time); unsigned tc_core_time2ktime(unsigned time);
long tc_core_ktime2time(long ktime); unsigned tc_core_ktime2time(unsigned ktime);
unsigned tc_calc_xmittime(unsigned rate, unsigned size); unsigned tc_calc_xmittime(unsigned rate, unsigned size);
unsigned tc_calc_xmitsize(unsigned rate, unsigned ticks); unsigned tc_calc_xmitsize(unsigned rate, unsigned ticks);
int tc_calc_rtable(unsigned bps, __u32 *rtab, int cell_log, unsigned mtu, unsigned mpu); int tc_calc_rtable(unsigned bps, __u32 *rtab, int cell_log, unsigned mtu, unsigned mpu);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment