realtek: l3: log route changes at debug level

Every route that is added prints three info lines and every route that
is deleted prints two. The printing holds the work queue that programs
the routes: with a serial console at 115200 baud, 1800 routes through
one gateway take 77 seconds to reach the hardware on RTL9303 and 53
seconds to leave it.

Print these lines at debug level, and with them the warning for the
addresses the driver never offloads. The same routes are then
programmed in 4 seconds and removed in 5 or less. Errors stay where
they were.

Assisted-by: Claude:claude-opus-5-5
Signed-off-by: Quinn O'Shea <qoshea@gmail.com>
Link: openwrt/openwrt#25885
Signed-off-by: Markus Stockhausen <markus.stockhausen@gmx.de>

Author: bloqaudio

target/linux/realtek/files-6.18/drivers/net/dsa/rtl83xx/l3.c

@@ -2043,11 +2043,11 @@ static void otto_l3_route_update_hw(struct otto_l3_ctrl *ctrl, struct otto_l3_ro
 		  r->nh.port == priv->r->port_ignore;
 	if ((no_port || no_l2) && (first || !trapped)) {
 		if (r->attr.type == ROUTE_TYPE_IP4UC)
-			dev_info(ctrl->dev, "no port for %pI4, routing %s in software\n",
-				 &r->gw_ip.s6_addr32[3], otto_l3_route_dst(r, dst, sizeof(dst)));
+			dev_dbg(ctrl->dev, "no port for %pI4, routing %s in software\n",
+				&r->gw_ip.s6_addr32[3], otto_l3_route_dst(r, dst, sizeof(dst)));
 		else
-			dev_info(ctrl->dev, "no port for %pI6c, routing %s in software\n",
-				 &r->gw_ip, otto_l3_route_dst(r, dst, sizeof(dst)));
+			dev_dbg(ctrl->dev, "no port for %pI6c, routing %s in software\n",
+				&r->gw_ip, otto_l3_route_dst(r, dst, sizeof(dst)));
 	}
 
 	trap = no_port || no_l2 || otto_l3_fwd_off(ctrl, r->attr.type) ||
@@ -2073,7 +2073,7 @@ static void otto_l3_route_update_hw(struct otto_l3_ctrl *ctrl, struct otto_l3_ro
 	if (!ctrl->cfg->use_l3_tables && r->pr.packet_cntr < 0) {
 		r->pr.packet_cntr = otto_stats_packet_cntr_alloc(priv);
 		if (r->pr.packet_cntr >= 0)
-			dev_info(ctrl->dev, "Using packet counter %d\n", r->pr.packet_cntr);
+			dev_dbg(ctrl->dev, "Using packet counter %d\n", r->pr.packet_cntr);
 	}
 
 	otto_l3_route_install(ctrl, r);
@@ -2111,11 +2111,11 @@ static void otto_l3_route_trap_hw(struct otto_l3_ctrl *ctrl, struct otto_l3_rout
 		return;
 
 	if (r->attr.type == ROUTE_TYPE_IP4UC)
-		dev_info(ctrl->dev, "no valid neighbour for %pI4, routing %s in software\n",
-			 &r->gw_ip.s6_addr32[3], otto_l3_route_dst(r, dst, sizeof(dst)));
+		dev_dbg(ctrl->dev, "no valid neighbour for %pI4, routing %s in software\n",
+			&r->gw_ip.s6_addr32[3], otto_l3_route_dst(r, dst, sizeof(dst)));
 	else
-		dev_info(ctrl->dev, "no valid neighbour for %pI6c, routing %s in software\n",
-			 &r->gw_ip, otto_l3_route_dst(r, dst, sizeof(dst)));
+		dev_dbg(ctrl->dev, "no valid neighbour for %pI6c, routing %s in software\n",
+			&r->gw_ip, otto_l3_route_dst(r, dst, sizeof(dst)));
 
 	otto_l3_route_rewrite(ctrl, r);
 }
@@ -2218,11 +2218,11 @@ static int otto_l3_port_gw_resolve(struct otto_l3_ctrl *ctrl, struct otto_l3_rou
 		read_lock_bh(&n->lock);
 		mac = ether_addr_to_u64(n->ha);
 		read_unlock_bh(&n->lock);
-		dev_info(ctrl->dev, "resolved mac: %016llx\n", mac);
+		dev_dbg(ctrl->dev, "resolved mac: %016llx\n", mac);
 		otto_l3_route_update_hw(ctrl, r, mac);
 		otto_l3_fib6_flags_report(r);
 	} else {
-		dev_info(ctrl->dev, "need to wait\n");
+		dev_dbg(ctrl->dev, "need to wait\n");
 		neigh_event_send(n, NULL);
 	}
 
@@ -2723,12 +2723,12 @@ static int otto_l3_fib_check_v4(struct otto_l3_ctrl *ctrl,
 	if (nh->fib_nh_gw4)
 		snprintf(gw_message, sizeof(gw_message), "via %pI4 ", &nh->fib_nh_gw4);
 
-	dev_info(ctrl->dev, "%s IPv4 route %pI4/%d %s(VLAN %d, MAC %pM)\n",
-		 event == FIB_EVENT_ENTRY_ADD ? "add" : "delete",
-		 &info->dst, info->dst_len, gw_message, vlan, ndev->dev_addr);
+	dev_dbg(ctrl->dev, "%s IPv4 route %pI4/%d %s(VLAN %d, MAC %pM)\n",
+		event == FIB_EVENT_ENTRY_ADD ? "add" : "delete",
+		&info->dst, info->dst_len, gw_message, vlan, ndev->dev_addr);
 
 	if ((info->type == RTN_BROADCAST) || ipv4_is_loopback(info->dst) || !info->dst) {
-		dev_warn(ctrl->dev, "skip loopback/broadcast addresses and default routes\n");
+		dev_dbg(ctrl->dev, "skip loopback/broadcast addresses and default routes\n");
 		return -EINVAL;
 	}
 
@@ -2883,7 +2883,7 @@ static int otto_l3_fib_add_v4(struct otto_l3_ctrl *ctrl, struct fib_entry_notifi
 				    ROUTE_HOST : ROUTE_PREFIX);
 
 	if (route)
-		dev_info(ctrl->dev, "route allocated for gw %pI4\n", &nh->fib_nh_gw4);
+		dev_dbg(ctrl->dev, "route allocated for gw %pI4\n", &nh->fib_nh_gw4);
 	else {
 		dev_err(ctrl->dev, "could not allocate a route for gw %pI4\n",
 			&nh->fib_nh_gw4);
@@ -2997,8 +2997,8 @@ static int otto_l3_fib_del_v4(struct otto_l3_ctrl *ctrl, struct fib_entry_notifi
 			&info->dst, info->dst_len, &nh->fib_nh_gw4);
 		return -ENOENT;
 	}
-	dev_info(ctrl->dev, "found a route with id %d, nh-id %d\n",
-		 route->id, route->nh.id);
+	dev_dbg(ctrl->dev, "found a route with id %d, nh-id %d\n",
+		route->id, route->nh.id);
 
 	otto_l3_route_teardown(ctrl, route);
 
@@ -3011,10 +3011,6 @@ static int otto_l3_fib_del_v4(struct otto_l3_ctrl *ctrl, struct fib_entry_notifi
  * next hop array is not allocated behind it at all, so nothing that reads
  * one may run before the object itself has been ruled out.
  *
- * The IPv4 twin of this narrates every event at info level. IPv6 routes
- * arrive in numbers IPv4 ones do not - every router advertisement brings
- * some - so this one speaks at debug level instead.
- *
  * The members are the ones the notifier counted. The route's own sibling
  * count is read later, and a member deleted in between has had it cleared,
  * so it would pass for a route of its own.