Commit 31e380f3 authored by Jakub Kicinski's avatar Jakub Kicinski Committed by David S. Miller

nfp: prefix rtsym error messages with symbol name

For ease of debug preface all error messages with the name
of the symbol which caused them.  Use the same message format
for existing messages while at it.
Signed-off-by: default avatarJakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: default avatarFrancois H. Theron <francois.theron@netronome.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3c576de3
...@@ -237,10 +237,10 @@ u64 nfp_rtsym_size(const struct nfp_rtsym *sym) ...@@ -237,10 +237,10 @@ u64 nfp_rtsym_size(const struct nfp_rtsym *sym)
{ {
switch (sym->type) { switch (sym->type) {
case NFP_RTSYM_TYPE_NONE: case NFP_RTSYM_TYPE_NONE:
pr_err("rtsym type NONE\n"); pr_err("rtsym '%s': type NONE\n", sym->name);
return 0; return 0;
default: default:
pr_warn("Unknown rtsym type: %d\n", sym->type); pr_warn("rtsym '%s': unknown type: %d\n", sym->name, sym->type);
/* fall through */ /* fall through */
case NFP_RTSYM_TYPE_OBJECT: case NFP_RTSYM_TYPE_OBJECT:
case NFP_RTSYM_TYPE_FUNCTION: case NFP_RTSYM_TYPE_FUNCTION:
...@@ -255,7 +255,8 @@ nfp_rtsym_to_dest(struct nfp_cpp *cpp, const struct nfp_rtsym *sym, ...@@ -255,7 +255,8 @@ nfp_rtsym_to_dest(struct nfp_cpp *cpp, const struct nfp_rtsym *sym,
u8 action, u8 token, u64 off, u32 *cpp_id, u64 *addr) u8 action, u8 token, u64 off, u32 *cpp_id, u64 *addr)
{ {
if (sym->type != NFP_RTSYM_TYPE_OBJECT) { if (sym->type != NFP_RTSYM_TYPE_OBJECT) {
nfp_err(cpp, "Direct access attempt to non-object rtsym\n"); nfp_err(cpp, "rtsym '%s': direct access to non-object rtsym\n",
sym->name);
return -EINVAL; return -EINVAL;
} }
...@@ -270,8 +271,8 @@ nfp_rtsym_to_dest(struct nfp_cpp *cpp, const struct nfp_rtsym *sym, ...@@ -270,8 +271,8 @@ nfp_rtsym_to_dest(struct nfp_cpp *cpp, const struct nfp_rtsym *sym,
*cpp_id = NFP_CPP_ISLAND_ID(NFP_CPP_TARGET_MU, action, token, *cpp_id = NFP_CPP_ISLAND_ID(NFP_CPP_TARGET_MU, action, token,
sym->domain); sym->domain);
} else if (sym->target < 0) { } else if (sym->target < 0) {
nfp_err(cpp, "Unhandled RTsym target encoding: %d\n", nfp_err(cpp, "rtsym '%s': unhandled target encoding: %d\n",
sym->target); sym->name, sym->target);
return -EINVAL; return -EINVAL;
} else { } else {
*cpp_id = NFP_CPP_ISLAND_ID(sym->target, action, token, *cpp_id = NFP_CPP_ISLAND_ID(sym->target, action, token,
...@@ -451,7 +452,7 @@ u64 nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl, const char *name, ...@@ -451,7 +452,7 @@ u64 nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl, const char *name,
break; break;
default: default:
nfp_err(rtbl->cpp, nfp_err(rtbl->cpp,
"rtsym '%s' unsupported or non-scalar size: %lld\n", "rtsym '%s': unsupported or non-scalar size: %lld\n",
name, nfp_rtsym_size(sym)); name, nfp_rtsym_size(sym));
err = -EINVAL; err = -EINVAL;
break; break;
...@@ -497,7 +498,7 @@ int nfp_rtsym_write_le(struct nfp_rtsym_table *rtbl, const char *name, ...@@ -497,7 +498,7 @@ int nfp_rtsym_write_le(struct nfp_rtsym_table *rtbl, const char *name,
break; break;
default: default:
nfp_err(rtbl->cpp, nfp_err(rtbl->cpp,
"rtsym '%s' unsupported or non-scalar size: %lld\n", "rtsym '%s': unsupported or non-scalar size: %lld\n",
name, nfp_rtsym_size(sym)); name, nfp_rtsym_size(sym));
err = -EINVAL; err = -EINVAL;
break; break;
...@@ -523,18 +524,18 @@ nfp_rtsym_map(struct nfp_rtsym_table *rtbl, const char *name, const char *id, ...@@ -523,18 +524,18 @@ nfp_rtsym_map(struct nfp_rtsym_table *rtbl, const char *name, const char *id,
err = nfp_rtsym_to_dest(rtbl->cpp, sym, NFP_CPP_ACTION_RW, 0, 0, err = nfp_rtsym_to_dest(rtbl->cpp, sym, NFP_CPP_ACTION_RW, 0, 0,
&cpp_id, &addr); &cpp_id, &addr);
if (err) { if (err) {
nfp_err(rtbl->cpp, "Symbol %s mapping failed\n", name); nfp_err(rtbl->cpp, "rtsym '%s': mapping failed\n", name);
return (u8 __iomem *)ERR_PTR(err); return (u8 __iomem *)ERR_PTR(err);
} }
if (sym->size < min_size) { if (sym->size < min_size) {
nfp_err(rtbl->cpp, "Symbol %s too small\n", name); nfp_err(rtbl->cpp, "rtsym '%s': too small\n", name);
return (u8 __iomem *)ERR_PTR(-EINVAL); return (u8 __iomem *)ERR_PTR(-EINVAL);
} }
mem = nfp_cpp_map_area(rtbl->cpp, id, cpp_id, addr, sym->size, area); mem = nfp_cpp_map_area(rtbl->cpp, id, cpp_id, addr, sym->size, area);
if (IS_ERR(mem)) { if (IS_ERR(mem)) {
nfp_err(rtbl->cpp, "Failed to map symbol %s: %ld\n", nfp_err(rtbl->cpp, "rtysm '%s': failed to map: %ld\n",
name, PTR_ERR(mem)); name, PTR_ERR(mem));
return mem; return mem;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment