timer: fix reloading after changes
authorVadim Suraev <vadim.suraev@gmail.com>
Wed, 21 May 2014 19:53:45 +0000 (22:53 +0300)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Mon, 26 May 2014 16:22:03 +0000 (18:22 +0200)
Bug: when a periodic timer's callback is running, if another
 timer is manipulated, the periodic timer is not reloaded.
Solution: set the update flag only if the modified timer is
 in RUNNING state

Signed-off-by: Vadim Suraev <vadim.suraev@gmail.com>
Acked-by: Olivier Matz <olivier.matz@6wind.com>
lib/librte_timer/rte_timer.c

index 884ee0e..d07232b 100755 (executable)
@@ -378,7 +378,9 @@ __rte_timer_reset(struct rte_timer *tim, uint64_t expire,
                return -1;
 
        __TIMER_STAT_ADD(reset, 1);
-       priv_timer[lcore_id].updated = 1;
+       if (prev_status.state == RTE_TIMER_RUNNING) {
+               priv_timer[lcore_id].updated = 1;
+       }
 
        /* remove it from list */
        if (prev_status.state == RTE_TIMER_PENDING) {
@@ -453,7 +455,9 @@ rte_timer_stop(struct rte_timer *tim)
                return -1;
 
        __TIMER_STAT_ADD(stop, 1);
-       priv_timer[lcore_id].updated = 1;
+       if (prev_status.state == RTE_TIMER_RUNNING) {
+               priv_timer[lcore_id].updated = 1;
+       }
 
        /* remove it from list */
        if (prev_status.state == RTE_TIMER_PENDING) {