mesh_plink: use msecs_to_jiffies for proper time conversion
authorNicholas Mc Guire <hofrat@osadl.org>
Mon, 2 Mar 2015 09:54:13 +0000 (04:54 -0500)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 4 Mar 2015 09:34:13 +0000 (10:34 +0100)
This is primarily an API consolidation and should make things more readable
it replaces var * HZ / 1000 by msecs_to_jiffies(var) which also handles
corner cases correctly.

There is a change of behavior as e.g. for HZ 100, t * HZ / 1000 will
return 0 for t < 10 but msecs_to_jiffies will return at least 1 always.

Signed-off-by: Nicholas Mc Guire <hofrat@osadl.org>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/mac80211/mesh_plink.c

index b488e1859b18e8ed7797cffbb5ab2319138fdb28..4eefd5df5b05cb2fbd808653b076d30ee79b708c 100644 (file)
@@ -17,7 +17,7 @@
 #define PLINK_GET_PLID(p) (p + 4)
 
 #define mod_plink_timer(s, t) (mod_timer(&s->plink_timer, \
-                               jiffies + HZ * t / 1000))
+                               jiffies + msecs_to_jiffies(t)))
 
 enum plink_event {
        PLINK_UNDEFINED,
@@ -623,7 +623,7 @@ static void mesh_plink_timer(unsigned long data)
 
 static inline void mesh_plink_timer_set(struct sta_info *sta, int timeout)
 {
-       sta->plink_timer.expires = jiffies + (HZ * timeout / 1000);
+       sta->plink_timer.expires = jiffies + msecs_to_jiffies(timeout);
        sta->plink_timer.data = (unsigned long) sta;
        sta->plink_timer.function = mesh_plink_timer;
        sta->plink_timeout = timeout;