net: Add some clarification to skb_tx_timestamp() comment.
authorDavid S. Miller <davem@davemloft.net>
Fri, 27 Dec 2013 18:04:33 +0000 (13:04 -0500)
committerDavid S. Miller <davem@davemloft.net>
Fri, 27 Dec 2013 18:04:33 +0000 (13:04 -0500)
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>
include/linux/skbuff.h

index 6aae8389052007b2ef49c37298efca836dc4cada..6f69b3f914fbf94c2d6d42beea62748a29a42d08 100644 (file)
@@ -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)