Turn consumer "normal" error into debug statement (with comment)

Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

diff --git a/ltt-kconsumerd/ltt-kconsumerd.c b/ltt-kconsumerd/ltt-kconsumerd.c
index cd4b00e0128013f59a602323557a6704c2fd8ec3..ca939656b7c888d84173d460aa4582d6bbad2e22 100644 (file)
--- a/ltt-kconsumerd/ltt-kconsumerd.c
+++ b/ltt-kconsumerd/ltt-kconsumerd.c
@@ -207,7 +207,13 @@ static int read_subbuffer(struct lttng_kconsumerd_fd *kconsumerd_fd)
        err = kernctl_get_next_subbuf(infd);
        if (err != 0) {
                ret = errno;
        err = kernctl_get_next_subbuf(infd);
        if (err != 0) {
                ret = errno;

-               perror("Reserving sub buffer failed (everything is normal, "
+               /*
+                * This is a debug message even for single-threaded consumer,
+                * because poll() have more relaxed criterions than get subbuf,
+                * so get_subbuf may fail for short race windows where poll()
+                * would issue wakeups.
+                */
+               DBG("Reserving sub buffer failed (everything is normal, "

                                "it is due to concurrency)");
                goto end;
        }
                                "it is due to concurrency)");
                goto end;
        }