Commit 79c57bff authored by David S. Miller's avatar David S. Miller
Browse files

Merge branch 'enetc-Support-extended-BD-rings-at-runtime' 



Claudiu Manoil says:

====================
enetc: Support extended BD rings at runtime

First two patches are just misc code cleanup.
The 3rd patch prepares the Rx BD processing code to be extended
to processing both normal and extended BDs.
The last one adds extended Rx BD support for timestamping
without the need of a static config. Finally, the config option
FSL_ENETC_HW_TIMESTAMPING can be dropped.
Care was taken not to impact non-timestamping usecases.
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 1e09e581 434cebab

drivers/net/ethernet/freescale/enetc/Kconfig

+0 −10
Original line number Diff line number Diff line
@@ -42,16 +42,6 @@ config FSL_ENETC_PTP_CLOCK 


	  If compiled as module (M), the module name is fsl-enetc-ptp.



config FSL_ENETC_HW_TIMESTAMPING

	bool "ENETC hardware timestamping support"

	depends on FSL_ENETC || FSL_ENETC_VF

	help

	  Enable hardware timestamping support on the Ethernet packets

	  using the SO_TIMESTAMPING API. Because the RX BD ring dynamic

	  allocation has not been supported and it is too expensive to use

	  extended RX BDs if timestamping is not used, this option enables

	  extended RX BDs in order to support hardware timestamping.



config FSL_ENETC_QOS

	bool "ENETC hardware Time-sensitive Network support"

	depends on (FSL_ENETC || FSL_ENETC_VF) && (NET_SCH_TAPRIO || NET_SCH_CBS)

drivers/net/ethernet/freescale/enetc/enetc.c

+39 −31
Original line number Diff line number Diff line
@@ -451,7 +451,7 @@ static int enetc_refill_rx_ring(struct enetc_bdr *rx_ring, const int buff_cnt) 


	i = rx_ring->next_to_use;

	rx_swbd = &rx_ring->rx_swbd[i];

	rxbd = ENETC_RXBD(*rx_ring, i);

	rxbd = enetc_rxbd(rx_ring, i);



	for (j = 0; j < buff_cnt; j++) {

		/* try reuse page */
@@ -468,13 +468,12 @@ static int enetc_refill_rx_ring(struct enetc_bdr *rx_ring, const int buff_cnt) 
		/* clear 'R" as well */

		rxbd->r.lstatus = 0;



		rxbd = enetc_rxbd_next(rx_ring, rxbd, i);

		rx_swbd++;

		rxbd++;

		i++;

		if (unlikely(i == rx_ring->bd_count)) {

			i = 0;

			rx_swbd = rx_ring->rx_swbd;

			rxbd = ENETC_RXBD(*rx_ring, 0);

		}

	}
@@ -488,7 +487,7 @@ static int enetc_refill_rx_ring(struct enetc_bdr *rx_ring, const int buff_cnt) 
	return j;

}



#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING

#ifdef CONFIG_FSL_ENETC_PTP_CLOCK

static void enetc_get_rx_tstamp(struct net_device *ndev,

				union enetc_rx_bd *rxbd,

				struct sk_buff *skb)
