git: 840f478eed2a - main - net80211: in net80211_vap_printf() also use vprintf()
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Tue, 10 Feb 2026 00:46:41 UTC
The branch main has been updated by bz:
URL: https://cgit.FreeBSD.org/src/commit/?id=840f478eed2ab18abd1088aa12587bb708a46b56
commit 840f478eed2ab18abd1088aa12587bb708a46b56
Author: Bjoern A. Zeeb <bz@FreeBSD.org>
AuthorDate: 2026-01-20 13:43:51 +0000
Commit: Bjoern A. Zeeb <bz@FreeBSD.org>
CommitDate: 2026-02-09 21:40:47 +0000
net80211: in net80211_vap_printf() also use vprintf()
While everything else uses vprintf() and net80211_vap_printf()
vlog() the debug output of wlandebug sessions can be weird.
For consistency use vprintf() everywhere to have homogeneous logging.
Sponosred by: The FreeBSD Foundation
MFC after: 3 days
Reviewed by: adrian
Differential Revision: https://reviews.freebsd.org/D54795
---
sys/net80211/ieee80211_freebsd.c | 10 ++++------
1 file changed, 4 insertions(+), 6 deletions(-)
diff --git a/sys/net80211/ieee80211_freebsd.c b/sys/net80211/ieee80211_freebsd.c
index b1380eb8a81c..095be79e7bfe 100644
--- a/sys/net80211/ieee80211_freebsd.c
+++ b/sys/net80211/ieee80211_freebsd.c
@@ -1324,7 +1324,7 @@ net80211_vap_printf(const struct ieee80211vap *vap, const char *fmt, ...)
va_start(ap, fmt);
snprintf(if_fmt, sizeof(if_fmt), "%s: %s", if_name(vap->iv_ifp), fmt);
- vlog(LOG_INFO, if_fmt, ap);
+ vprintf(if_fmt, ap);
va_end(ap);
}
@@ -1334,14 +1334,12 @@ net80211_vap_printf(const struct ieee80211vap *vap, const char *fmt, ...)
void
net80211_ic_printf(const struct ieee80211com *ic, const char *fmt, ...)
{
+ char if_fmt[256];
va_list ap;
- /*
- * TODO: do the vap_printf stuff above, use vlog(LOG_INFO, ...)
- */
- printf("%s: ", ic->ic_name);
+ snprintf(if_fmt, sizeof(if_fmt), "%s: %s", ic->ic_name, fmt);
va_start(ap, fmt);
- vprintf(fmt, ap);
+ vprintf(if_fmt, ap);
va_end(ap);
}