futex error should be a warning

* Matthew Khouzam (matthew.khouzam@ericsson.com) wrote:
> Hi tracing warlords,
> I am testing UST 2.0 on ubuntu Lucid Lynx, and I got the following
> message while running a program:
> libust[22376/22377]: Error: Linux kernels 2.6.33 to 3.0 (with the
> exception of stable versions) do not support FUTEX_WAKE on read-only
> memory mappings correctly. Please upgrade your kernel (fix is commit
> 9ea71503a8ed9184d2d0b8ccc4d269d05f7940ae in Linux kernel mainline).
> LTTng-UST will use polling mode fallback. (in wait_for_sessiond() at
> lttng-ust-comm.c:564)
>
> I think since there is a fallback, and this is not a showstopper just an
> irritation that it should be a warning.

Reported-by: Matthew Khouzam <matthew.khouzam@ericsson.com>
Signed-off-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

diff --git a/libust/lttng-ust-comm.c b/libust/lttng-ust-comm.c
index ba60053627ace1c0bbf01737890272f4b90178ba..5f51812566e52d5e2ae5a9fb96160f4ccaeb8f66 100644 (file)
--- a/libust/lttng-ust-comm.c
+++ b/libust/lttng-ust-comm.c
@@ -556,7 +556,7 @@ void wait_for_sessiond(struct sock_info *sock_info)
                if (ret < 0) {
                        if (errno == EFAULT) {
                                wait_poll_fallback = 1;
-                               ERR(
+                               WARN(
 "Linux kernels 2.6.33 to 3.0 (with the exception of stable versions) "
 "do not support FUTEX_WAKE on read-only memory mappings correctly. "
 "Please upgrade your kernel "