Commit e075996e authored by Jacob Keller's avatar Jacob Keller Committed by Jeff Kirsher

fm10k: move setting shinfo inside ts_tx_enqueue

This patch simplifies the code flow for setting the IN_PROGRESS bit of
the shinfo for an skb we will be timestamping.
Reported-by: default avatarEric Dumazet <eric.dumazet@gmail.com>
Signed-off-by: default avatarJacob Keller <jacob.e.keller@intel.com>
Tested-by: default avatarKrishneil Singh <Krishneil.k.singh@intel.com>
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
parent 608bb146
...@@ -70,16 +70,16 @@ void fm10k_ts_tx_enqueue(struct fm10k_intfc *interface, struct sk_buff *skb) ...@@ -70,16 +70,16 @@ void fm10k_ts_tx_enqueue(struct fm10k_intfc *interface, struct sk_buff *skb)
* if none are present then insert skb in tail of list * if none are present then insert skb in tail of list
*/ */
skb = fm10k_ts_tx_skb(interface, FM10K_CB(clone)->fi.w.dglort); skb = fm10k_ts_tx_skb(interface, FM10K_CB(clone)->fi.w.dglort);
if (!skb) if (!skb) {
skb_shinfo(clone)->tx_flags |= SKBTX_IN_PROGRESS;
__skb_queue_tail(list, clone); __skb_queue_tail(list, clone);
}
spin_unlock_irqrestore(&list->lock, flags); spin_unlock_irqrestore(&list->lock, flags);
/* if list is already has one then we just free the clone */ /* if list is already has one then we just free the clone */
if (skb) if (skb)
kfree_skb(skb); kfree_skb(skb);
else
skb_shinfo(clone)->tx_flags |= SKBTX_IN_PROGRESS;
} }
void fm10k_ts_tx_hwtstamp(struct fm10k_intfc *interface, __le16 dglort, void fm10k_ts_tx_hwtstamp(struct fm10k_intfc *interface, __le16 dglort,
......
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