From e46efdcd94b8efef0ae2fee15faade349a5cff47 Mon Sep 17 00:00:00 2001 From: Krzysztof Chruscinski Date: Thu, 16 Sep 2021 07:26:35 +0200 Subject: [PATCH] lib: os: mpsc_pbuf: Improve debugging Improved debugging messages. Signed-off-by: Krzysztof Chruscinski --- lib/os/mpsc_pbuf.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/lib/os/mpsc_pbuf.c b/lib/os/mpsc_pbuf.c index 4fafa4eaefd..ba4dae102a0 100644 --- a/lib/os/mpsc_pbuf.c +++ b/lib/os/mpsc_pbuf.c @@ -19,7 +19,7 @@ static inline void mpsc_state_print(struct mpsc_pbuf_buffer *buffer) { if (MPSC_PBUF_DEBUG) { - printk("wr:%d/%d, rd:%d/%d\n", + printk(", wr:%d/%d, rd:%d/%d\n", buffer->wr_idx, buffer->tmp_wr_idx, buffer->rd_idx, buffer->tmp_rd_idx); } @@ -242,7 +242,7 @@ union mpsc_pbuf_generic *mpsc_pbuf_alloc(struct mpsc_pbuf_buffer *buffer, uint32_t free_wlen; bool valid_drop; - MPSC_PBUF_DBG(buffer, "alloc %d words, ", (int)wlen); + MPSC_PBUF_DBG(buffer, "alloc %d words", (int)wlen); if (wlen > (buffer->size - 1)) { MPSC_PBUF_DBG(buffer, "Failed to alloc, "); @@ -296,7 +296,7 @@ union mpsc_pbuf_generic *mpsc_pbuf_alloc(struct mpsc_pbuf_buffer *buffer, } } while (cont); - MPSC_PBUF_DBG(buffer, "allocated %p ", item); + MPSC_PBUF_DBG(buffer, "allocated %p", item); if (IS_ENABLED(CONFIG_MPSC_CLEAR_ALLOCATED) && item) { /* During test fill with 0's to simplify message comparison */ @@ -317,7 +317,7 @@ void mpsc_pbuf_commit(struct mpsc_pbuf_buffer *buffer, buffer->wr_idx = idx_inc(buffer, buffer->wr_idx, wlen); max_utilization_update(buffer); k_spin_unlock(&buffer->lock, key); - MPSC_PBUF_DBG(buffer, "committed %p ", item); + MPSC_PBUF_DBG(buffer, "committed %p", item); } void mpsc_pbuf_put_word_ext(struct mpsc_pbuf_buffer *buffer, @@ -456,7 +456,7 @@ const union mpsc_pbuf_generic *mpsc_pbuf_claim(struct mpsc_pbuf_buffer *buffer) } if (!cont) { - MPSC_PBUF_DBG(buffer, "claimed: %p ", item); + MPSC_PBUF_DBG(buffer, "claimed: %p", item); } k_spin_unlock(&buffer->lock, key); } while (cont); @@ -479,7 +479,7 @@ void mpsc_pbuf_free(struct mpsc_pbuf_buffer *buffer, } else { witem->skip.len = wlen; } - MPSC_PBUF_DBG(buffer, "freed: %p ", item); + MPSC_PBUF_DBG(buffer, "freed: %p", item); k_spin_unlock(&buffer->lock, key); k_sem_give(&buffer->sem);