qcacmn: Route variable argument list logs through QDF
Currently, the variable argument list messages are going through vprintk.
As a result, these messages are appearing only on dmesg, but not on any
external logger. Use QDF logging functions instead to fix this.
CRs-Fixed: 2561051
Change-Id: I9cfc6aaa7b81de4945f7cfd5f7cd15540c3fc1f2
diff --git a/qdf/inc/qdf_types.h b/qdf/inc/qdf_types.h
index 80c586c..1c5536c 100644
--- a/qdf/inc/qdf_types.h
+++ b/qdf/inc/qdf_types.h
@@ -673,7 +673,6 @@
#define qdf_rl_nofl_debug(params...) \
QDF_TRACE_DEBUG_RL_NO_FL(QDF_MODULE_ID_QDF, ## params)
-#define qdf_vprint __qdf_vprint
#define qdf_snprint __qdf_snprint
#define qdf_kstrtoint __qdf_kstrtoint
diff --git a/qdf/linux/src/i_qdf_trace.h b/qdf/linux/src/i_qdf_trace.h
index de73879..2da0f24 100644
--- a/qdf/linux/src/i_qdf_trace.h
+++ b/qdf/linux/src/i_qdf_trace.h
@@ -106,7 +106,8 @@
#define __QDF_TRACE_RL_NO_FL(log_level, module_id, format, args...) \
__QDF_TRACE_RATE_LIMITED(module_id, log_level, format, ## args)
-static inline void __qdf_trace_noop(QDF_MODULE_ID module, char *format, ...) { }
+static inline void __qdf_trace_noop(QDF_MODULE_ID module,
+ const char *format, ...) { }
#ifdef WLAN_LOG_FATAL
#define QDF_TRACE_FATAL(params...) \
@@ -117,11 +118,14 @@
__QDF_TRACE_RL(QDF_TRACE_LEVEL_FATAL, ## params)
#define QDF_TRACE_FATAL_RL_NO_FL(params...) \
__QDF_TRACE_RL_NO_FL(QDF_TRACE_LEVEL_FATAL, ## params)
+#define QDF_VTRACE_FATAL(module_id, fmt, args) \
+ QDF_VTRACE(module_id, QDF_TRACE_LEVEL_FATAL, fmt, args)
#else
#define QDF_TRACE_FATAL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_FATAL_NO_FL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_FATAL_RL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_FATAL_RL_NO_FL(params...) __qdf_trace_noop(params)
+#define QDF_VTRACE_FATAL(params...) __qdf_trace_noop(params)
#endif
#ifdef WLAN_LOG_ERROR
@@ -133,11 +137,14 @@
__QDF_TRACE_RL(QDF_TRACE_LEVEL_ERROR, ## params)
#define QDF_TRACE_ERROR_RL_NO_FL(params...) \
__QDF_TRACE_RL_NO_FL(QDF_TRACE_LEVEL_ERROR, ## params)
+#define QDF_VTRACE_ERROR(module_id, fmt, args) \
+ QDF_VTRACE(module_id, QDF_TRACE_LEVEL_ERROR, fmt, args)
#else
#define QDF_TRACE_ERROR(params...) __qdf_trace_noop(params)
#define QDF_TRACE_ERROR_NO_FL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_ERROR_RL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_ERROR_RL_NO_FL(params...) __qdf_trace_noop(params)
+#define QDF_VTRACE_ERROR(params...) __qdf_trace_noop(params)
#endif
#ifdef WLAN_LOG_WARN
@@ -149,11 +156,14 @@
__QDF_TRACE_RL(QDF_TRACE_LEVEL_WARN, ## params)
#define QDF_TRACE_WARN_RL_NO_FL(params...) \
__QDF_TRACE_RL_NO_FL(QDF_TRACE_LEVEL_WARN, ## params)
+#define QDF_VTRACE_WARN(module_id, fmt, args) \
+ QDF_VTRACE(module_id, QDF_TRACE_LEVEL_WARN, fmt, args)
#else
#define QDF_TRACE_WARN(params...) __qdf_trace_noop(params)
#define QDF_TRACE_WARN_NO_FL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_WARN_RL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_WARN_RL_NO_FL(params...) __qdf_trace_noop(params)
+#define QDF_VTRACE_WARN(params...) __qdf_trace_noop(params)
#endif
#ifdef WLAN_LOG_INFO
@@ -165,11 +175,14 @@
__QDF_TRACE_RL(QDF_TRACE_LEVEL_INFO, ## params)
#define QDF_TRACE_INFO_RL_NO_FL(params...) \
__QDF_TRACE_RL_NO_FL(QDF_TRACE_LEVEL_INFO, ## params)
+#define QDF_VTRACE_INFO(module_id, fmt, args) \
+ QDF_VTRACE(module_id, QDF_TRACE_LEVEL_INFO, fmt, args)
#else
#define QDF_TRACE_INFO(params...) __qdf_trace_noop(params)
#define QDF_TRACE_INFO_NO_FL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_INFO_RL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_INFO_RL_NO_FL(params...) __qdf_trace_noop(params)
+#define QDF_VTRACE_INFO(params...) __qdf_trace_noop(params)
#endif
#ifdef WLAN_LOG_DEBUG
@@ -181,11 +194,14 @@
__QDF_TRACE_RL(QDF_TRACE_LEVEL_DEBUG, ## params)
#define QDF_TRACE_DEBUG_RL_NO_FL(params...) \
__QDF_TRACE_RL_NO_FL(QDF_TRACE_LEVEL_DEBUG, ## params)
+#define QDF_VTRACE_DEBUG(module_id, fmt, args) \
+ QDF_VTRACE(module_id, QDF_TRACE_LEVEL_DEBUG, fmt, args)
#else
#define QDF_TRACE_DEBUG(params...) __qdf_trace_noop(params)
#define QDF_TRACE_DEBUG_NO_FL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_DEBUG_RL(params...) __qdf_trace_noop(params)
#define QDF_TRACE_DEBUG_RL_NO_FL(params...) __qdf_trace_noop(params)
+#define QDF_VTRACE_DEBUG(params...) __qdf_trace_noop(params)
#endif
#ifdef WLAN_LOG_ENTER
@@ -271,6 +287,18 @@
#endif
+#ifdef QDF_TRACE_PRINT_ENABLE
+static inline void qdf_vprint(const char *fmt, va_list args)
+{
+ QDF_VTRACE_INFO(QDF_MODULE_ID_ANY, fmt, args);
+}
+#else /* QDF_TRACE_PRINT_ENABLE */
+static inline void qdf_vprint(const char *fmt, va_list args)
+{
+ QDF_VTRACE_ERROR(QDF_MODULE_ID_QDF, fmt, args);
+}
+#endif
+
#ifdef PANIC_ON_BUG
#ifdef CONFIG_SLUB_DEBUG
/**
diff --git a/qdf/linux/src/i_qdf_types.h b/qdf/linux/src/i_qdf_types.h
index df9d51b..ac519dd 100644
--- a/qdf/linux/src/i_qdf_types.h
+++ b/qdf/linux/src/i_qdf_types.h
@@ -329,7 +329,6 @@
__QDF_CUSTOM_PUSH_BUTTON = IWEVCUSTOM,
};
-#define __qdf_vprint vprintk
#define __qdf_snprint snprintf
#define __qdf_vsnprint vsnprintf
#define __qdf_toupper toupper