@@ -502,7 +501,8 @@ static void enetc_get_rx_tstamp(struct net_device *ndev, 
	if (le16_to_cpu(rxbd->r.flags) & ENETC_RXBD_FLAG_TSTMP) {

		lo = enetc_rd(hw, ENETC_SICTR0);

		hi = enetc_rd(hw, ENETC_SICTR1);

		tstamp_lo = le32_to_cpu(rxbd->r.tstamp);

		rxbd = enetc_rxbd_ext(rxbd);

		tstamp_lo = le32_to_cpu(rxbd->ext.tstamp);

		if (lo <= tstamp_lo)

			hi -= 1;
@@ -516,7 +516,7 @@ static void enetc_get_rx_tstamp(struct net_device *ndev, 
static void enetc_get_offloads(struct enetc_bdr *rx_ring,

			       union enetc_rx_bd *rxbd, struct sk_buff *skb)

{

#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING

#ifdef CONFIG_FSL_ENETC_PTP_CLOCK

	struct enetc_ndev_priv *priv = netdev_priv(rx_ring->ndev);

#endif

	/* TODO: hashing */
@@ -533,7 +533,7 @@ static void enetc_get_offloads(struct enetc_bdr *rx_ring, 
	if (le16_to_cpu(rxbd->r.flags) & ENETC_RXBD_FLAG_VLAN)

		__vlan_hwaccel_put_tag(skb, htons(ETH_P_8021Q),

				       le16_to_cpu(rxbd->r.vlan_opt));

#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING

#ifdef CONFIG_FSL_ENETC_PTP_CLOCK

	if (priv->active_offloads & ENETC_F_RX_TSTAMP)

		enetc_get_rx_tstamp(rx_ring->ndev, rxbd, skb);

#endif
@@ -655,7 +655,7 @@ static int enetc_clean_rx_ring(struct enetc_bdr *rx_ring, 
			cleaned_cnt -= count;

		}



		rxbd = ENETC_RXBD(*rx_ring, i);

		rxbd = enetc_rxbd(rx_ring, i);

		bd_status = le32_to_cpu(rxbd->r.lstatus);

		if (!bd_status)

			break;
@@ -670,12 +670,10 @@ static int enetc_clean_rx_ring(struct enetc_bdr *rx_ring, 
		enetc_get_offloads(rx_ring, rxbd, skb);



		cleaned_cnt++;

		rxbd++;

		i++;

		if (unlikely(i == rx_ring->bd_count)) {



		rxbd = enetc_rxbd_next(rx_ring, rxbd, i);

		if (unlikely(++i == rx_ring->bd_count))

			i = 0;

			rxbd = ENETC_RXBD(*rx_ring, 0);

		}



		if (unlikely(bd_status &

			     ENETC_RXBD_LSTATUS(ENETC_RXBD_ERR_MASK))) {
@@ -683,12 +681,10 @@ static int enetc_clean_rx_ring(struct enetc_bdr *rx_ring, 
			while (!(bd_status & ENETC_RXBD_LSTATUS_F)) {

				dma_rmb();

				bd_status = le32_to_cpu(rxbd->r.lstatus);

				rxbd++;

				i++;

				if (unlikely(i == rx_ring->bd_count)) {



				rxbd = enetc_rxbd_next(rx_ring, rxbd, i);

				if (unlikely(++i == rx_ring->bd_count))

					i = 0;

					rxbd = ENETC_RXBD(*rx_ring, 0);

				}

			}



			rx_ring->ndev->stats.rx_dropped++;
@@ -710,12 +706,10 @@ static int enetc_clean_rx_ring(struct enetc_bdr *rx_ring, 
			enetc_add_rx_buff_to_skb(rx_ring, i, size, skb);



			cleaned_cnt++;

			rxbd++;

			i++;

			if (unlikely(i == rx_ring->bd_count)) {



			rxbd = enetc_rxbd_next(rx_ring, rxbd, i);

			if (unlikely(++i == rx_ring->bd_count))

				i = 0;

				rxbd = ENETC_RXBD(*rx_ring, 0);

			}

		}



		rx_byte_cnt += skb->len;
@@ -845,15 +839,19 @@ static void enetc_free_tx_resources(struct enetc_ndev_priv *priv) 
		enetc_free_txbdr(priv->tx_ring[i]);

}



static int enetc_alloc_rxbdr(struct enetc_bdr *rxr)

static int enetc_alloc_rxbdr(struct enetc_bdr *rxr, bool extended)

{

	size_t size = sizeof(union enetc_rx_bd);

	int err;



	rxr->rx_swbd = vzalloc(rxr->bd_count * sizeof(struct enetc_rx_swbd));

	if (!rxr->rx_swbd)

		return -ENOMEM;



	err = enetc_dma_alloc_bdr(rxr, sizeof(union enetc_rx_bd));

	if (extended)

		size *= 2;



	err = enetc_dma_alloc_bdr(rxr, size);

	if (err) {

		vfree(rxr->rx_swbd);

		return err;
@@ -862,6 +860,7 @@ static int enetc_alloc_rxbdr(struct enetc_bdr *rxr) 
	rxr->next_to_clean = 0;

	rxr->next_to_use = 0;

	rxr->next_to_alloc = 0;

	rxr->ext_en = extended;



	return 0;

}
@@ -881,10 +880,11 @@ static void enetc_free_rxbdr(struct enetc_bdr *rxr) 


static int enetc_alloc_rx_resources(struct enetc_ndev_priv *priv)

{

	bool extended = !!(priv->active_offloads & ENETC_F_RX_TSTAMP);

	int i, err;



	for (i = 0; i < priv->num_rx_rings; i++) {

		err = enetc_alloc_rxbdr(priv->rx_ring[i]);

		err = enetc_alloc_rxbdr(priv->rx_ring[i], extended);



		if (err)

			goto fail;
@@ -1174,9 +1174,10 @@ static void enetc_setup_rxbdr(struct enetc_hw *hw, struct enetc_bdr *rx_ring) 
	enetc_rxbdr_wr(hw, idx, ENETC_RBICIR0, ENETC_RBICIR0_ICEN | 0x1);



	rbmr = ENETC_RBMR_EN;

#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING



	if (rx_ring->ext_en)

		rbmr |= ENETC_RBMR_BDS;

#endif



	if (rx_ring->ndev->features & NETIF_F_HW_VLAN_CTAG_RX)

		rbmr |= ENETC_RBMR_VTE;
@@ -1577,11 +1578,12 @@ int enetc_set_features(struct net_device *ndev, 
	return 0;

}



#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING

#ifdef CONFIG_FSL_ENETC_PTP_CLOCK

static int enetc_hwtstamp_set(struct net_device *ndev, struct ifreq *ifr)

{

	struct enetc_ndev_priv *priv = netdev_priv(ndev);

	struct hwtstamp_config config;

	int ao;



	if (copy_from_user(&config, ifr->ifr_data, sizeof(config)))

		return -EFAULT;
@@ -1597,6 +1599,7 @@ static int enetc_hwtstamp_set(struct net_device *ndev, struct ifreq *ifr) 
		return -ERANGE;

	}



	ao = priv->active_offloads;

	switch (config.rx_filter) {

	case HWTSTAMP_FILTER_NONE:

		priv->active_offloads &= ~ENETC_F_RX_TSTAMP;
@@ -1606,6 +1609,11 @@ static int enetc_hwtstamp_set(struct net_device *ndev, struct ifreq *ifr) 
		config.rx_filter = HWTSTAMP_FILTER_ALL;

	}



	if (netif_running(ndev) && ao != priv->active_offloads) {

		enetc_close(ndev);

		enetc_open(ndev);

	}



	return copy_to_user(ifr->ifr_data, &config, sizeof(config)) ?

	       -EFAULT : 0;

}
@@ -1632,7 +1640,7 @@ static int enetc_hwtstamp_get(struct net_device *ndev, struct ifreq *ifr) 


int enetc_ioctl(struct net_device *ndev, struct ifreq *rq, int cmd)

{

#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING

#ifdef CONFIG_FSL_ENETC_PTP_CLOCK

	if (cmd == SIOCSHWTSTAMP)

		return enetc_hwtstamp_set(ndev, rq);

	if (cmd == SIOCGHWTSTAMP)

drivers/net/ethernet/freescale/enetc/enetc.h

+32 −1
Original line number Diff line number Diff line
@@ -73,6 +73,7 @@ struct enetc_bdr { 


	dma_addr_t bd_dma_base;

	u8 tsd_enable; /* Time specific departure */

	bool ext_en; /* enable h/w descriptor extensions */

} ____cacheline_aligned_in_smp;



static inline void enetc_bdr_idx_inc(struct enetc_bdr *bdr, int *i)
@@ -104,7 +105,37 @@ struct enetc_cbdr { 
};



#define ENETC_TXBD(BDR, i) (&(((union enetc_tx_bd *)((BDR).bd_base))[i]))

#define ENETC_RXBD(BDR, i) (&(((union enetc_rx_bd *)((BDR).bd_base))[i]))



static inline union enetc_rx_bd *enetc_rxbd(struct enetc_bdr *rx_ring, int i)

{

	int hw_idx = i;



#ifdef CONFIG_FSL_ENETC_PTP_CLOCK

	if (rx_ring->ext_en)

		hw_idx = 2 * i;

#endif

	return &(((union enetc_rx_bd *)rx_ring->bd_base)[hw_idx]);

}



static inline union enetc_rx_bd *enetc_rxbd_next(struct enetc_bdr *rx_ring,

						 union enetc_rx_bd *rxbd,

						 int i)

{

	rxbd++;

#ifdef CONFIG_FSL_ENETC_PTP_CLOCK

	if (rx_ring->ext_en)

		rxbd++;

#endif

	if (unlikely(++i == rx_ring->bd_count))

		rxbd = rx_ring->bd_base;



	return rxbd;

}



static inline union enetc_rx_bd *enetc_rxbd_ext(union enetc_rx_bd *rxbd)

{

	return ++rxbd;

}



struct enetc_msg_swbd {

	void *vaddr;

drivers/net/ethernet/freescale/enetc/enetc_ethtool.c

+14 −8
Original line number Diff line number Diff line
@@ -195,15 +195,21 @@ static const char tx_ring_stats[][ETH_GSTRING_LEN] = { 
static int enetc_get_sset_count(struct net_device *ndev, int sset)

{

	struct enetc_ndev_priv *priv = netdev_priv(ndev);

	int len;



	if (sset != ETH_SS_STATS)

		return -EOPNOTSUPP;



	if (sset == ETH_SS_STATS)

		return ARRAY_SIZE(enetc_si_counters) +

	len = ARRAY_SIZE(enetc_si_counters) +

	      ARRAY_SIZE(tx_ring_stats) * priv->num_tx_rings +

			ARRAY_SIZE(rx_ring_stats) * priv->num_rx_rings +

			(enetc_si_is_pf(priv->si) ?

			ARRAY_SIZE(enetc_port_counters) : 0);

	      ARRAY_SIZE(rx_ring_stats) * priv->num_rx_rings;



	return -EOPNOTSUPP;

	if (!enetc_si_is_pf(priv->si))

		return len;



	len += ARRAY_SIZE(enetc_port_counters);



	return len;

}



static void enetc_get_strings(struct net_device *ndev, u32 stringset, u8 *data)
@@ -568,7 +574,7 @@ static int enetc_get_ts_info(struct net_device *ndev, 
		info->phc_index = -1;

	}



#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING

#ifdef CONFIG_FSL_ENETC_PTP_CLOCK

	info->so_timestamping = SOF_TIMESTAMPING_TX_HARDWARE |

				SOF_TIMESTAMPING_RX_HARDWARE |

				SOF_TIMESTAMPING_RAW_HARDWARE;

drivers/net/ethernet/freescale/enetc/enetc_hw.h

+3 −6
Original line number Diff line number Diff line
@@ -418,9 +418,6 @@ union enetc_rx_bd { 
	struct {

		__le64 addr;

		u8 reserved[8];

#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING

		u8 reserved1[16];

#endif

	} w;

	struct {

		__le16 inet_csum;
@@ -435,11 +432,11 @@ union enetc_rx_bd { 
			};

			__le32 lstatus;

		};

#ifdef CONFIG_FSL_ENETC_HW_TIMESTAMPING

	} r;

	struct {

		__le32 tstamp;

		u8 reserved[12];

#endif

	} r;

	} ext;

};



#define ENETC_RXBD_LSTATUS_R	BIT(30)

CRA Git | Maintained and supported by SUSTech CRA and CCSE