nfp: add helper for mapping runtime symbols
authorJakub Kicinski <jakub.kicinski@netronome.com>
Tue, 27 Jun 2017 07:50:17 +0000 (00:50 -0700)
committerDavid S. Miller <davem@davemloft.net>
Tue, 27 Jun 2017 19:48:47 +0000 (15:48 -0400)
Move most of the helper for mapping RTsyms from nfp_net_main.c
to nfpcore.  Use the new helper directly for mapping MAC statistics,
since they don't need to include the PCIe interface ID in the symbol
name.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: Simon Horman <simon.horman@netronome.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/netronome/nfp/nfp_net_main.c
drivers/net/ethernet/netronome/nfp/nfpcore/nfp_nffw.h
drivers/net/ethernet/netronome/nfp/nfpcore/nfp_rtsym.c

index 16ee904db0cd6e16a9b23e16d37cd3c64052d37f..93d6ea1839566630aa043c2148b2b70b22695dd6 100644 (file)
@@ -174,31 +174,12 @@ static u8 __iomem *
 nfp_net_pf_map_rtsym(struct nfp_pf *pf, const char *name, const char *sym_fmt,
                     unsigned int min_size, struct nfp_cpp_area **area)
 {
-       const struct nfp_rtsym *sym;
        char pf_symbol[256];
-       u8 __iomem *mem;
 
        snprintf(pf_symbol, sizeof(pf_symbol), sym_fmt,
                 nfp_cppcore_pcie_unit(pf->cpp));
 
-       sym = nfp_rtsym_lookup(pf->rtbl, pf_symbol);
-       if (!sym)
-               return (u8 __iomem *)ERR_PTR(-ENOENT);
-
-       if (sym->size < min_size) {
-               nfp_err(pf->cpp, "PF symbol %s too small\n", pf_symbol);
-               return (u8 __iomem *)ERR_PTR(-EINVAL);
-       }
-
-       mem = nfp_cpp_map_area(pf->cpp, name, sym->domain, sym->target,
-                              sym->addr, sym->size, area);
-       if (IS_ERR(mem)) {
-               nfp_err(pf->cpp, "Failed to map PF symbol %s: %ld\n",
-                       pf_symbol, PTR_ERR(mem));
-               return mem;
-       }
-
-       return mem;
+       return nfp_rtsym_map(pf->rtbl, pf_symbol, name, min_size, area);
 }
 
 static void nfp_net_pf_free_vnic(struct nfp_pf *pf, struct nfp_net *nn)
@@ -528,23 +509,22 @@ static void nfp_net_pci_unmap_mem(struct nfp_pf *pf)
 
 static int nfp_net_pci_map_mem(struct nfp_pf *pf)
 {
-       u32 ctrl_bar_sz;
        u8 __iomem *mem;
+       u32 min_size;
        int err;
 
-       ctrl_bar_sz = pf->max_data_vnics * NFP_PF_CSR_SLICE_SIZE;
+       min_size = pf->max_data_vnics * NFP_PF_CSR_SLICE_SIZE;
        mem = nfp_net_pf_map_rtsym(pf, "net.ctrl", "_pf%d_net_bar0",
-                                  ctrl_bar_sz, &pf->data_vnic_bar);
+                                  min_size, &pf->data_vnic_bar);
        if (IS_ERR(mem)) {
                nfp_err(pf->cpp, "Failed to find data vNIC memory symbol\n");
                return PTR_ERR(mem);
        }
 
-       pf->mac_stats_mem = nfp_net_pf_map_rtsym(pf, "net.macstats",
-                                                "_mac_stats",
-                                                NFP_MAC_STATS_SIZE *
-                                                (pf->eth_tbl->max_index + 1),
-                                                &pf->mac_stats_bar);
+       min_size =  NFP_MAC_STATS_SIZE * (pf->eth_tbl->max_index + 1);
+       pf->mac_stats_mem = nfp_rtsym_map(pf->rtbl, "_mac_stats",
+                                         "net.macstats", min_size,
+                                         &pf->mac_stats_bar);
        if (IS_ERR(pf->mac_stats_mem)) {
                if (PTR_ERR(pf->mac_stats_mem) != -ENOENT) {
                        err = PTR_ERR(pf->mac_stats_mem);
index d27d29782a1262be31f7e067122ec2f2bcca2f18..c9724fb7ea4b23cf695dc28d6dc1e2922930262a 100644 (file)
@@ -97,7 +97,11 @@ int nfp_rtsym_count(struct nfp_rtsym_table *rtbl);
 const struct nfp_rtsym *nfp_rtsym_get(struct nfp_rtsym_table *rtbl, int idx);
 const struct nfp_rtsym *
 nfp_rtsym_lookup(struct nfp_rtsym_table *rtbl, const char *name);
+
 u64 nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl, const char *name,
                      int *error);
+u8 __iomem *
+nfp_rtsym_map(struct nfp_rtsym_table *rtbl, const char *name, const char *id,
+             unsigned int min_size, struct nfp_cpp_area **area);
 
 #endif /* NFP_NFFW_H */
index 203f9cbae0fbc79271f4e6c724a90acb89a20137..ecda474ac7c3e256730e1319766caf40f44d8053 100644 (file)
@@ -289,3 +289,30 @@ exit:
                return ~0ULL;
        return val;
 }
+
+u8 __iomem *
+nfp_rtsym_map(struct nfp_rtsym_table *rtbl, const char *name, const char *id,
+             unsigned int min_size, struct nfp_cpp_area **area)
+{
+       const struct nfp_rtsym *sym;
+       u8 __iomem *mem;
+
+       sym = nfp_rtsym_lookup(rtbl, name);
+       if (!sym)
+               return (u8 __iomem *)ERR_PTR(-ENOENT);
+
+       if (sym->size < min_size) {
+               nfp_err(rtbl->cpp, "Symbol %s too small\n", name);
+               return (u8 __iomem *)ERR_PTR(-EINVAL);
+       }
+
+       mem = nfp_cpp_map_area(rtbl->cpp, id, sym->domain, sym->target,
+                              sym->addr, sym->size, area);
+       if (IS_ERR(mem)) {
+               nfp_err(rtbl->cpp, "Failed to map symbol %s: %ld\n",
+                       name, PTR_ERR(mem));
+               return mem;
+       }
+
+       return mem;
+}