Commit 983e0e4e authored by Pauli Virtanen's avatar Pauli Virtanen Committed by Luiz Augusto von Dentz
Browse files

net-timestamp: COMPLETION timestamp on packet tx completion 



Add SOF_TIMESTAMPING_TX_COMPLETION, for requesting a software timestamp
when hardware reports a packet completed.

Completion tstamp is useful for Bluetooth, as hardware timestamps do not
exist in the HCI specification except for ISO packets, and the hardware
has a queue where packets may wait.  In this case the software SND
timestamp only reflects the kernel-side part of the total latency
(usually small) and queue length (usually 0 unless HW buffers
congested), whereas the completion report time is more informative of
the true latency.

It may also be useful in other cases where HW TX timestamps cannot be
obtained and user wants to estimate an upper bound to when the TX
probably happened.

Signed-off-by: default avatarPauli Virtanen <pav@iki.fi>
Reviewed-by: default avatarWillem de Bruijn <willemb@google.com>
Signed-off-by: default avatarLuiz Augusto von Dentz <luiz.von.dentz@intel.com>
parent b257e02e

Documentation/networking/timestamping.rst

+8 −0
Original line number Diff line number Diff line
@@ -140,6 +140,14 @@ SOF_TIMESTAMPING_TX_ACK: 
  cumulative acknowledgment. The mechanism ignores SACK and FACK.

  This flag can be enabled via both socket options and control messages.



SOF_TIMESTAMPING_TX_COMPLETION:

  Request tx timestamps on packet tx completion.  The completion

  timestamp is generated by the kernel when it receives packet a

  completion report from the hardware. Hardware may report multiple

  packets at once, and completion timestamps reflect the timing of the

  report and not actual tx time. This flag can be enabled via both

  socket options and control messages.





1.3.2 Timestamp Reporting

^^^^^^^^^^^^^^^^^^^^^^^^^

include/linux/skbuff.h

+4 −3
Original line number Diff line number Diff line
@@ -478,8 +478,8 @@ enum { 
	/* device driver is going to provide hardware time stamp */

	SKBTX_IN_PROGRESS = 1 << 2,



	/* reserved */

	SKBTX_RESERVED = 1 << 3,

	/* generate software time stamp on packet tx completion */

	SKBTX_COMPLETION_TSTAMP = 1 << 3,



	/* generate wifi status information (where possible) */

	SKBTX_WIFI_STATUS = 1 << 4,
@@ -498,7 +498,8 @@ enum { 


#define SKBTX_ANY_SW_TSTAMP	(SKBTX_SW_TSTAMP    | \

				 SKBTX_SCHED_TSTAMP | \

				 SKBTX_BPF)

				 SKBTX_BPF          | \

				 SKBTX_COMPLETION_TSTAMP)

#define SKBTX_ANY_TSTAMP	(SKBTX_HW_TSTAMP | \

				 SKBTX_ANY_SW_TSTAMP)

include/uapi/linux/errqueue.h

+1 −0
Original line number Diff line number Diff line
@@ -73,6 +73,7 @@ enum { 
	SCM_TSTAMP_SND,		/* driver passed skb to NIC, or HW */

	SCM_TSTAMP_SCHED,	/* data entered the packet scheduler */

	SCM_TSTAMP_ACK,		/* data acknowledged by peer */

	SCM_TSTAMP_COMPLETION,	/* packet tx completion */

};



#endif /* _UAPI_LINUX_ERRQUEUE_H */

include/uapi/linux/net_tstamp.h

+4 −2
Original line number Diff line number Diff line
@@ -44,8 +44,9 @@ enum { 
	SOF_TIMESTAMPING_BIND_PHC = (1 << 15),

	SOF_TIMESTAMPING_OPT_ID_TCP = (1 << 16),

	SOF_TIMESTAMPING_OPT_RX_FILTER = (1 << 17),

	SOF_TIMESTAMPING_TX_COMPLETION = (1 << 18),



	SOF_TIMESTAMPING_LAST = SOF_TIMESTAMPING_OPT_RX_FILTER,

	SOF_TIMESTAMPING_LAST = SOF_TIMESTAMPING_TX_COMPLETION,

	SOF_TIMESTAMPING_MASK = (SOF_TIMESTAMPING_LAST - 1) |

				 SOF_TIMESTAMPING_LAST

};
@@ -58,7 +59,8 @@ enum { 
#define SOF_TIMESTAMPING_TX_RECORD_MASK	(SOF_TIMESTAMPING_TX_HARDWARE | \

					 SOF_TIMESTAMPING_TX_SOFTWARE | \

					 SOF_TIMESTAMPING_TX_SCHED | \

					 SOF_TIMESTAMPING_TX_ACK)

					 SOF_TIMESTAMPING_TX_ACK | \

					 SOF_TIMESTAMPING_TX_COMPLETION)



/**

 * struct so_timestamping - SO_TIMESTAMPING parameter

net/core/skbuff.c

+2 −0
Original line number Diff line number Diff line
@@ -5523,6 +5523,8 @@ static bool skb_tstamp_tx_report_so_timestamping(struct sk_buff *skb, 
						    SKBTX_SW_TSTAMP);

	case SCM_TSTAMP_ACK:

		return TCP_SKB_CB(skb)->txstamp_ack & TSTAMP_ACK_SK;

	case SCM_TSTAMP_COMPLETION:

		return skb_shinfo(skb)->tx_flags & SKBTX_COMPLETION_TSTAMP;

	}



	return false;