netgraph7: rewrite netgraph core stage 3/many.
authorNuno Antunes <nuno.antunes@gmail.com>
Tue, 7 Aug 2012 00:19:14 +0000 (01:19 +0100)
committerNuno Antunes <nuno.antunes@gmail.com>
Thu, 4 Oct 2012 13:01:36 +0000 (14:01 +0100)
* Remove unneeded #include.

* Remove NG_FREE_HOOK and NG_FREE_NODE macros.

* Remove TRAP_ERROR macro.

* Remove CTR* macros.

* Remove #if 0'd code.

sys/netgraph7/dragonfly.h
sys/netgraph7/netgraph/ng_base.c
sys/netgraph7/pppoe/ng_pppoe.c

index 50574e7..244d099 100644 (file)
@@ -84,12 +84,6 @@ typedef void *               uma_fini;
                        objcache_put(zone, item)
 #define uma_zone_set_max(zone, nitems)
 
-#define CTR1(ktr_line, ...)
-#define CTR2(ktr_line, ...)
-#define CTR3(ktr_line, ...)
-#define CTR4(ktr_line, ...)
-#define CTR5(ktr_line, ...)
-#define CTR6(ktr_line, ...)
 #define cpu_spinwait() cpu_pause()
 
 #define CTLFLAG_RDTUN  CTLFLAG_RD
index 5ba86e3..553a332 100644 (file)
@@ -65,8 +65,6 @@
 #include <sys/taskqueue.h>
 #include <machine/cpu.h>
 
-#include <net/netisr.h>
-
 #include <netgraph7/ng_message.h>
 #include <netgraph7/netgraph.h>
 #include <netgraph7/ng_parse.h>
@@ -200,7 +198,6 @@ int ng_path_parse(char *addr, char **node, char **path, char **hook);
 void   ng_rmnode(node_p node, hook_p dummy1, void *dummy2, int dummy3);
 void   ng_unname(node_p node);
 
-
 /* Our own netgraph malloc type */
 MALLOC_DEFINE(M_NETGRAPH, "netgraph", "netgraph structures and ctrl messages");
 MALLOC_DEFINE(M_NETGRAPH_HOOK, "netgraph_hook", "netgraph hook structures");
@@ -223,14 +220,6 @@ MALLOC_DEFINE(M_NETGRAPH_MSG, "netgraph_msg", "netgraph name storage");
 #define        NG_WORKLIST_UNLOCK()                    \
        mtx_unlock(&ng_worklist_mtx)
 
-#define NG_FREE_HOOK(hook) do { kfree((hook), M_NETGRAPH_HOOK); } while (0)
-#define NG_FREE_NODE(node) do { kfree((node), M_NETGRAPH_NODE); } while (0)
-
-/* Set this to kdb_enter("X") to catch all errors as they occur */
-#ifndef TRAP_ERROR
-#define TRAP_ERROR()
-#endif
-
 static ng_ID_t nextID = 1;
 
 #ifdef INVARIANTS
@@ -450,10 +439,8 @@ ng_make_node(const char *typename, node_p *nodepp)
        int     error;
 
        /* Check that the type makes sense */
-       if (typename == NULL) {
-               TRAP_ERROR();
+       if (typename == NULL)
                return (EINVAL);
-       }
 
        /* Locate the node type. If we fail we return. Do not try to load
         * module.
@@ -461,17 +448,7 @@ ng_make_node(const char *typename, node_p *nodepp)
        if ((type = ng_findtype(typename)) == NULL)
                return (ENXIO);
 
-       /*
-        * If we have a constructor, then make the node and
-        * call the constructor to do type specific initialisation.
-        */
-       if (type->constructor != NULL) {
-               if ((error = ng_make_node_common(type, nodepp)) == 0) {
-                       if ((error = ((*type->constructor)(*nodepp)) != 0)) {
-                               NG_NODE_UNREF(*nodepp);
-                       }
-               }
-       } else {
+       if (type->constructor == NULL) {
                /*
                 * Node has no constructor. We cannot ask for one
                 * to be made. It must be brought into existence by
@@ -479,9 +456,19 @@ ng_make_node(const char *typename, node_p *nodepp)
                 * call ng_make_node_common() directly to get the
                 * netgraph part initialised.
                 */
-               TRAP_ERROR();
-               error = EINVAL;
+               return (EINVAL);
        }
+
+       /*
+        * If we have a constructor, then make the node and
+        * call the constructor to do type specific initialisation.
+        */
+       if ((error = ng_make_node_common(type, nodepp)) == 0) {
+               if ((error = ((*type->constructor)(*nodepp)) != 0)) {
+                       NG_NODE_UNREF(*nodepp);
+               }
+       }
+
        return (error);
 }
 
@@ -497,7 +484,6 @@ ng_make_node_common(struct ng_type *type, node_p *nodepp)
 
        /* Require the node type to have been already installed */
        if (ng_findtype(type->name) == NULL) {
-               TRAP_ERROR();
                return (EINVAL);
        }
 
@@ -659,7 +645,7 @@ ng_unref_node(node_p node)
                mtx_unlock(&ng_idhash_mtx);
 
                mtx_uninit(&node->nd_input_queue.q_mtx);
-               NG_FREE_NODE(node);
+               kfree(node, M_NETGRAPH_NODE);
        }
        return (v - 1);
 }
@@ -704,18 +690,15 @@ ng_name_node(node_p node, const char *name)
                        break;
        }
        if (i == 0 || name[i] != '\0') {
-               TRAP_ERROR();
                return (EINVAL);
        }
        if (ng_decodeidname(name) != 0) { /* valid IDs not allowed here */
-               TRAP_ERROR();
                return (EINVAL);
        }
 
        /* Check the name isn't already being used */
        if ((node2 = ng_name2noderef(node, name)) != NULL) {
                NG_NODE_UNREF(node2);
-               TRAP_ERROR();
                return (EADDRINUSE);
        }
 
@@ -838,7 +821,7 @@ ng_unref_hook(hook_p hook)
        if (v == 1) { /* we were the last */
                if (_NG_HOOK_NODE(hook)) /* it'll probably be ng_deadnode */
                        _NG_NODE_UNREF((_NG_HOOK_NODE(hook)));
-               NG_FREE_HOOK(hook);
+               kfree(hook, M_NETGRAPH_HOOK);
        }
 }
 
@@ -854,11 +837,9 @@ ng_add_hook(node_p node, const char *name, hook_p *hookp)
 
        /* Check that the given name is good */
        if (name == NULL) {
-               TRAP_ERROR();
                return (EINVAL);
        }
        if (ng_findhook(node, name) != NULL) {
-               TRAP_ERROR();
                return (EEXIST);
        }
 
@@ -1019,7 +1000,6 @@ int
 ng_bypass(hook_p hook1, hook_p hook2)
 {
        if (hook1->hk_node != hook2->hk_node) {
-               TRAP_ERROR();
                return (EINVAL);
        }
        hook1->hk_peer->hk_peer = hook2->hk_peer;
@@ -1049,7 +1029,6 @@ ng_newtype(struct ng_type *tp)
        if ((tp->version != NG_ABI_VERSION)
        || (namelen == 0)
        || (namelen >= NG_TYPESIZ)) {
-               TRAP_ERROR();
                if (tp->version != NG_ABI_VERSION) {
                        printf("Netgraph: Node type rejected. ABI mismatch. Suggest recompile\n");
                }
@@ -1058,7 +1037,6 @@ ng_newtype(struct ng_type *tp)
 
        /* Check for name collision */
        if (ng_findtype(tp->name) != NULL) {
-               TRAP_ERROR();
                return (EEXIST);
        }
 
@@ -1080,7 +1058,6 @@ ng_rmtype(struct ng_type *tp)
 {
        /* Check for name collision */
        if (tp->refs != 1) {
-               TRAP_ERROR();
                return (EBUSY);
        }
 
@@ -1174,7 +1151,6 @@ ng_con_part2(node_p node, item_p item, hook_p hook)
         * Should not happen because we checked before queueing this.
         */
        if (ng_findhook(node, NG_HOOK_NAME(hook)) != NULL) {
-               TRAP_ERROR();
                ng_destroy_hook(hook); /* should destroy peer too */
                printf("failed in ng_con_part2()\n");
                ERROUT(EEXIST);
@@ -1504,7 +1480,6 @@ ng_path2noderef(node_p here, const char *address,
 
        /* Initialize */
        if (destp == NULL) {
-               TRAP_ERROR();
                return EINVAL;
        }
        *destp = NULL;
@@ -1515,7 +1490,6 @@ ng_path2noderef(node_p here, const char *address,
 
        /* Parse out node and sequence of hooks */
        if (ng_path_parse(fullpath, &nodename, &path, NULL) < 0) {
-               TRAP_ERROR();
                return EINVAL;
        }
        if (path == NULL) {
@@ -1532,12 +1506,10 @@ ng_path2noderef(node_p here, const char *address,
        if (nodename) {
                node = ng_name2noderef(here, nodename);
                if (node == NULL) {
-                       TRAP_ERROR();
                        return (ENOENT);
                }
        } else {
                if (here == NULL) {
-                       TRAP_ERROR();
                        return (EINVAL);
                }
                node = here;
@@ -1584,17 +1556,7 @@ ng_path2noderef(node_p here, const char *address,
                    || NG_HOOK_PEER(hook) == NULL
                    || NG_HOOK_NOT_VALID(hook)
                    || NG_HOOK_NOT_VALID(NG_HOOK_PEER(hook))) {
-                       TRAP_ERROR();
                        NG_NODE_UNREF(node);
-#if 0
-                       printf("hooknotvalid %s %s %d %d %d %d ",
-                                       path,
-                                       segment,
-                                       hook == NULL,
-                                       NG_HOOK_PEER(hook) == NULL,
-                                       NG_HOOK_NOT_VALID(hook),
-                                       NG_HOOK_NOT_VALID(NG_HOOK_PEER(hook)));
-#endif
                        return (ENOENT);
                }
 
@@ -1617,7 +1579,6 @@ ng_path2noderef(node_p here, const char *address,
 
        /* If node somehow missing, fail here (probably this is not needed) */
        if (node == NULL) {
-               TRAP_ERROR();
                return (ENXIO);
        }
 
@@ -1735,9 +1696,6 @@ ng_dequeue(node_p node, int *rw)
 
        /* If there is nothing queued, then just return. */
        if (!QUEUE_ACTIVE(ngq)) {
-               CTR4(KTR_NET, "%20s: node [%x] (%p) queue empty; "
-                   "queue flags 0x%lx", __func__,
-                   node->nd_ID, node, ngq->q_flags);
                return (NULL);
        }
 
@@ -1751,9 +1709,6 @@ ng_dequeue(node_p node, int *rw)
                        long t = ngq->q_flags;
                        if (t & WRITER_ACTIVE) {
                                /* There is writer, reader can't proceed. */
-                               CTR4(KTR_NET, "%20s: node [%x] (%p) queued reader "
-                                   "can't proceed; queue flags 0x%lx", __func__,
-                                   node->nd_ID, node, t);
                                return (NULL);
                        }
                        if (atomic_cmpset_acq_int(&ngq->q_flags, t,
@@ -1769,9 +1724,6 @@ ng_dequeue(node_p node, int *rw)
                *rw = NGQRW_W;
        } else {
                /* There is somebody other, writer can't proceed. */
-               CTR4(KTR_NET, "%20s: node [%x] (%p) queued writer "
-                   "can't proceed; queue flags 0x%lx", __func__,
-                   node->nd_ID, node, ngq->q_flags);
                return (NULL);
        }
 
@@ -1783,10 +1735,6 @@ ng_dequeue(node_p node, int *rw)
        STAILQ_REMOVE_HEAD(&ngq->queue, el_next);
        if (STAILQ_EMPTY(&ngq->queue))
                atomic_clear_int(&ngq->q_flags, OP_PENDING);
-       CTR6(KTR_NET, "%20s: node [%x] (%p) returning item %p as %s; "
-           "queue flags 0x%lx", __func__,
-           node->nd_ID, node, item, *rw ? "WRITER" : "READER" ,
-           ngq->q_flags);
        return (item);
 }
 
@@ -1808,9 +1756,6 @@ ng_queue_rw(node_p node, item_p  item, int rw)
        atomic_set_int(&ngq->q_flags, OP_PENDING);
        STAILQ_INSERT_TAIL(&ngq->queue, item, el_next);
 
-       CTR5(KTR_NET, "%20s: node [%x] (%p) queued item %p as %s", __func__,
-           node->nd_ID, node, item, rw ? "WRITER" : "READER" );
-           
        /*
         * We can take the worklist lock with the node locked
         * BUT NOT THE REVERSE!
@@ -1835,8 +1780,6 @@ ng_acquire_read(node_p node, item_p item)
                if (atomic_cmpset_acq_int(&node->nd_input_queue.q_flags,
                    t, t + READER_INCREMENT)) {
                        /* Successfully grabbed node */
-                       CTR4(KTR_NET, "%20s: node [%x] (%p) acquired item %p",
-                           __func__, node->nd_ID, node, item);
                        return (item);
                }
                cpu_spinwait();
@@ -1859,8 +1802,6 @@ ng_acquire_write(node_p node, item_p item)
        if (atomic_cmpset_acq_int(&node->nd_input_queue.q_flags,
            0, WRITER_ACTIVE)) {
                /* Successfully grabbed node */
-               CTR4(KTR_NET, "%20s: node [%x] (%p) acquired item %p",
-                   __func__, node->nd_ID, node, item);
                return (item);
        }
 
@@ -1870,74 +1811,6 @@ ng_acquire_write(node_p node, item_p item)
        return (NULL);
 }
 
-#if 0
-static __inline item_p
-ng_upgrade_write(node_p node, item_p item)
-{
-       struct ng_queue *ngq = &node->nd_input_queue;
-       KASSERT(node != &ng_deadnode,
-           ("%s: working on deadnode", __func__));
-
-       NGI_SET_WRITER(item);
-
-       NG_QUEUE_LOCK(ngq);
-
-       /*
-        * There will never be no readers as we are there ourselves.
-        * Set the WRITER_ACTIVE flags ASAP to block out fast track readers.
-        * The caller we are running from will call ng_leave_read()
-        * soon, so we must account for that. We must leave again with the
-        * READER lock. If we find other readers, then
-        * queue the request for later. However "later" may be rignt now
-        * if there are no readers. We don't really care if there are queued
-        * items as we will bypass them anyhow.
-        */
-       atomic_add_int(&ngq->q_flags, WRITER_ACTIVE - READER_INCREMENT);
-       if ((ngq->q_flags & (NGQ_WMASK & ~OP_PENDING)) == WRITER_ACTIVE) {
-               NG_QUEUE_UNLOCK(ngq);
-               
-               /* It's just us, act on the item. */
-               /* will NOT drop writer lock when done */
-               ng_apply_item(node, item, 0);
-
-               /*
-                * Having acted on the item, atomically 
-                * down grade back to READER and finish up
-                */
-               atomic_add_int(&ngq->q_flags,
-                   READER_INCREMENT - WRITER_ACTIVE);
-
-               /* Our caller will call ng_leave_read() */
-               return;
-       }
-       /*
-        * It's not just us active, so queue us AT THE HEAD.
-        * "Why?" I hear you ask.
-        * Put us at the head of the queue as we've already been
-        * through it once. If there is nothing else waiting,
-        * set the correct flags.
-        */
-       if (STAILQ_EMPTY(&ngq->queue)) {
-               /* We've gone from, 0 to 1 item in the queue */
-               atomic_set_int(&ngq->q_flags, OP_PENDING);
-
-               CTR3(KTR_NET, "%20s: node [%x] (%p) set OP_PENDING", __func__,
-                   node->nd_ID, node);
-       };
-       STAILQ_INSERT_HEAD(&ngq->queue, item, el_next);
-       CTR4(KTR_NET, "%20s: node [%x] (%p) requeued item %p as WRITER",
-           __func__, node->nd_ID, node, item );
-
-       /* Reverse what we did above. That downgrades us back to reader */
-       atomic_add_int(&ngq->q_flags, READER_INCREMENT - WRITER_ACTIVE);
-       if (QUEUE_ACTIVE(ngq) && NEXT_QUEUED_ITEM_CAN_PROCEED(ngq))
-               ng_worklist_add(node);
-       NG_QUEUE_UNLOCK(ngq);
-
-       return;
-}
-#endif
-
 /* Release reader lock. */
 static __inline void
 ng_leave_read(node_p node)
@@ -2193,7 +2066,6 @@ ng_apply_item(node_p node, item_p item, int rw)
                 * nothing we can do with it, drop everything.
                 */
                if (NG_NODE_NOT_VALID(node)) {
-                       TRAP_ERROR();
                        error = EINVAL;
                        NG_FREE_ITEM(item);
                        break;
@@ -2215,7 +2087,6 @@ ng_apply_item(node_p node, item_p item, int rw)
                }
                if (((!hook) || (!(rcvmsg = hook->hk_rcvmsg))) &&
                    (!(rcvmsg = node->nd_type->rcvmsg))) {
-                       TRAP_ERROR();
                        error = 0;
                        NG_FREE_ITEM(item);
                        break;
@@ -2233,7 +2104,6 @@ ng_apply_item(node_p node, item_p item, int rw)
                 */
                if ((NG_NODE_NOT_VALID(node))
                && (NGI_FN(item) != &ng_rmnode)) {
-                       TRAP_ERROR();
                        error = EINVAL;
                        NG_FREE_ITEM(item);
                        break;
@@ -2282,7 +2152,6 @@ ng_generic_msg(node_p here, item_p item, hook_p lasthook)
 
        NGI_GET_MSG(item, msg);
        if (msg->header.typecookie != NGM_GENERIC_COOKIE) {
-               TRAP_ERROR();
                error = EINVAL;
                goto out;
        }
@@ -2295,7 +2164,6 @@ ng_generic_msg(node_p here, item_p item, hook_p lasthook)
                struct ngm_mkpeer *const mkp = (struct ngm_mkpeer *) msg->data;
 
                if (msg->header.arglen != sizeof(*mkp)) {
-                       TRAP_ERROR();
                        error = EINVAL;
                        break;
                }
@@ -2312,7 +2180,6 @@ ng_generic_msg(node_p here, item_p item, hook_p lasthook)
                node_p node2;
 
                if (msg->header.arglen != sizeof(*con)) {
-                       TRAP_ERROR();
                        error = EINVAL;
                        break;
                }
@@ -2333,7 +2200,6 @@ ng_generic_msg(node_p here, item_p item, hook_p lasthook)
                struct ngm_name *const nam = (struct ngm_name *) msg->data;
 
                if (msg->header.arglen != sizeof(*nam)) {
-                       TRAP_ERROR();
                        error = EINVAL;
                        break;
                }
@@ -2347,7 +2213,6 @@ ng_generic_msg(node_p here, item_p item, hook_p lasthook)
                hook_p hook;
 
                if (msg->header.arglen != sizeof(*rmh)) {
-                       TRAP_ERROR();
                        error = EINVAL;
                        break;
                }
@@ -2535,7 +2400,6 @@ ng_generic_msg(node_p here, item_p item, hook_p lasthook)
                if (msg->header.arglen < sizeof(struct ng_mesg) ||
                    (msg->header.arglen - sizeof(struct ng_mesg) <
                    binary->header.arglen)) {
-                       TRAP_ERROR();
                        error = EINVAL;
                        break;
                }
@@ -2610,7 +2474,6 @@ ng_generic_msg(node_p here, item_p item, hook_p lasthook)
                    (ascii->header.arglen < 1) ||
                    (msg->header.arglen < sizeof(*ascii) +
                    ascii->header.arglen)) {
-                       TRAP_ERROR();
                        error = EINVAL;
                        break;
                }
@@ -2682,7 +2545,6 @@ ng_generic_msg(node_p here, item_p item, hook_p lasthook)
                }
                /* Fall through if rcvmsg not supported */
        default:
-               TRAP_ERROR();
                error = EINVAL;
        }
        /*
@@ -2950,8 +2812,6 @@ ngtask(void *context, int pending)
                }
                STAILQ_REMOVE_HEAD(&ng_worklist, nd_input_queue.q_work);
                NG_WORKLIST_UNLOCK();
-               CTR3(KTR_NET, "%20s: node [%x] (%p) taken off worklist",
-                   __func__, node->nd_ID, node);
                /*
                 * We have the node. We also take over the reference
                 * that the list had on it.
@@ -2993,6 +2853,7 @@ ng_worklist_add(node_p node)
 
        KKASSERT(mtx_owned(&node->nd_input_queue.q_mtx));
 
+       /* If not already on worklist */
        if ((node->nd_input_queue.q_flags2 & NGQ2_WORKQ) == 0) {
                static struct task ng_task;
 
@@ -3007,11 +2868,6 @@ ng_worklist_add(node_p node)
                NG_WORKLIST_UNLOCK();
                TASK_INIT(&ng_task, 0, ngtask, NULL);
                taskqueue_enqueue(taskqueue_swi, &ng_task);
-               CTR3(KTR_NET, "%20s: node [%x] (%p) put on worklist", __func__,
-                   node->nd_ID, node);
-       } else {
-               CTR3(KTR_NET, "%20s: node [%x] (%p) already on worklist",
-                   __func__, node->nd_ID, node);
        }
 }
 
@@ -3113,7 +2969,6 @@ ng_address_hook(node_p here, item_p item, hook_p hook, ng_ID_t retaddr)
            NG_HOOK_NOT_VALID(peer = NG_HOOK_PEER(hook)) ||
            NG_NODE_NOT_VALID(peernode = NG_PEER_NODE(hook))) {
                NG_FREE_ITEM(item);
-               TRAP_ERROR();
                return (ENETDOWN);
        }
 
@@ -3164,7 +3019,6 @@ ng_address_ID(node_p here, item_p item, ng_ID_t ID, ng_ID_t retaddr)
        dest = ng_ID2noderef(ID); /* GETS REFERENCE! */
        if (dest == NULL) {
                NG_FREE_ITEM(item);
-               TRAP_ERROR();
                return(EINVAL);
        }
        /* Fill out the contents */
index 08057c6..f0ca696 100644 (file)
@@ -315,19 +315,15 @@ get_tag(const struct pppoe_hdr* ph, uint16_t idx)
                 */
                ptn = (((const char *)(pt + 1)) + ntohs(pt->tag_len));
                if (ptn > end) {
-                       CTR2(KTR_NET, "%20s: invalid length for tag %d",
-                           __func__, idx);
                        return (NULL);
                }
                if (pt->tag_type == idx) {
-                       CTR2(KTR_NET, "%20s: found tag %d", __func__, idx);
                        return (pt);
                }
 
                pt = (const struct pppoe_tag*)ptn;
        }
 
-       CTR2(KTR_NET, "%20s: not found tag %d", __func__, idx);
        return (NULL);
 }
 
@@ -379,7 +375,6 @@ make_packet(sessp sp) {
 
        KASSERT((sp->neg != NULL) && (sp->neg->m != NULL),
            ("%s: called from wrong state", __func__));
-       CTR2(KTR_NET, "%20s: called %d", __func__, sp->Session_ID);
 
        dp = (char *)wh->ph.tag;
        for (count = 0, tag = sp->neg->tags;
@@ -438,8 +433,6 @@ pppoe_match_svc(node_p node, const struct pppoe_tag *tag)
                    ntohs(tag->tag_len)) == 0)
                        break;
        }
-       CTR3(KTR_NET, "%20s: matched %p for %s", __func__,
-           sp?sp->hook:NULL, tag->tag_data);
 
        return (sp?sp->hook:NULL);
 }
@@ -525,8 +518,6 @@ restart:
        if (tsp)
                goto restart;
 
-       CTR2(KTR_NET, "%20s: new sid %d", __func__, val);
-
        return (val);
 }
 
@@ -571,8 +562,6 @@ pppoe_findsession(priv_p privp, const struct pppoe_full_hdr *wh)
                }
        }
        mtx_unlock(&privp->sesshash[hash].mtx);
-       CTR3(KTR_NET, "%20s: matched %p for %d", __func__, sp?sp->hook:NULL,
-           session);
 
        return (sp);
 }
@@ -592,7 +581,6 @@ pppoe_finduniq(node_p node, const struct pppoe_tag *tag)
                if (uniq.pointer == NG_HOOK_PRIVATE(hook))
                        break;
        }
-       CTR3(KTR_NET, "%20s: matched %p for %p", __func__, hook, uniq.pointer);
 
        return (hook);
 }
@@ -629,9 +617,6 @@ ng_pppoe_constructor(node_p node)
            LIST_INIT(&privp->sesshash[i].head);
        }
 
-       CTR3(KTR_NET, "%20s: created node [%x] (%p)",
-           __func__, node->nd_ID, node);
-
        return (0);
 }
 
@@ -669,8 +654,6 @@ ng_pppoe_newhook(node_p node, hook_p hook, const char *name)
                NG_HOOK_SET_PRIVATE(hook, sp);
                sp->hook = hook;
        }
-       CTR5(KTR_NET, "%20s: node [%x] (%p) connected hook %s (%p)",
-           __func__, node->nd_ID, node, name, hook);
 
        return(0);
 }
