From 5c0f59fd3c3d07a4ec6ca7fc251d90160bbc3b49 Mon Sep 17 00:00:00 2001 From: Shiva Krishna Pittala Date: Thu, 7 Nov 2019 13:30:52 +0530 Subject: [PATCH] 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 --- qdf/inc/qdf_types.h | 1 - qdf/linux/src/i_qdf_trace.h | 30 +++++++++++++++++++++++++++++- qdf/linux/src/i_qdf_types.h | 1 - 3 files changed, 29 insertions(+), 3 deletions(-) diff --git a/qdf/inc/qdf_types.h b/qdf/inc/qdf_types.h index 80c586c0e278..1c5536ccd6cb 100644 --- a/qdf/inc/qdf_types.h +++ b/qdf/inc/qdf_types.h @@ -673,7 +673,6 @@ enum QDF_GLOBAL_MODE { #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 de7387949050..2da0f2464a69 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 @@ static inline void __qdf_trace_noop(QDF_MODULE_ID module, char *format, ...) { } __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 @@ static inline void __qdf_trace_noop(QDF_MODULE_ID module, char *format, ...) { } __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 @@ static inline void __qdf_trace_noop(QDF_MODULE_ID module, char *format, ...) { } __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 @@ static inline void __qdf_trace_noop(QDF_MODULE_ID module, char *format, ...) { } __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 @@ static inline void __qdf_trace_noop(QDF_MODULE_ID module, char *format, ...) { } __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 @@ static inline void qdf_trace_msg(QDF_MODULE_ID module, QDF_TRACE_LEVEL level, #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 df9d51bdf83d..ac519dda5d04 100644 --- a/qdf/linux/src/i_qdf_types.h +++ b/qdf/linux/src/i_qdf_types.h @@ -329,7 +329,6 @@ enum __qdf_net_wireless_evcode { __QDF_CUSTOM_PUSH_BUTTON = IWEVCUSTOM, }; -#define __qdf_vprint vprintk #define __qdf_snprint snprintf #define __qdf_vsnprint vsnprintf #define __qdf_toupper toupper