From 1cf97ac4d0cd1658243075c968ff2af038a74e06 Mon Sep 17 00:00:00 2001 From: Yongrong Wang Date: Wed, 17 Jul 2024 14:09:01 +0800 Subject: [PATCH] rpmsg_ping.c: improve log level to LOG_EMERG Usually rpmsg ping logs are needed during testing, but too many other module logs can easily overwrite it, so improve the rpmsg ping log level, later we can setlogmask to a higher log level to mask other modules' logs. Signed-off-by: Yongrong Wang --- drivers/rpmsg/rpmsg_ping.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/drivers/rpmsg/rpmsg_ping.c b/drivers/rpmsg/rpmsg_ping.c index 7849637cb68..d615c3547ae 100644 --- a/drivers/rpmsg/rpmsg_ping.c +++ b/drivers/rpmsg/rpmsg_ping.c @@ -181,9 +181,9 @@ static void rpmsg_ping_logout(FAR const char *s, clock_t value) perf_convert(value, &ts); #ifdef CONFIG_SYSTEM_TIME64 - syslog(LOG_INFO, "%s: %" PRIu64 " s, %ld ns\n", s, ts.tv_sec, ts.tv_nsec); + syslog(LOG_EMERG, "%s: %" PRIu64 " s, %ld ns\n", s, ts.tv_sec, ts.tv_nsec); #else - syslog(LOG_INFO, "%s: %" PRIu32 " s, %ld ns\n", s, ts.tv_sec, ts.tv_nsec); + syslog(LOG_EMERG, "%s: %" PRIu32 " s, %ld ns\n", s, ts.tv_sec, ts.tv_nsec); #endif } @@ -200,7 +200,7 @@ static void rpmsg_ping_logout_rate(uint64_t len, clock_t avg) rateint = ratebits / 1000000; ratedec = ratebits - rateint * 1000000; - syslog(LOG_INFO, "rate: %zu.%06zu Mbits/sec\n", rateint, ratedec); + syslog(LOG_EMERG, "rate: %zu.%06zu Mbits/sec\n", rateint, ratedec); } /**************************************************************************** @@ -243,7 +243,9 @@ int rpmsg_ping(FAR struct rpmsg_endpoint *ept, } } - syslog(LOG_INFO, "ping times: %d\n", ping->times); + syslog(LOG_EMERG, "ping times: %d\n", ping->times); + syslog(LOG_EMERG, "buffer_len: %" PRIu32 ", send_len: %d\n", + buf_len, send_len); rpmsg_ping_logout("avg", total / ping->times); rpmsg_ping_logout("min", min);