@@ -726,9 +709,6 @@ ng_pppoe_rcvmsg(node_p node, item_p item, hook_p lasthook)
        struct ng_mesg *msg;
 
        NGI_GET_MSG(item, msg);
-       CTR5(KTR_NET, "%20s: node [%x] (%p) got message %d with cookie %d",
-           __func__, node->nd_ID, node, msg->header.cmd,
-           msg->header.typecookie);
 
        /* Deal with message according to cookie and command. */
        switch (msg->header.typecookie) {
@@ -1030,7 +1010,6 @@ ng_pppoe_rcvmsg(node_p node, item_p item, hook_p lasthook)
 
        /* Take care of synchronous response, if any. */
 quit:
-       CTR2(KTR_NET, "%20s: returning %d", __func__, error);
        NG_RESPOND_MSG(error, node, item, resp);
        /* Free the message and return. */
        NG_FREE_MSG(msg);
@@ -1058,7 +1037,6 @@ pppoe_start(sessp sp)
        /*
         * Kick the state machine into starting up.
         */
-       CTR2(KTR_NET, "%20s: called %d", __func__, sp->Session_ID);
        sp->state = PPPOE_SINIT;
        /*
         * Reset the packet header to broadcast. Since we are
@@ -1091,8 +1069,6 @@ send_acname(sessp sp, const struct pppoe_tag *tag)
        struct ng_mesg *msg;
        struct ngpppoe_sts *sts;
 
-       CTR2(KTR_NET, "%20s: called %d", __func__, sp->Session_ID);
-
        NG_MKMESSAGE(msg, NGM_PPPOE_COOKIE, NGM_PPPOE_ACNAME,
            sizeof(struct ngpppoe_sts), M_WAITOK | M_NULLOK);
        if (msg == NULL)
@@ -1113,8 +1089,6 @@ send_sessionid(sessp sp)
        int error;
        struct ng_mesg *msg;
 
-       CTR2(KTR_NET, "%20s: called %d", __func__, sp->Session_ID);
-
        NG_MKMESSAGE(msg, NGM_PPPOE_COOKIE, NGM_PPPOE_SESSIONID,
            sizeof(uint16_t), M_WAITOK | M_NULLOK);
        if (msg == NULL)
@@ -1139,9 +1113,6 @@ ng_pppoe_rcvdata(hook_p hook, item_p item)
        struct mbuf             *m;
        int                     error;
 
-       CTR6(KTR_NET, "%20s: node [%x] (%p) received %p on \"%s\" (%p)",
-           __func__, node->nd_ID, node, item, hook->hk_name, hook);
-
        NGI_GET_M(item, m);
        switch (sp->state) {
        case    PPPOE_NEWCONNECTED:
@@ -1294,9 +1265,6 @@ ng_pppoe_rcvdata_ether(hook_p hook, item_p item)
        uint8_t                 code;
        struct  mbuf            *m0;
 
-       CTR6(KTR_NET, "%20s: node [%x] (%p) received %p on \"%s\" (%p)",
-           __func__, node->nd_ID, node, item, hook->hk_name, hook);
-
        NGI_GET_M(item, m);
        /*
         * Dig out various fields from the packet.
@@ -1373,8 +1341,6 @@ ng_pppoe_rcvdata_ether(hook_p hook, item_p item)
                         */
                        tag = get_tag(ph, PTT_SRV_NAME);
                        if (tag == NULL) {
-                               CTR1(KTR_NET, "%20s: PADI w/o Service-Name",
-                                   __func__);
                                LEAVE(ENETUNREACH);
                        }
 
@@ -1675,9 +1641,6 @@ ng_pppoe_rcvdata_debug(hook_p hook, item_p item)
        const priv_p    privp = NG_NODE_PRIVATE(node);
        int             error;
 
-       CTR6(KTR_NET, "%20s: node [%x] (%p) received %p on \"%s\" (%p)",
-           __func__, node->nd_ID, node, item, hook->hk_name, hook);
-
        NG_FWD_ITEM_HOOK(error, item, privp->ethernet_hook);
        privp->packets_out++;
        return (error);
@@ -1813,8 +1776,6 @@ pppoe_ticker(node_p node, hook_p hook, void *arg1, int arg2)
        struct mbuf *m0 = NULL;
        int     error = 0;
 
-       CTR6(KTR_NET, "%20s: node [%x] (%p) hook \"%s\" (%p) session %d",
-           __func__, node->nd_ID, node, hook->hk_name, hook, sp->Session_ID);
        switch(sp->state) {
                /*
                 * Resend the last packet, using an exponential backoff.
@@ -1864,8 +1825,6 @@ scan_tags(sessp   sp, const struct pppoe_hdr* ph)
        /*
         * Keep processing tags while a tag header will still fit.
         */
-       CTR2(KTR_NET, "%20s: called %d", __func__, sp->Session_ID);
-
        while((const char*)(pt + 1) <= end) {
                /*
                 * If the tag data would go past the end of the packet, abort.
@@ -1902,8 +1861,6 @@ pppoe_send_event(sessp sp, enum cmd cmdid)
        struct ng_mesg *msg;
        struct ngpppoe_sts *sts;
 
-       CTR2(KTR_NET, "%20s: called %d", __func__, sp->Session_ID);
-
        NG_MKMESSAGE(msg, NGM_PPPOE_COOKIE, cmdid,
                        sizeof(struct ngpppoe_sts), M_WAITOK | M_NULLOK);
        if (msg == NULL)