Hi,

Currently in ipv4_input.c we have a warning that a packet was not for us and was dropped. This pollutes the output of the console for no real value when connected to a real network.

I suggest we turn this warning into an info to keep things clean.

The file contains real warnings that are much more important and relevant that this line.


Could be configurable (with a potential CONFIG_NET_IP_NOTFORUS_WARN) or fixed, here is the relevant code.

I can send a PR but I would like your general feeling on this issue first, thank you!

Sebastien


diff --git a/net/devif/ipv4_input.c b/net/devif/ipv4_input.c
index bb16940cbf..bb109f353d 100644
--- a/net/devif/ipv4_input.c
+++ b/net/devif/ipv4_input.c
@@ -310,8 +310,13 @@ int ipv4_input(FAR struct net_driver_s *dev)
                * packet.
                */

+#ifdef CONFIG_NET_IP_NOTFORUS_WARN
               nwarn("WARNING: Not destined for us; not forwardable... "
                     "Dropping!\n");
+#else
+              ninfo("WARNING: Not destined for us; not forwardable... "
+                    "Dropping!\n");
+#endif

 #ifdef CONFIG_NET_STATISTICS
               g_netstats.ipv4.drop++;

Reply via email to