From: <erik.hugne@xxxxxxxxxxxx> Date: Tue, 4 Mar 2014 09:05:38 +0100 > @@ -401,7 +401,7 @@ static void cfg_conn_msg_event(int conid, struct sockaddr_tipc *addr, > > ret = tipc_conn_sendmsg(&cfgsrv, conid, addr, rep_buf->data, > rep_buf->len); > - if (ret < 0) > + if (ret == -ENOMEM) > pr_err("Sending cfg reply message failed, no memory\n"); > > kfree_skb(rep_buf); Even -ENOMEM doesn't deserve a pr_err() log message. The memory allocators in the kernel generate log messages on memory allocation failure, when appropriate, with full stack backtraces. Please just remove this test entirely, thanks. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html