diff options
author | Steven Luong <sluong@cisco.com> | 2021-04-02 12:23:05 -0700 |
---|---|---|
committer | Damjan Marion <dmarion@me.com> | 2021-04-08 21:42:15 +0000 |
commit | 7fbc5f9b1d613ba9c2b3ba2211f5de2e7bf43db3 (patch) | |
tree | af8b6da970888c772ae4744aefa32c6da23f981a /src/plugins/memif | |
parent | 5d46e52fd28ccc52539031bbbe5c67d749116a09 (diff) |
memif: Use log_debug for clib_file_add and clib_file_del
It is more appropriate to use log_debug instead of log_warn when inoking
clib_file_add and clib_file_del.
Type: fix
Signed-off-by: Steven Luong <sluong@cisco.com>
Change-Id: I544d97037cfca3f22ce94242aef0c7b0fea2fa9b
Diffstat (limited to 'src/plugins/memif')
-rw-r--r-- | src/plugins/memif/private.h | 39 |
1 files changed, 24 insertions, 15 deletions
diff --git a/src/plugins/memif/private.h b/src/plugins/memif/private.h index 22b2ab8a6d2..c9d6a79da5c 100644 --- a/src/plugins/memif/private.h +++ b/src/plugins/memif/private.h @@ -63,21 +63,30 @@ ##__VA_ARGS__); \ } while (0) -#define memif_file_add(a, b) do { \ - *a = clib_file_add (&file_main, b); \ - memif_log_warn (0, "clib_file_add fd %d private_data %u idx %u", \ - (b)->file_descriptor, (b)->private_data, *a); \ -} while (0) - -#define memif_file_del(a) do { \ - memif_log_warn (0, "clib_file_del idx %u", a - file_main.file_pool); \ - clib_file_del (&file_main, a); \ -} while (0) - -#define memif_file_del_by_index(a) do { \ - memif_log_warn (0, "clib_file_del idx %u", a); \ - clib_file_del_by_index (&file_main, a); \ -} while (0) +#define memif_file_add(a, b) \ + do \ + { \ + *a = clib_file_add (&file_main, b); \ + memif_log_debug (0, "clib_file_add fd %d private_data %u idx %u", \ + (b)->file_descriptor, (b)->private_data, *a); \ + } \ + while (0) + +#define memif_file_del(a) \ + do \ + { \ + memif_log_debug (0, "clib_file_del idx %u", a - file_main.file_pool); \ + clib_file_del (&file_main, a); \ + } \ + while (0) + +#define memif_file_del_by_index(a) \ + do \ + { \ + memif_log_debug (0, "clib_file_del idx %u", a); \ + clib_file_del_by_index (&file_main, a); \ + } \ + while (0) typedef struct { |