From: mvs <mvs@openbsd.org>
Date: Mon, 2 Aug 2021 21:10:55 +0000 (+0000)
Subject: Don't call rtm_ifchg() in trunk_port_state().
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=60a682ede06ee952e3392035baf97763a51a8beb;p=openbsd

Don't call rtm_ifchg() in trunk_port_state().

The preceding trunk_link_active() already produced RTM_IFINFO message when
trunk(4) state was changed. I such case we double RTM_IFINFO message or we
produce false message when trunk(4) state was not changed.

ok florian@
---

diff --git a/sys/net/if_trunk.c b/sys/net/if_trunk.c
index 828331a534f..1efc8ab4389 100644
--- a/sys/net/if_trunk.c
+++ b/sys/net/if_trunk.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: if_trunk.c,v 1.151 2021/01/28 20:04:44 mvs Exp $	*/
+/*	$OpenBSD: if_trunk.c,v 1.152 2021/08/02 21:10:55 mvs Exp $	*/
 
 /*
  * Copyright (c) 2005, 2006, 2007 Reyk Floeter <reyk@openbsd.org>
@@ -1244,7 +1244,6 @@ trunk_port_state(void *arg)
 	if (tr->tr_linkstate != NULL)
 		(*tr->tr_linkstate)(tp);
 	trunk_link_active(tr, tp);
-	rtm_ifchg(&tr->tr_ac.ac_if);
 }
 
 struct trunk_port *