tipc: embed jiffies in macro TIPC_BC_RETR_LIM
authorJon Maloy <jon.maloy@ericsson.com>
Fri, 28 Jun 2019 15:06:20 +0000 (17:06 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 2 Jul 2019 02:10:57 +0000 (19:10 -0700)
The macro TIPC_BC_RETR_LIM is always used in combination with 'jiffies',
so we can just as well perform the addition in the macro itself. This
way, we get a few shorter code lines and one less line break.

Signed-off-by: Jon Maloy <jon.maloy@ericsson.com>
Acked-by: Ying Xue <ying.xue@windriver.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/tipc/link.c

index f8bf63befe1f5a8d544cf580f05b05c073656a16..66d3a07bc5711ff404332e2224d139ad71daaaee 100644 (file)
@@ -207,7 +207,7 @@ enum {
        BC_NACK_SND_SUPPRESS,
 };
 
-#define TIPC_BC_RETR_LIM msecs_to_jiffies(10)   /* [ms] */
+#define TIPC_BC_RETR_LIM  (jiffies + msecs_to_jiffies(10))
 #define TIPC_UC_RETR_TIME (jiffies + msecs_to_jiffies(1))
 
 /*
@@ -976,8 +976,7 @@ int tipc_link_xmit(struct tipc_link *l, struct sk_buff_head *list,
                        __skb_queue_tail(transmq, skb);
                        /* next retransmit attempt */
                        if (link_is_bc_sndlink(l))
-                               TIPC_SKB_CB(skb)->nxt_retr =
-                                       jiffies + TIPC_BC_RETR_LIM;
+                               TIPC_SKB_CB(skb)->nxt_retr = TIPC_BC_RETR_LIM;
                        __skb_queue_tail(xmitq, _skb);
                        TIPC_SKB_CB(skb)->ackers = l->ackers;
                        l->rcv_unacked = 0;
@@ -1027,7 +1026,7 @@ static void tipc_link_advance_backlog(struct tipc_link *l,
                __skb_queue_tail(&l->transmq, skb);
                /* next retransmit attempt */
                if (link_is_bc_sndlink(l))
-                       TIPC_SKB_CB(skb)->nxt_retr = jiffies + TIPC_BC_RETR_LIM;
+                       TIPC_SKB_CB(skb)->nxt_retr = TIPC_BC_RETR_LIM;
 
                __skb_queue_tail(xmitq, _skb);
                TIPC_SKB_CB(skb)->ackers = l->ackers;
@@ -1123,7 +1122,7 @@ static int tipc_link_bc_retrans(struct tipc_link *l, struct tipc_link *r,
                if (link_is_bc_sndlink(l)) {
                        if (time_before(jiffies, TIPC_SKB_CB(skb)->nxt_retr))
                                continue;
-                       TIPC_SKB_CB(skb)->nxt_retr = jiffies + TIPC_BC_RETR_LIM;
+                       TIPC_SKB_CB(skb)->nxt_retr = TIPC_BC_RETR_LIM;
                }
                _skb = __pskb_copy(skb, LL_MAX_HEADER + MIN_H_SIZE, GFP_ATOMIC);
                if (!_skb)