Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1675721 > unrolled thread

[PATCH net-next 0/3] fix sw timestamping for non PTP packets

Back to article view | Back to linux.kernel

Contents

  [PATCH net-next 0/3] fix sw timestamping for non PTP packets Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> - 2017-06-27 16:00 +0200
    [PATCH net-next 3/3] net: ethernet: ti: netcp_ethss: use cpts to check if packet needs timestamping Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> - 2017-06-27 16:00 +0200
    [PATCH net-next 1/3] net: ethernet: ti: cpsw: move skb timestamp to packet_submit Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org> - 2017-06-27 16:10 +0200
    Re: [PATCH net-next 0/3] fix sw timestamping for non PTP packets Grygorii Strashko <grygorii.strashko@ti.com> - 2017-06-28 23:30 +0200
    Re: [PATCH net-next 0/3] fix sw timestamping for non PTP packets David Miller <davem@davemloft.net> - 2017-06-29 18:30 +0200

#1675721 — [PATCH net-next 0/3] fix sw timestamping for non PTP packets

This series contains several corrections connected with timestamping
for cpsw and netcp drivers based on same cpts module.

Based on net/next

Ivan Khoronzhuk (3):
  net: ethernet: ti: cpsw: move skb timestamp to packet_submit
  net: ethernet: ti: cpsw: fix sw timestamping for non PTP packets
  net: ethernet: ti: netcp_ethss: use cpts to check if packet needs
    timestamping

 drivers/net/ethernet/ti/cpsw.c        |  6 +++---
 drivers/net/ethernet/ti/cpts.h        | 16 ++++++++++++++++
 drivers/net/ethernet/ti/netcp_ethss.c | 18 +-----------------
 3 files changed, 20 insertions(+), 20 deletions(-)

-- 
2.7.4

[toc] | [next] | [standalone]

#1675726 — [PATCH net-next 3/3] net: ethernet: ti: netcp_ethss: use cpts to check if packet needs timestamping

There is cpts function to check if packet can be timstamped with cpts.
Seems that ptp_classify_raw cover all cases listed with "case".

Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
---
 drivers/net/ethernet/ti/netcp_ethss.c | 18 +-----------------
 1 file changed, 1 insertion(+), 17 deletions(-)

diff --git a/drivers/net/ethernet/ti/netcp_ethss.c b/drivers/net/ethernet/ti/netcp_ethss.c
index 0847a8f..28cb38a 100644
--- a/drivers/net/ethernet/ti/netcp_ethss.c
+++ b/drivers/net/ethernet/ti/netcp_ethss.c
@@ -2503,24 +2503,8 @@ static bool gbe_need_txtstamp(struct gbe_intf *gbe_intf,
 			      const struct netcp_packet *p_info)
 {
 	struct sk_buff *skb = p_info->skb;
-	unsigned int class = ptp_classify_raw(skb);
 
-	if (class == PTP_CLASS_NONE)
-		return false;
-
-	switch (class) {
-	case PTP_CLASS_V1_IPV4:
-	case PTP_CLASS_V1_IPV6:
-	case PTP_CLASS_V2_IPV4:
-	case PTP_CLASS_V2_IPV6:
-	case PTP_CLASS_V2_L2:
-	case (PTP_CLASS_V2_VLAN | PTP_CLASS_L2):
-	case (PTP_CLASS_V2_VLAN | PTP_CLASS_IPV4):
-	case (PTP_CLASS_V2_VLAN | PTP_CLASS_IPV6):
-		return true;
-	}
-
-	return false;
+	return cpts_can_timestamp(gbe_intf->gbe_dev->cpts, skb);
 }
 
 static int gbe_txtstamp_mark_pkt(struct gbe_intf *gbe_intf,
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1675730 — [PATCH net-next 1/3] net: ethernet: ti: cpsw: move skb timestamp to packet_submit

Move sw timestamp function close to channel submit function.

Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
---
 drivers/net/ethernet/ti/cpsw.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/net/ethernet/ti/cpsw.c b/drivers/net/ethernet/ti/cpsw.c
index b7a0f5e..422994e 100644
--- a/drivers/net/ethernet/ti/cpsw.c
+++ b/drivers/net/ethernet/ti/cpsw.c
@@ -1236,6 +1236,7 @@ static inline int cpsw_tx_packet_submit(struct cpsw_priv *priv,
 {
 	struct cpsw_common *cpsw = priv->cpsw;
 
+	skb_tx_timestamp(skb);
 	return cpdma_chan_submit(txch, skb, skb->data, skb->len,
 				 priv->emac_port + cpsw->data.dual_emac);
 }
@@ -1611,8 +1612,6 @@ static netdev_tx_t cpsw_ndo_start_xmit(struct sk_buff *skb,
 	    cpts_is_tx_enabled(cpsw->cpts))
 		skb_shinfo(skb)->tx_flags |= SKBTX_IN_PROGRESS;
 
-	skb_tx_timestamp(skb);
-
 	q_idx = skb_get_queue_mapping(skb);
 	if (q_idx >= cpsw->tx_ch_num)
 		q_idx = q_idx % cpsw->tx_ch_num;
-- 
2.7.4

[toc] | [prev] | [next] | [standalone]

#1677196

On 06/27/2017 08:58 AM, Ivan Khoronzhuk wrote:
> This series contains several corrections connected with timestamping
> for cpsw and netcp drivers based on same cpts module.
> 
> Based on net/next

Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>

> 
> Ivan Khoronzhuk (3):
>    net: ethernet: ti: cpsw: move skb timestamp to packet_submit
>    net: ethernet: ti: cpsw: fix sw timestamping for non PTP packets
>    net: ethernet: ti: netcp_ethss: use cpts to check if packet needs
>      timestamping
> 
>   drivers/net/ethernet/ti/cpsw.c        |  6 +++---
>   drivers/net/ethernet/ti/cpts.h        | 16 ++++++++++++++++
>   drivers/net/ethernet/ti/netcp_ethss.c | 18 +-----------------
>   3 files changed, 20 insertions(+), 20 deletions(-)
> 

-- 
regards,
-grygorii

[toc] | [prev] | [next] | [standalone]

#1677931

From: Ivan Khoronzhuk <ivan.khoronzhuk@linaro.org>
Date: Tue, 27 Jun 2017 16:58:50 +0300

> This series contains several corrections connected with timestamping
> for cpsw and netcp drivers based on same cpts module.
> 
> Based on net/next

Series applied.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web