diff options
| author | David S. Miller <davem@davemloft.net> | 2013-12-27 13:04:33 -0500 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-12-27 13:04:33 -0500 |
| commit | 73409f3b0ff0ae7bc1f647936b23e6d5d5dcbe28 (patch) | |
| tree | 51dbcef70dc3b374e6e720aa5fde10f783301277 | |
| parent | 37ec274e9713eafc2ba6c4471420f06cb8f68ecf (diff) | |
net: Add some clarification to skb_tx_timestamp() comment.
We've seen so many instances of people invoking skb_tx_timestamp()
after the device already has been given the packet, that it's worth
being a little bit more verbose and explicit in this comment.
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | include/linux/skbuff.h | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/include/linux/skbuff.h b/include/linux/skbuff.h index 6aae8389052..6f69b3f914f 100644 --- a/include/linux/skbuff.h +++ b/include/linux/skbuff.h @@ -2531,6 +2531,10 @@ static inline void sw_tx_timestamp(struct sk_buff *skb) * Ethernet MAC Drivers should call this function in their hard_xmit() * function immediately before giving the sk_buff to the MAC hardware. * + * Specifically, one should make absolutely sure that this function is + * called before TX completion of this packet can trigger. Otherwise + * the packet could potentially already be freed. + * * @skb: A socket buffer. */ static inline void skb_tx_timestamp(struct sk_buff *skb) |