Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 97d731d8 authored by David S. Miller's avatar David S. Miller
Browse files

Merge branch 'nfp-port-enumeration-change-and-FW-ABI-adjustment' 



Jakub Kicinski says:

====================
nfp: port enumeration change and FW ABI adjustment

This set changes the way ports are numbered internally to avoid MAC address
changes and invalid link information when breakout is configured.  Second
patch gets rid of old way of looking up MAC addresses in device information
which caused all this confusion.

Patch 3 is a small adjustment to the new FW ABI version we introduced in
this release cycle.
====================

Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parents 42af627b 64a919a9

drivers/net/ethernet/netronome/nfp/flower/main.c

+1 −2
Original line number Diff line number Diff line
@@ -245,8 +245,7 @@ nfp_flower_spawn_phy_reprs(struct nfp_app *app, struct nfp_flower_priv *priv) 
		}



		SET_NETDEV_DEV(reprs->reprs[phys_port], &priv->nn->pdev->dev);

		nfp_net_get_mac_addr(app->pf, port,

				     eth_tbl->ports[i].eth_index);

		nfp_net_get_mac_addr(app->pf, port);



		cmsg_port_id = nfp_flower_cmsg_phys_port(phys_port);

		err = nfp_repr_init(app, reprs->reprs[phys_port],

drivers/net/ethernet/netronome/nfp/nfp_app_nic.c

+1 −1
Original line number Diff line number Diff line
@@ -69,7 +69,7 @@ int nfp_app_nic_vnic_init(struct nfp_app *app, struct nfp_net *nn, 
	if (err)

		return err < 0 ? err : 0;



	nfp_net_get_mac_addr(app->pf, nn->port, id);

	nfp_net_get_mac_addr(app->pf, nn->port);



	return 0;

}

drivers/net/ethernet/netronome/nfp/nfp_main.h

+1 −4
Original line number Diff line number Diff line
@@ -149,10 +149,7 @@ void nfp_net_pci_remove(struct nfp_pf *pf); 
int nfp_hwmon_register(struct nfp_pf *pf);

void nfp_hwmon_unregister(struct nfp_pf *pf);



struct nfp_eth_table_port *

nfp_net_find_port(struct nfp_eth_table *eth_tbl, unsigned int id);

void

nfp_net_get_mac_addr(struct nfp_pf *pf, struct nfp_port *port, unsigned int id);

void nfp_net_get_mac_addr(struct nfp_pf *pf, struct nfp_port *port);



bool nfp_ctrl_tx(struct nfp_net *nn, struct sk_buff *skb);

drivers/net/ethernet/netronome/nfp/nfp_net_common.c

+8 −1
Original line number Diff line number Diff line
@@ -3719,10 +3719,17 @@ int nfp_net_init(struct nfp_net *nn) 
	nn->cap = nn_readl(nn, NFP_NET_CFG_CAP);

	nn->max_mtu = nn_readl(nn, NFP_NET_CFG_MAX_MTU);



	/* Chained metadata is signalled by capabilities except in version 4 */

	/* ABI 4.x and ctrl vNIC always use chained metadata, in other cases

	 * we allow use of non-chained metadata if RSS(v1) is the only

	 * advertised capability requiring metadata.

	 */

	nn->dp.chained_metadata_format = nn->fw_ver.major == 4 ||

					 !nn->dp.netdev ||

					 !(nn->cap & NFP_NET_CFG_CTRL_RSS) ||

					 nn->cap & NFP_NET_CFG_CTRL_CHAIN_META;

	/* RSS(v1) uses non-chained metadata format, except in ABI 4.x where

	 * it has the same meaning as RSSv2.

	 */

	if (nn->dp.chained_metadata_format && nn->fw_ver.major != 4)

		nn->cap &= ~NFP_NET_CFG_CTRL_RSS;

drivers/net/ethernet/netronome/nfp/nfp_net_main.c

+10 −34
Original line number Diff line number Diff line
@@ -84,55 +84,31 @@ static int nfp_is_ready(struct nfp_pf *pf) 
 * nfp_net_get_mac_addr() - Get the MAC address.

 * @pf:       NFP PF handle

 * @port:     NFP port structure

 * @id:	      NFP port id

 *

 * First try to get the MAC address from NSP ETH table. If that

 * fails try HWInfo.  As a last resort generate a random address.

 * fails generate a random address.

 */

void

nfp_net_get_mac_addr(struct nfp_pf *pf, struct nfp_port *port, unsigned int id)

void nfp_net_get_mac_addr(struct nfp_pf *pf, struct nfp_port *port)

{

	struct nfp_eth_table_port *eth_port;

	u8 mac_addr[ETH_ALEN];

	const char *mac_str;

	char name[32];



	eth_port = __nfp_port_get_eth_port(port);

	if (eth_port) {

		ether_addr_copy(port->netdev->dev_addr, eth_port->mac_addr);

		ether_addr_copy(port->netdev->perm_addr, eth_port->mac_addr);

		return;

	}



	snprintf(name, sizeof(name), "eth%d.mac", id);



	mac_str = nfp_hwinfo_lookup(pf->hwinfo, name);

	if (!mac_str) {

		nfp_warn(pf->cpp, "Can't lookup MAC address. Generate\n");

		eth_hw_addr_random(port->netdev);

		return;

	}



	if (sscanf(mac_str, "%02hhx:%02hhx:%02hhx:%02hhx:%02hhx:%02hhx",

		   &mac_addr[0], &mac_addr[1], &mac_addr[2],

		   &mac_addr[3], &mac_addr[4], &mac_addr[5]) != 6) {

		nfp_warn(pf->cpp, "Can't parse MAC address (%s). Generate.\n",

			 mac_str);

	if (!eth_port) {

		eth_hw_addr_random(port->netdev);

		return;

	}



	ether_addr_copy(port->netdev->dev_addr, mac_addr);

	ether_addr_copy(port->netdev->perm_addr, mac_addr);

	ether_addr_copy(port->netdev->dev_addr, eth_port->mac_addr);

	ether_addr_copy(port->netdev->perm_addr, eth_port->mac_addr);

}



struct nfp_eth_table_port *

nfp_net_find_port(struct nfp_eth_table *eth_tbl, unsigned int id)

static struct nfp_eth_table_port *

nfp_net_find_port(struct nfp_eth_table *eth_tbl, unsigned int index)

{

	int i;



	for (i = 0; eth_tbl && i < eth_tbl->count; i++)

		if (eth_tbl->ports[i].eth_index == id)

		if (eth_tbl->ports[i].index == index)

			return &eth_tbl->ports[i];



	return NULL;
@@ -202,7 +178,7 @@ static void nfp_net_pf_free_vnics(struct nfp_pf *pf) 
static struct nfp_net *

nfp_net_pf_alloc_vnic(struct nfp_pf *pf, bool needs_netdev,

		      void __iomem *ctrl_bar, void __iomem *qc_bar,

		      int stride, unsigned int eth_id)

		      int stride, unsigned int id)

{

	u32 tx_base, rx_base, n_tx_rings, n_rx_rings;

	struct nfp_net *nn;
@@ -228,7 +204,7 @@ nfp_net_pf_alloc_vnic(struct nfp_pf *pf, bool needs_netdev, 
	nn->stride_tx = stride;



	if (needs_netdev) {

		err = nfp_app_vnic_init(pf->app, nn, eth_id);

		err = nfp_app_vnic_init(pf->app, nn, id);

		if (err) {

			nfp_net_free(nn);

			return ERR_PTR(err);