Adds ntnic specific implementation for logging.
NT NIC uses this logging abstraction layer
to ensure that FPGA module implementations function both
within and outside in DPDK environment

Signed-off-by: Serhii Iliushyk <sil-...@napatech.com>
---
 drivers/net/ntnic/meson.build    |  2 ++
 drivers/net/ntnic/ntlog/ntlog.c  | 53 ++++++++++++++++++++++++++++++++
 drivers/net/ntnic/ntlog/ntlog.h  | 49 +++++++++++++++++++++++++++++
 drivers/net/ntnic/ntnic_ethdev.c |  2 ++
 4 files changed, 106 insertions(+)
 create mode 100644 drivers/net/ntnic/ntlog/ntlog.c
 create mode 100644 drivers/net/ntnic/ntlog/ntlog.h

diff --git a/drivers/net/ntnic/meson.build b/drivers/net/ntnic/meson.build
index 194353230b..80f0f3eecf 100644
--- a/drivers/net/ntnic/meson.build
+++ b/drivers/net/ntnic/meson.build
@@ -10,9 +10,11 @@ endif
 # includes
 includes = [
     include_directories('.'),
+    include_directories('ntlog'),
 ]
 
 # all sources
 sources = files(
+    'ntlog/ntlog.c',
     'ntnic_ethdev.c',
 )
diff --git a/drivers/net/ntnic/ntlog/ntlog.c b/drivers/net/ntnic/ntlog/ntlog.c
new file mode 100644
index 0000000000..2e4fba799d
--- /dev/null
+++ b/drivers/net/ntnic/ntlog/ntlog.c
@@ -0,0 +1,53 @@
+/*
+ * SPDX-License-Identifier: BSD-3-Clause
+ * Copyright(c) 2023 Napatech A/S
+ */
+
+#include "ntlog.h"
+
+#include <stdarg.h>
+#include <stddef.h>
+#include <string.h>
+#include <stdlib.h>
+#include <stdbool.h>
+
+#include <rte_log.h>
+#include <rte_string_fns.h>
+
+#define NTLOG_HELPER_STR_SIZE_MAX (1024)
+
+RTE_LOG_REGISTER_DEFAULT(nt_logtype, NOTICE)
+
+char *ntlog_helper_str_alloc(const char *sinit)
+{
+       char *s = malloc(NTLOG_HELPER_STR_SIZE_MAX);
+
+       if (!s)
+               return NULL;
+
+       if (sinit)
+               snprintf(s, NTLOG_HELPER_STR_SIZE_MAX, "%s", sinit);
+
+       else
+               s[0] = '\0';
+
+       return s;
+}
+
+__rte_format_printf(2, 0)
+void ntlog_helper_str_add(char *s, const char *format, ...)
+{
+       if (!s)
+               return;
+
+       va_list args;
+       va_start(args, format);
+       int len = strlen(s);
+       vsnprintf(&s[len], (NTLOG_HELPER_STR_SIZE_MAX - 1 - len), format, args);
+       va_end(args);
+}
+
+void ntlog_helper_str_free(char *s)
+{
+       free(s);
+}
diff --git a/drivers/net/ntnic/ntlog/ntlog.h b/drivers/net/ntnic/ntlog/ntlog.h
new file mode 100644
index 0000000000..58dcce0580
--- /dev/null
+++ b/drivers/net/ntnic/ntlog/ntlog.h
@@ -0,0 +1,49 @@
+/*
+ * SPDX-License-Identifier: BSD-3-Clause
+ * Copyright(c) 2023 Napatech A/S
+ */
+
+#ifndef NTOSS_SYSTEM_NTLOG_H
+#define NTOSS_SYSTEM_NTLOG_H
+
+#include <stdarg.h>
+#include <stdint.h>
+#include <rte_log.h>
+
+extern int nt_logtype;
+
+#define NT_DRIVER_NAME "ntnic"
+
+#define NT_PMD_DRV_LOG(level, ...) \
+       rte_log(RTE_LOG_ ## level, nt_logtype, \
+               RTE_FMT(NT_DRIVER_NAME ": " \
+                       RTE_FMT_HEAD(__VA_ARGS__, ""), \
+               RTE_FMT_TAIL(__VA_ARGS__, "")))
+
+
+#define NT_LOG_ERR(...) NT_PMD_DRV_LOG(ERR, __VA_ARGS__)
+#define NT_LOG_WRN(...) NT_PMD_DRV_LOG(WARNING, __VA_ARGS__)
+#define NT_LOG_INF(...) NT_PMD_DRV_LOG(INFO, __VA_ARGS__)
+#define NT_LOG_DBG(...) NT_PMD_DRV_LOG(DEBUG, __VA_ARGS__)
+
+#define NT_LOG(level, module, ...) \
+       NT_LOG_##level(#module ": " #level ":" __VA_ARGS__)
+
+#define NT_LOG_DBGX(level, module, ...) \
+               rte_log(RTE_LOG_ ##level, nt_logtype, \
+                               RTE_FMT(NT_DRIVER_NAME #module ": [%s:%u]" \
+                                       RTE_FMT_HEAD(__VA_ARGS__, ""), 
__func__, __LINE__, \
+                               RTE_FMT_TAIL(__VA_ARGS__, "")))
+/*
+ * nt log helper functions
+ * to create a string for NT_LOG usage to output a one-liner log
+ * to use when one single function call to NT_LOG is not optimal - that is
+ * you do not know the number of parameters at programming time or it is 
variable
+ */
+char *ntlog_helper_str_alloc(const char *sinit);
+
+void ntlog_helper_str_add(char *s, const char *format, ...);
+
+void ntlog_helper_str_free(char *s);
+
+#endif /* NTOSS_SYSTEM_NTLOG_H */
diff --git a/drivers/net/ntnic/ntnic_ethdev.c b/drivers/net/ntnic/ntnic_ethdev.c
index 7e5231d2c1..68df9be2ff 100644
--- a/drivers/net/ntnic/ntnic_ethdev.c
+++ b/drivers/net/ntnic/ntnic_ethdev.c
@@ -7,6 +7,8 @@
 #include <rte_bus_pci.h>
 #include <ethdev_pci.h>
 
+#include "ntlog.h"
+
 static const struct rte_pci_id nthw_pci_id_map[] = {
        {
                .vendor_id = 0,
-- 
2.45.0

Reply via email to