nfp: register ports as devlink ports
authorJakub Kicinski <jakub.kicinski@netronome.com>
Fri, 26 May 2017 08:03:34 +0000 (01:03 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 26 May 2017 15:01:49 +0000 (11:01 -0400)
Extend nfp_port to contain devlink_port structures.  Register the
ports to allow users inspecting device ports.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/netronome/nfp/nfp_devlink.c
drivers/net/ethernet/netronome/nfp/nfp_net_main.c
drivers/net/ethernet/netronome/nfp/nfp_port.h

index a8a52b3ff42b146ab854fde58cde896f36308cae..2cfcb66b04bbbc525c614672c9ceb0eaee1c4e68 100644 (file)
  * SOFTWARE.
  */
 
+#include <linux/rtnetlink.h>
 #include <net/devlink.h>
 
+#include "nfpcore/nfp_nsp.h"
+#include "nfp_app.h"
 #include "nfp_main.h"
+#include "nfp_port.h"
+
+static int
+nfp_devlink_fill_eth_port(struct nfp_port *port,
+                         struct nfp_eth_table_port *copy)
+{
+       struct nfp_eth_table_port *eth_port;
+
+       eth_port = __nfp_port_get_eth_port(port);
+       if (!eth_port)
+               return -EINVAL;
+
+       memcpy(copy, eth_port, sizeof(*eth_port));
+
+       return 0;
+}
 
 const struct devlink_ops nfp_devlink_ops = {
 };
+
+int nfp_devlink_port_register(struct nfp_app *app, struct nfp_port *port)
+{
+       struct nfp_eth_table_port eth_port;
+       struct devlink *devlink;
+       int ret;
+
+       rtnl_lock();
+       ret = nfp_devlink_fill_eth_port(port, &eth_port);
+       rtnl_unlock();
+       if (ret)
+               return ret;
+
+       devlink_port_type_eth_set(&port->dl_port, port->netdev);
+       if (eth_port.is_split)
+               devlink_port_split_set(&port->dl_port, eth_port.label_port);
+
+       devlink = priv_to_devlink(app->pf);
+
+       return devlink_port_register(devlink, &port->dl_port, port->eth_id);
+}
+
+void nfp_devlink_port_unregister(struct nfp_port *port)
+{
+       devlink_port_unregister(&port->dl_port);
+}
index b0a6ec4fe0979fa2a587f4bef88c72de86771b54..b733c97677fb6b8f11b0aa0d140938c069fef67d 100644 (file)
@@ -354,9 +354,20 @@ nfp_net_pf_init_vnic(struct nfp_pf *pf, struct nfp_net *nn, unsigned int id)
 
        nfp_net_debugfs_vnic_add(nn, pf->ddir, id);
 
+       if (nn->port) {
+               err = nfp_devlink_port_register(pf->app, nn->port);
+               if (err)
+                       goto err_dfs_clean;
+       }
+
        nfp_net_info(nn);
 
        return 0;
+
+err_dfs_clean:
+       nfp_net_debugfs_dir_clean(&nn->debugfs_dir);
+       nfp_net_clean(nn);
+       return err;
 }
 
 static int
@@ -420,6 +431,8 @@ err_free_prev:
 
 static void nfp_net_pf_clean_vnic(struct nfp_pf *pf, struct nfp_net *nn)
 {
+       if (nn->port)
+               nfp_devlink_port_unregister(nn->port);
        nfp_net_debugfs_dir_clean(&nn->debugfs_dir);
        nfp_net_clean(nn);
 }
index 641617de41cce4c291bb26b264d86807a33c146e..36a540b514be334a5bb3b3e983f7ecef2b24b6a9 100644 (file)
@@ -34,6 +34,8 @@
 #ifndef _NFP_PORT_H_
 #define _NFP_PORT_H_
 
+#include <net/devlink.h>
+
 struct net_device;
 struct nfp_app;
 struct nfp_port;
@@ -66,6 +68,7 @@ enum nfp_port_flags {
  * @type:      what port type does the entity represent
  * @flags:     port flags
  * @app:       backpointer to the app structure
+ * @dl_port:   devlink port structure
  * @eth_id:    for %NFP_PORT_PHYS_PORT port ID in NFP enumeration scheme
  * @eth_port:  for %NFP_PORT_PHYS_PORT translated ETH Table port entry
  * @port_list: entry on pf's list of ports
@@ -78,6 +81,8 @@ struct nfp_port {
 
        struct nfp_app *app;
 
+       struct devlink_port dl_port;
+
        unsigned int eth_id;
        struct nfp_eth_table_port *eth_port;
 
@@ -99,4 +104,7 @@ void nfp_port_free(struct nfp_port *port);
 int nfp_net_refresh_eth_port(struct nfp_port *port);
 void nfp_net_refresh_port_table(struct nfp_port *port);
 
+int nfp_devlink_port_register(struct nfp_app *app, struct nfp_port *port);
+void nfp_devlink_port_unregister(struct nfp_port *port);
+
 #endif