ratelimit: fix log levels of log suppression messages

When logging a suppression message do so on the same log level as the
suppressed messages.
This commit is contained in:
Lennart Poettering 2011-01-22 01:08:36 +01:00
parent eb966f7455
commit ec5a785712
11 changed files with 17 additions and 16 deletions

View file

@ -439,7 +439,7 @@ static size_t check_left_to_play(struct userdata *u, size_t n_bytes, pa_bool_t o
#endif
if (!u->first && !u->after_rewind)
if (pa_log_ratelimit())
if (pa_log_ratelimit(PA_LOG_INFO))
pa_log_info("Underrun!");
}
@ -1440,7 +1440,7 @@ static void thread_func(void *userdata) {
* we have filled the buffer at least once
* completely.*/
if (pa_log_ratelimit())
if (pa_log_ratelimit(PA_LOG_DEBUG))
pa_log_debug("Cutting sleep time for the initial iterations by half.");
sleep_usec /= 2;
}
@ -1492,7 +1492,7 @@ static void thread_func(void *userdata) {
u->first = TRUE;
u->since_start = 0;
} else if (revents && u->use_tsched && pa_log_ratelimit())
} else if (revents && u->use_tsched && pa_log_ratelimit(PA_LOG_DEBUG))
pa_log_debug("Wakeup from ALSA!");
} else

View file

@ -428,7 +428,7 @@ static size_t check_left_to_record(struct userdata *u, size_t n_bytes, pa_bool_t
PA_DEBUG_TRAP;
#endif
if (pa_log_ratelimit())
if (pa_log_ratelimit(PA_LOG_INFO))
pa_log_info("Overrun!");
}
@ -1315,7 +1315,7 @@ static void thread_func(void *userdata) {
goto fail;
u->first = TRUE;
} else if (revents && u->use_tsched && pa_log_ratelimit())
} else if (revents && u->use_tsched && pa_log_ratelimit(PA_LOG_DEBUG))
pa_log_debug("Wakeup from ALSA!");
} else

View file

@ -321,7 +321,7 @@ static void verify_access(struct userdata *u, struct device *d) {
* during opening was canceled by a "try again"
* failure or a "fatal" failure. */
if (pa_ratelimit_test(&d->ratelimit)) {
if (pa_ratelimit_test(&d->ratelimit, PA_LOG_DEBUG)) {
pa_log_debug("Loading module-alsa-card with arguments '%s'", d->args);
m = pa_module_load(u->core, "module-alsa-card", d->args);

View file

@ -206,7 +206,7 @@ void pa_asyncq_post(pa_asyncq*l, void *p) {
/* OK, we couldn't push anything in the queue. So let's queue it
* locally and push it later */
if (pa_log_ratelimit())
if (pa_log_ratelimit(PA_LOG_WARN))
pa_log_warn("q overrun, queuing locally");
if (!(q = pa_flist_pop(PA_STATIC_FLIST_GET(localq))))

View file

@ -431,7 +431,7 @@ void pa_log_level(pa_log_level_t level, const char *format, ...) {
va_end(ap);
}
pa_bool_t pa_log_ratelimit(void) {
pa_bool_t pa_log_ratelimit(pa_log_level_t level) {
/* Not more than 10 messages every 5s */
static PA_DEFINE_RATELIMIT(ratelimit, 5 * PA_USEC_PER_SEC, 10);
@ -440,5 +440,5 @@ pa_bool_t pa_log_ratelimit(void) {
if (no_rate_limit)
return TRUE;
return pa_ratelimit_test(&ratelimit);
return pa_ratelimit_test(&ratelimit, level);
}

View file

@ -135,6 +135,6 @@ LOG_FUNC(error, PA_LOG_ERROR)
#define pa_log pa_log_error
pa_bool_t pa_log_ratelimit(void);
pa_bool_t pa_log_ratelimit(pa_log_level_t level);
#endif

View file

@ -258,7 +258,7 @@ static struct mempool_slot* mempool_allocate_slot(pa_mempool *p) {
slot = (struct mempool_slot*) ((uint8_t*) p->memory.ptr + (p->block_size * (size_t) idx));
if (!slot) {
if (pa_log_ratelimit())
if (pa_log_ratelimit(PA_LOG_DEBUG))
pa_log_debug("Pool full");
pa_atomic_inc(&p->stat.n_pool_full);
return NULL;

View file

@ -1384,7 +1384,7 @@ static int sink_input_process_msg(pa_msgobject *o, int code, void *userdata, int
if (pa_memblockq_push_align(s->memblockq, chunk) < 0) {
if (pa_log_ratelimit())
if (pa_log_ratelimit(PA_LOG_WARN))
pa_log_warn("Failed to push data into queue");
pa_asyncmsgq_post(pa_thread_mq_get()->outq, PA_MSGOBJECT(s), PLAYBACK_STREAM_MESSAGE_OVERFLOW, NULL, 0, NULL, NULL);
pa_memblockq_seek(s->memblockq, (int64_t) chunk->length, PA_SEEK_RELATIVE, TRUE);

View file

@ -832,7 +832,7 @@ static int do_read(pa_pstream *p) {
ntohl(p->read.shm_info[PA_PSTREAM_SHM_INDEX]),
ntohl(p->read.shm_info[PA_PSTREAM_SHM_LENGTH])))) {
if (pa_log_ratelimit())
if (pa_log_ratelimit(PA_LOG_DEBUG))
pa_log_debug("Failed to import memory block.");
}

View file

@ -35,7 +35,7 @@ static pa_static_mutex mutex = PA_STATIC_MUTEX_INIT;
/* Modelled after Linux' lib/ratelimit.c by Dave Young
* <hidave.darkstar@gmail.com>, which is licensed GPLv2. */
pa_bool_t pa_ratelimit_test(pa_ratelimit *r) {
pa_bool_t pa_ratelimit_test(pa_ratelimit *r, pa_log_level_t t) {
pa_usec_t now;
pa_mutex *m;
@ -52,7 +52,7 @@ pa_bool_t pa_ratelimit_test(pa_ratelimit *r) {
r->begin + r->interval < now) {
if (r->n_missed > 0)
pa_log_warn("%u events suppressed", r->n_missed);
pa_logl(t, "%u events suppressed", r->n_missed);
r->begin = now;

View file

@ -23,6 +23,7 @@
***/
#include <pulse/sample.h>
#include <pulsecore/log.h>
#include <pulsecore/macro.h>
typedef struct pa_ratelimit {
@ -51,6 +52,6 @@ typedef struct pa_ratelimit {
r->begin = 0; \
} while (FALSE);
pa_bool_t pa_ratelimit_test(pa_ratelimit *r);
pa_bool_t pa_ratelimit_test(pa_ratelimit *r, pa_log_level_t t);
#endif