Date: Sun, 27 Feb 2022 23:43:37 GMT From: "Bjoern A. Zeeb" <bz@FreeBSD.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org Subject: git: 94ee6a9e0725 - stable/13 - iwlwifi: enhance debug information Message-ID: <202202272343.21RNhbXK002999@gitrepo.freebsd.org>
next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by bz: URL: https://cgit.FreeBSD.org/src/commit/?id=94ee6a9e07258e3609401776137c846ebe42d2ad commit 94ee6a9e07258e3609401776137c846ebe42d2ad Author: Bjoern A. Zeeb <bz@FreeBSD.org> AuthorDate: 2022-02-24 21:38:27 +0000 Commit: Bjoern A. Zeeb <bz@FreeBSD.org> CommitDate: 2022-02-27 23:41:55 +0000 iwlwifi: enhance debug information Add a string of the debug type to the output of the debug message so it is easier to search for specific events in a trace with lots of debugging on. While here remove superflous (). Sponsored by: The FreeBSD Foundation (cherry picked from commit 51152e5e6150746ad5514f169c8fd1d04b010b35) --- sys/contrib/dev/iwlwifi/iwl-debug.h | 81 ++++++++++++++++++++----------------- 1 file changed, 43 insertions(+), 38 deletions(-) diff --git a/sys/contrib/dev/iwlwifi/iwl-debug.h b/sys/contrib/dev/iwlwifi/iwl-debug.h index e1316925442d..dac2c6b31562 100644 --- a/sys/contrib/dev/iwlwifi/iwl-debug.h +++ b/sys/contrib/dev/iwlwifi/iwl-debug.h @@ -89,11 +89,11 @@ void __iwl_warn(struct device *, const char *, ...); void __iwl_err(struct device *, enum iwl_err_mode, const char *, ...); #define IWL_CRIT(_subsys, _fmt, ...) \ - __iwl_crit((_subsys)->dev, (_fmt), ##__VA_ARGS__) + __iwl_crit((_subsys)->dev, _fmt, ##__VA_ARGS__) #define IWL_INFO(_subsys, _fmt, ...) \ - __iwl_info((_subsys)->dev, (_fmt), ##__VA_ARGS__) + __iwl_info((_subsys)->dev, _fmt, ##__VA_ARGS__) #define IWL_WARN(_subsys, _fmt, ...) \ - __iwl_warn((_subsys)->dev, (_fmt), ##__VA_ARGS__) + __iwl_warn((_subsys)->dev, _fmt, ##__VA_ARGS__) /* XXX Not sure what the two bools are good for if never passed. */ #define __IWL_ERR_DEV(_dev, _mode, _fmt, ...) \ __iwl_err((_dev), IWL_ERR_MODE_REGULAR, _fmt, ##__VA_ARGS__) @@ -112,82 +112,87 @@ bool iwl_have_debug_level(enum iwl_dl); void iwl_print_hex_dump(void *, enum iwl_dl, const char *, uint8_t *, size_t); void __iwl_dbg(struct device *, u32, bool, const char *, const char *fmt, ...); +#define IWL_DPRINTF_DEV_PREFIX(_dev, _e, _prefix, _fmt, ...) \ + __iwl_dbg(_dev, _e, false, __func__, #_prefix " " _fmt, ##__VA_ARGS__) #define IWL_DPRINTF_DEV(_dev, _e, _fmt, ...) \ - __iwl_dbg((_dev), (_e), false, __func__, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF_DEV_PREFIX(_dev, _e, _e, _fmt, ##__VA_ARGS__) #define IWL_DPRINTF(_subsys, _e, _fmt, ...) \ - IWL_DPRINTF_DEV((_subsys)->dev, (_e), (_fmt), ##__VA_ARGS__) + IWL_DPRINTF_DEV((_subsys)->dev, _e, _fmt, ##__VA_ARGS__) +#define IWL_DPRINTF_PREFIX(_subsys, _e, _prefix, _fmt, ...) \ + IWL_DPRINTF_DEV_PREFIX((_subsys)->dev, _e, _prefix, _fmt, ##__VA_ARGS__) #else /* !CONFIG_IWLWIFI_DEBUG */ #define IWL_DPRINTF_DEV(_dev, _e, _fmt, ...) #define IWL_DPRINTF(_subsys, _e, _fmt, ...) +#define IWL_DPRINTF_PREFIX(_subsys, _e, _prefix, _fmt, ...) #endif /* CONFIG_IWLWIFI_DEBUG */ #define IWL_DEBUG_ASSOC(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_ASSOC, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_ASSOC, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_CALIB(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_CALIB, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_CALIB, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_COEX(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_COEX, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_COEX, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_DEV(_dev, _level, _fmt, ...) \ - IWL_DPRINTF_DEV((_dev), (_level), (_fmt), ##__VA_ARGS__) + IWL_DPRINTF_DEV((_dev), (_level), _fmt, ##__VA_ARGS__) #define IWL_DEBUG_DROP(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_DROP, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_DROP, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_EEPROM(_dev, _fmt, ...) \ - IWL_DPRINTF_DEV((_dev), IWL_DL_EEPROM, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF_DEV((_dev), IWL_DL_EEPROM, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_FW(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_FW, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_FW, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_FW_INFO(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_FW | IWL_DL_INFO, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF_PREFIX(_subsys, IWL_DL_FW | IWL_DL_INFO, IWL_DL_FW_INFO, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_HC(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_HC, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_HC, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_HT(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_HT, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_HT, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_INFO(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_INFO, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_INFO, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_ISR(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_ISR, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_ISR, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_LAR(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_LAR, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_LAR, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_MAC80211(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_MAC80211, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_MAC80211, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_POWER(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_POWER, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_POWER, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_QUOTA(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_QUOTA, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_QUOTA, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_RADIO(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_RADIO, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_RADIO, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_RATE(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_RATE, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_RATE, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_RF_KILL(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_RF_KILL, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_RF_KILL, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_RX(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_RX, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_RX, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_SCAN(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_SCAN, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_SCAN, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_STATS(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_STATS, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_STATS, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_STATS_LIMIT(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_STATS, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_STATS, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_TDLS(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_TDLS, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_TDLS, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_TE(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_TE, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_TE, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_TEMP(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_TEMP, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_TEMP, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_TPT(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_TPT, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_TPT, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_TX(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_TX, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_TX, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_TX_QUEUES(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_TX_QUEUES, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_TX_QUEUES, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_TX_REPLY(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_TX_REPLY, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_TX_REPLY, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_WEP(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_WEP, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_WEP, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_WOWLAN(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_WOWLAN, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_WOWLAN, _fmt, ##__VA_ARGS__) #define IWL_DEBUG_PCI_RW(_subsys, _fmt, ...) \ - IWL_DPRINTF((_subsys), IWL_DL_PCI_RW, (_fmt), ##__VA_ARGS__) + IWL_DPRINTF(_subsys, IWL_DL_PCI_RW, _fmt, ##__VA_ARGS__) #endif /* _IWL_DEBUG_H */
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202202272343.21RNhbXK002999>