summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--TODO1
-rw-r--r--nest/config.Y3
-rw-r--r--nest/iface.c55
-rw-r--r--nest/protocol.h7
4 files changed, 47 insertions, 19 deletions
diff --git a/TODO b/TODO
index 3f7db5c..5ba0433 100644
--- a/TODO
+++ b/TODO
@@ -1,6 +1,5 @@
Core
~~~~
-- debug: interfaces
- debug: static
- debug: pipe
- debug: krt
diff --git a/nest/config.Y b/nest/config.Y
index a4863dd..f4e8cdd 100644
--- a/nest/config.Y
+++ b/nest/config.Y
@@ -19,7 +19,7 @@ CF_DECLS
CF_KEYWORDS(ROUTER, ID, PROTOCOL, PREFERENCE, DISABLED, DEBUG, ALL, OFF, DIRECT)
CF_KEYWORDS(INTERFACE, IMPORT, EXPORT, FILTER, NONE, TABLE, STATES, ROUTES, FILTERS)
-CF_KEYWORDS(PASSWORD, FROM, PASSIVE, TO, ID, EVENTS, PACKETS, PROTOCOLS)
+CF_KEYWORDS(PASSWORD, FROM, PASSIVE, TO, ID, EVENTS, PACKETS, PROTOCOLS, INTERFACES)
CF_ENUM(T_ENUM_RTS, RTS_, DUMMY, STATIC, INHERIT, DEVICE, STATIC_DEVICE, REDIRECT,
RIP, RIP_EXT, OSPF, OSPF_EXT, OSPF_IA, OSPF_BOUNDARY, BGP, PIPE)
@@ -177,6 +177,7 @@ debug_flag:
STATES { $$ = D_STATES; }
| ROUTES { $$ = D_ROUTES; }
| FILTERS { $$ = D_FILTERS; }
+ | INTERFACES { $$ = D_IFACES; }
| EVENTS { $$ = D_EVENTS; }
| PACKETS { $$ = D_PACKETS; }
;
diff --git a/nest/iface.c b/nest/iface.c
index 4fa2c72..cf91833 100644
--- a/nest/iface.c
+++ b/nest/iface.c
@@ -6,7 +6,7 @@
* Can be freely distributed and used under the terms of the GNU GPL.
*/
-#define LOCAL_DEBUG
+#undef LOCAL_DEBUG
#include "nest/bird.h"
#include "nest/iface.h"
@@ -99,15 +99,44 @@ if_copy(struct iface *to, struct iface *from)
to->mtu = from->mtu;
}
+static inline void
+ifa_send_notify(struct proto *p, unsigned c, struct ifa *a)
+{
+ if (p->ifa_notify)
+ {
+ if (p->debug & D_IFACES)
+ log(L_TRACE "%s < %s address %I/%d on interface %s %s",
+ p->name, (a->flags & IA_PRIMARY) ? "primary" : "secondary",
+ a->prefix, a->pxlen, a->iface->name,
+ (c & IF_CHANGE_UP) ? "added" : "removed");
+ p->ifa_notify(p, c, a);
+ }
+}
+
static void
ifa_notify_change(unsigned c, struct ifa *a)
{
struct proto *p;
- debug("IFA change notification (%x) for %s:%I\n", c, a->iface->name, a->ip);
+ DBG("IFA change notification (%x) for %s:%I\n", c, a->iface->name, a->ip);
WALK_LIST(p, active_proto_list)
- if (p->ifa_notify)
- p->ifa_notify(p, c, a);
+ ifa_send_notify(p, c, a);
+}
+
+static inline void
+if_send_notify(struct proto *p, unsigned c, struct iface *i)
+{
+ if (p->if_notify)
+ {
+ if (p->debug & D_IFACES)
+ log(L_TRACE "%s < interface %s %s", p->name, i->name,
+ (c & IF_CHANGE_UP) ? "goes up" :
+ (c & IF_CHANGE_DOWN) ? "goes down" :
+ (c & IF_CHANGE_MTU) ? "changes MTU" :
+ (c & IF_CHANGE_CREATE) ? "created" :
+ "sends unknown event");
+ p->if_notify(p, c, i);
+ }
}
static void
@@ -122,7 +151,7 @@ if_notify_change(unsigned c, struct iface *i)
c |= IF_CHANGE_CREATE | IF_CHANGE_MTU;
}
- debug("Interface change notification (%x) for %s\n", c, i->name);
+ DBG("Interface change notification (%x) for %s\n", c, i->name);
if_dump(i);
if (c & IF_CHANGE_UP)
@@ -135,8 +164,7 @@ if_notify_change(unsigned c, struct iface *i)
}
WALK_LIST(p, active_proto_list)
- if (p->if_notify)
- p->if_notify(p, c, i);
+ if_send_notify(p, c, i);
if (c & IF_CHANGE_UP)
WALK_LIST(a, i->addrs)
@@ -261,16 +289,15 @@ if_feed_baby(struct proto *p)
struct iface *i;
struct ifa *a;
- if (!p->if_notify && !p->ifa_notify)
+ if (!p->if_notify && !p->ifa_notify) /* shortcut */
return;
- debug("Announcing interfaces to new protocol %s\n", p->name);
+ DBG("Announcing interfaces to new protocol %s\n", p->name);
WALK_LIST(i, iface_list)
{
- if (p->if_notify)
- p->if_notify(p, IF_CHANGE_CREATE | ((i->flags & IF_UP) ? IF_CHANGE_UP : 0), i);
- if (p->ifa_notify && (i->flags & IF_UP))
+ if_send_notify(p, IF_CHANGE_CREATE | ((i->flags & IF_UP) ? IF_CHANGE_UP : 0), i);
+ if (i->flags & IF_UP)
WALK_LIST(a, i->addrs)
- p->ifa_notify(p, IF_CHANGE_CREATE | IF_CHANGE_UP, a);
+ ifa_send_notify(p, IF_CHANGE_CREATE | IF_CHANGE_UP, a);
}
}
@@ -399,7 +426,7 @@ auto_router_id(void)
j = i;
if (!j)
die("Cannot determine router ID (no suitable network interface found), please configure it manually");
- debug("Guessed router ID %I (%s)\n", j->addr->ip, j->name);
+ log(L_INFO, "Guessed router ID %I according to interface %s", j->addr->ip, j->name);
config->router_id = ipa_to_u32(j->addr->ip);
#endif
}
diff --git a/nest/protocol.h b/nest/protocol.h
index f4ce384..9dd32fc 100644
--- a/nest/protocol.h
+++ b/nest/protocol.h
@@ -240,9 +240,10 @@ void proto_notify_state(struct proto *p, unsigned state);
#define D_STATES 1 /* [core] State transitions */
#define D_ROUTES 2 /* [core] Routes passed by the filters */
-#define D_FILTERS 4 /* [core] Filtering of routes */
-#define D_EVENTS 8 /* Protocol events */
-#define D_PACKETS 16 /* Packets sent/received */
+#define D_FILTERS 4 /* [core] Routes rejected by the filters */
+#define D_IFACES 8 /* [core] Interface events */
+#define D_EVENTS 16 /* Protocol events */
+#define D_PACKETS 32 /* Packets sent/received */
/*
* Known unique protocol instances as referenced by config routines