nfp: bpf: improve map offload info messages

FW can put constraints on map element size to maximize resource
use and efficiency.  When user attempts offload of a map which
does not fit into those constraints an informational message is
printed to kernel logs to inform user about the reason offload
failed.  Map offload does not have access to any advanced error
reporting like verifier log or extack.  There is also currently
no way for us to nicely expose the FW capabilities to user
space.  Given all those constraints we should make sure log
messages are as informative as possible.  Improve them.

Signed-off-by: Jakub Kicinski <jakub.kicinski@netronome.com>
Reviewed-by: Dirk van der Merwe <dirk.vandermerwe@netronome.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>

diff --git a/drivers/net/ethernet/netronome/nfp/bpf/offload.c b/drivers/net/ethernet/netronome/nfp/bpf/offload.c
index b1fbb3b..1ccd637 100644 (file)
--- a/drivers/net/ethernet/netronome/nfp/bpf/offload.c
+++ b/drivers/net/ethernet/netronome/nfp/bpf/offload.c
@@ -380,11 +380,23 @@ nfp_bpf_map_alloc(struct nfp_app_bpf *bpf, struct bpf_offloaded_map *offmap)
                        bpf->maps.max_elems - bpf->map_elems_in_use);
                return -ENOMEM;
        }
-       if (offmap->map.key_size > bpf->maps.max_key_sz ||
-           offmap->map.value_size > bpf->maps.max_val_sz ||
-           round_up(offmap->map.key_size, 8) +
+
+       if (round_up(offmap->map.key_size, 8) +
            round_up(offmap->map.value_size, 8) > bpf->maps.max_elem_sz) {
-               pr_info("elements don't fit in device constraints\n");
+               pr_info("map elements too large: %u, FW max element size (key+value): %u\n",
+                       round_up(offmap->map.key_size, 8) +
+                       round_up(offmap->map.value_size, 8),
+                       bpf->maps.max_elem_sz);
+               return -ENOMEM;
+       }
+       if (offmap->map.key_size > bpf->maps.max_key_sz) {
+               pr_info("map key size %u, FW max is %u\n",
+                       offmap->map.key_size, bpf->maps.max_key_sz);
+               return -ENOMEM;
+       }
+       if (offmap->map.value_size > bpf->maps.max_val_sz) {
+               pr_info("map value size %u, FW max is %u\n",
+                       offmap->map.value_size, bpf->maps.max_val_sz);
                return -ENOMEM;
        }