From 6c78a88c3bb5f7c87b56f79915289532c9997f86 Mon Sep 17 00:00:00 2001 From: glebius Date: Mon, 18 Apr 2011 09:12:27 +0000 Subject: Node constructor methods are supposed to be called in syscall context always. Convert nodes to consistently use M_WAITOK flag for memory allocation. Reviewed by: julian --- sys/netgraph/atm/ccatm/ng_ccatm.c | 4 +--- sys/netgraph/atm/sscfu/ng_sscfu.c | 3 +-- sys/netgraph/atm/sscop/ng_sscop.c | 3 +-- sys/netgraph/atm/uni/ng_uni.c | 3 +-- sys/netgraph/bluetooth/hci/ng_hci_main.c | 5 +---- sys/netgraph/bluetooth/l2cap/ng_l2cap_main.c | 5 +---- sys/netgraph/netflow/ng_netflow.c | 5 +---- sys/netgraph/ng_UI.c | 5 +---- sys/netgraph/ng_async.c | 16 +++------------- sys/netgraph/ng_atmllc.c | 7 +------ sys/netgraph/ng_bridge.c | 10 ++-------- sys/netgraph/ng_car.c | 4 +--- sys/netgraph/ng_cisco.c | 4 +--- sys/netgraph/ng_device.c | 4 +--- sys/netgraph/ng_eiface.c | 4 +--- sys/netgraph/ng_etf.c | 5 +---- sys/netgraph/ng_fec.c | 4 +--- sys/netgraph/ng_frame_relay.c | 4 +--- sys/netgraph/ng_gif_demux.c | 5 +---- sys/netgraph/ng_hub.c | 4 +--- sys/netgraph/ng_iface.c | 4 +--- sys/netgraph/ng_ksocket.c | 5 +---- sys/netgraph/ng_l2tp.c | 4 +--- sys/netgraph/ng_lmi.c | 4 +--- sys/netgraph/ng_mppc.c | 4 +--- sys/netgraph/ng_nat.c | 5 +---- sys/netgraph/ng_one2many.c | 4 +--- sys/netgraph/ng_pipe.c | 4 +--- sys/netgraph/ng_ppp.c | 4 +--- sys/netgraph/ng_pppoe.c | 4 +--- sys/netgraph/ng_pptpgre.c | 4 +--- sys/netgraph/ng_rfc1490.c | 4 +--- sys/netgraph/ng_sample.c | 5 +---- sys/netgraph/ng_source.c | 4 +--- sys/netgraph/ng_split.c | 4 +--- sys/netgraph/ng_sppp.c | 4 +--- sys/netgraph/ng_tee.c | 4 +--- sys/netgraph/ng_tty.c | 4 +--- sys/netgraph/ng_vjc.c | 4 +--- sys/netgraph/ng_vlan.c | 4 +--- 40 files changed, 43 insertions(+), 144 deletions(-) (limited to 'sys') diff --git a/sys/netgraph/atm/ccatm/ng_ccatm.c b/sys/netgraph/atm/ccatm/ng_ccatm.c index 7030369..92e162c 100644 --- a/sys/netgraph/atm/ccatm/ng_ccatm.c +++ b/sys/netgraph/atm/ccatm/ng_ccatm.c @@ -348,9 +348,7 @@ ng_ccatm_constructor(node_p node) { struct ccnode *priv; - priv = malloc(sizeof(*priv), M_NG_CCATM, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NG_CCATM, M_WAITOK | M_ZERO); priv->node = node; priv->data = cc_create(&cc_funcs); diff --git a/sys/netgraph/atm/sscfu/ng_sscfu.c b/sys/netgraph/atm/sscfu/ng_sscfu.c index 47413b5..cca8198 100644 --- a/sys/netgraph/atm/sscfu/ng_sscfu.c +++ b/sys/netgraph/atm/sscfu/ng_sscfu.c @@ -538,8 +538,7 @@ ng_sscfu_constructor(node_p node) { struct priv *priv; - if ((priv = malloc(sizeof(*priv), M_NG_SSCFU, M_NOWAIT|M_ZERO)) == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NG_SSCFU, M_WAITOK | M_ZERO); if ((priv->sscf = sscfu_create(node, &sscfu_funcs)) == NULL) { free(priv, M_NG_SSCFU); diff --git a/sys/netgraph/atm/sscop/ng_sscop.c b/sys/netgraph/atm/sscop/ng_sscop.c index cf52cc9..729c818 100644 --- a/sys/netgraph/atm/sscop/ng_sscop.c +++ b/sys/netgraph/atm/sscop/ng_sscop.c @@ -246,8 +246,7 @@ ng_sscop_constructor(node_p node) { struct priv *p; - if ((p = malloc(sizeof(*p), M_NG_SSCOP, M_NOWAIT | M_ZERO)) == NULL) - return (ENOMEM); + p = malloc(sizeof(*p), M_NG_SSCOP, M_WAITOK | M_ZERO); if ((p->sscop = sscop_create(node, &sscop_funcs)) == NULL) { free(p, M_NG_SSCOP); diff --git a/sys/netgraph/atm/uni/ng_uni.c b/sys/netgraph/atm/uni/ng_uni.c index 5b2a7ff..2fc9599 100644 --- a/sys/netgraph/atm/uni/ng_uni.c +++ b/sys/netgraph/atm/uni/ng_uni.c @@ -223,8 +223,7 @@ ng_uni_constructor(node_p node) { struct priv *priv; - if ((priv = malloc(sizeof(*priv), M_NG_UNI, M_NOWAIT | M_ZERO)) == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NG_UNI, M_WAITOK | M_ZERO); if ((priv->uni = uni_create(node, &uni_funcs)) == NULL) { free(priv, M_NG_UNI); diff --git a/sys/netgraph/bluetooth/hci/ng_hci_main.c b/sys/netgraph/bluetooth/hci/ng_hci_main.c index eeea79f..c5b3040 100644 --- a/sys/netgraph/bluetooth/hci/ng_hci_main.c +++ b/sys/netgraph/bluetooth/hci/ng_hci_main.c @@ -109,10 +109,7 @@ ng_hci_constructor(node_p node) { ng_hci_unit_p unit = NULL; - unit = malloc(sizeof(*unit), M_NETGRAPH_HCI, - M_NOWAIT | M_ZERO); - if (unit == NULL) - return (ENOMEM); + unit = malloc(sizeof(*unit), M_NETGRAPH_HCI, M_WAITOK | M_ZERO); unit->node = node; unit->debug = NG_HCI_WARN_LEVEL; diff --git a/sys/netgraph/bluetooth/l2cap/ng_l2cap_main.c b/sys/netgraph/bluetooth/l2cap/ng_l2cap_main.c index 23c6b01..5ed61aa 100644 --- a/sys/netgraph/bluetooth/l2cap/ng_l2cap_main.c +++ b/sys/netgraph/bluetooth/l2cap/ng_l2cap_main.c @@ -113,10 +113,7 @@ ng_l2cap_constructor(node_p node) ng_l2cap_p l2cap = NULL; /* Create new L2CAP node */ - l2cap = malloc(sizeof(*l2cap), - M_NETGRAPH_L2CAP, M_NOWAIT|M_ZERO); - if (l2cap == NULL) - return (ENOMEM); + l2cap = malloc(sizeof(*l2cap), M_NETGRAPH_L2CAP, M_WAITOK | M_ZERO); l2cap->node = node; l2cap->debug = NG_L2CAP_WARN_LEVEL; diff --git a/sys/netgraph/netflow/ng_netflow.c b/sys/netgraph/netflow/ng_netflow.c index 53d0d6f..74036fa 100644 --- a/sys/netgraph/netflow/ng_netflow.c +++ b/sys/netgraph/netflow/ng_netflow.c @@ -223,10 +223,7 @@ ng_netflow_constructor(node_p node) int error = 0, i; /* Initialize private data */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT); - if (priv == NULL) - return (ENOMEM); - bzero(priv, sizeof(*priv)); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); /* Make node and its data point at each other */ NG_NODE_SET_PRIVATE(node, priv); diff --git a/sys/netgraph/ng_UI.c b/sys/netgraph/ng_UI.c index 2c3f132..b06c17a 100644 --- a/sys/netgraph/ng_UI.c +++ b/sys/netgraph/ng_UI.c @@ -103,10 +103,7 @@ ng_UI_constructor(node_p node) priv_p priv; /* Allocate private structure */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) { - return (ENOMEM); - } + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); return (0); } diff --git a/sys/netgraph/ng_async.c b/sys/netgraph/ng_async.c index acbd76a..283ba15 100644 --- a/sys/netgraph/ng_async.c +++ b/sys/netgraph/ng_async.c @@ -183,25 +183,15 @@ nga_constructor(node_p node) { sc_p sc; - sc = malloc(sizeof(*sc), M_NETGRAPH_ASYNC, M_NOWAIT | M_ZERO); - if (sc == NULL) - return (ENOMEM); + sc = malloc(sizeof(*sc), M_NETGRAPH_ASYNC, M_WAITOK | M_ZERO); sc->amode = MODE_HUNT; sc->cfg.accm = ~0; sc->cfg.amru = NG_ASYNC_DEFAULT_MRU; sc->cfg.smru = NG_ASYNC_DEFAULT_MRU; sc->abuf = malloc(ASYNC_BUF_SIZE(sc->cfg.smru), - M_NETGRAPH_ASYNC, M_NOWAIT); - if (sc->abuf == NULL) - goto fail; + M_NETGRAPH_ASYNC, M_WAITOK); sc->sbuf = malloc(SYNC_BUF_SIZE(sc->cfg.amru), - M_NETGRAPH_ASYNC, M_NOWAIT); - if (sc->sbuf == NULL) { - free(sc->abuf, M_NETGRAPH_ASYNC); -fail: - free(sc, M_NETGRAPH_ASYNC); - return (ENOMEM); - } + M_NETGRAPH_ASYNC, M_WAITOK); NG_NODE_SET_PRIVATE(node, sc); sc->node = node; return (0); diff --git a/sys/netgraph/ng_atmllc.c b/sys/netgraph/ng_atmllc.c index 6498056..336d4b1 100644 --- a/sys/netgraph/ng_atmllc.c +++ b/sys/netgraph/ng_atmllc.c @@ -80,12 +80,7 @@ ng_atmllc_constructor(node_p node) { struct ng_atmllc_priv *priv; - priv = malloc(sizeof(*priv), M_NETGRAPH, - M_NOWAIT | M_ZERO); - if (priv == NULL) { - return (ENOMEM); - } - + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); return (0); diff --git a/sys/netgraph/ng_bridge.c b/sys/netgraph/ng_bridge.c index 0bf0866..e9631b8 100644 --- a/sys/netgraph/ng_bridge.c +++ b/sys/netgraph/ng_bridge.c @@ -309,18 +309,12 @@ ng_bridge_constructor(node_p node) priv_p priv; /* Allocate and initialize private info */ - priv = malloc(sizeof(*priv), M_NETGRAPH_BRIDGE, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_BRIDGE, M_WAITOK | M_ZERO); ng_callout_init(&priv->timer); /* Allocate and initialize hash table, etc. */ priv->tab = malloc(MIN_BUCKETS * sizeof(*priv->tab), - M_NETGRAPH_BRIDGE, M_NOWAIT | M_ZERO); - if (priv->tab == NULL) { - free(priv, M_NETGRAPH_BRIDGE); - return (ENOMEM); - } + M_NETGRAPH_BRIDGE, M_WAITOK | M_ZERO); priv->numBuckets = MIN_BUCKETS; priv->hashMask = MIN_BUCKETS - 1; priv->conf.debugLevel = 1; diff --git a/sys/netgraph/ng_car.c b/sys/netgraph/ng_car.c index 0a4b8a3..84e8808 100644 --- a/sys/netgraph/ng_car.c +++ b/sys/netgraph/ng_car.c @@ -186,9 +186,7 @@ ng_car_constructor(node_p node) priv_p priv; /* Initialize private descriptor. */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); priv->node = node; diff --git a/sys/netgraph/ng_cisco.c b/sys/netgraph/ng_cisco.c index 785b055..76dc1ec 100644 --- a/sys/netgraph/ng_cisco.c +++ b/sys/netgraph/ng_cisco.c @@ -193,9 +193,7 @@ cisco_constructor(node_p node) { sc_p sc; - sc = malloc(sizeof(*sc), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (sc == NULL) - return (ENOMEM); + sc = malloc(sizeof(*sc), M_NETGRAPH, M_WAITOK | M_ZERO); ng_callout_init(&sc->handle); NG_NODE_SET_PRIVATE(node, sc); diff --git a/sys/netgraph/ng_device.c b/sys/netgraph/ng_device.c index eb094be..b8c339c 100644 --- a/sys/netgraph/ng_device.c +++ b/sys/netgraph/ng_device.c @@ -163,9 +163,7 @@ ng_device_constructor(node_p node) DBG; - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); /* Allocate unit number */ priv->unit = alloc_unr(ngd_unit); diff --git a/sys/netgraph/ng_eiface.c b/sys/netgraph/ng_eiface.c index 58d3e5d..cbae38f 100644 --- a/sys/netgraph/ng_eiface.c +++ b/sys/netgraph/ng_eiface.c @@ -343,9 +343,7 @@ ng_eiface_constructor(node_p node) u_char eaddr[6] = {0,0,0,0,0,0}; /* Allocate node and interface private structures */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); ifp = priv->ifp = if_alloc(IFT_ETHER); if (ifp == NULL) { diff --git a/sys/netgraph/ng_etf.c b/sys/netgraph/ng_etf.c index 7e607f7..2167b5d 100644 --- a/sys/netgraph/ng_etf.c +++ b/sys/netgraph/ng_etf.c @@ -180,10 +180,7 @@ ng_etf_constructor(node_p node) int i; /* Initialize private descriptor */ - privdata = malloc(sizeof(*privdata), M_NETGRAPH_ETF, - M_NOWAIT | M_ZERO); - if (privdata == NULL) - return (ENOMEM); + privdata = malloc(sizeof(*privdata), M_NETGRAPH_ETF, M_WAITOK | M_ZERO); for (i = 0; i < HASHSIZE; i++) { LIST_INIT((privdata->hashtable + i)); } diff --git a/sys/netgraph/ng_fec.c b/sys/netgraph/ng_fec.c index ce5a2d5..eff70ea 100644 --- a/sys/netgraph/ng_fec.c +++ b/sys/netgraph/ng_fec.c @@ -1197,9 +1197,7 @@ ng_fec_constructor(node_p node) int error = 0; /* Allocate node and interface private structures */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); ifp = priv->ifp = if_alloc(IFT_ETHER); if (ifp == NULL) { diff --git a/sys/netgraph/ng_frame_relay.c b/sys/netgraph/ng_frame_relay.c index 670ef17..05c311d 100644 --- a/sys/netgraph/ng_frame_relay.c +++ b/sys/netgraph/ng_frame_relay.c @@ -212,9 +212,7 @@ ngfrm_constructor(node_p node) { sc_p sc; - sc = malloc(sizeof(*sc), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (!sc) - return (ENOMEM); + sc = malloc(sizeof(*sc), M_NETGRAPH, M_WAITOK | M_ZERO); sc->addrlen = 2; /* default */ /* Link the node and our private info */ diff --git a/sys/netgraph/ng_gif_demux.c b/sys/netgraph/ng_gif_demux.c index 3e88219..857f10f 100644 --- a/sys/netgraph/ng_gif_demux.c +++ b/sys/netgraph/ng_gif_demux.c @@ -233,10 +233,7 @@ ng_gif_demux_constructor(node_p node) priv_p priv; /* Allocate and initialize private info */ - priv = malloc(sizeof(*priv), M_NETGRAPH_GIF_DEMUX, - M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_GIF_DEMUX, M_WAITOK | M_ZERO); priv->node = node; NG_NODE_SET_PRIVATE(node, priv); diff --git a/sys/netgraph/ng_hub.c b/sys/netgraph/ng_hub.c index 9cc91e3..f013aac 100644 --- a/sys/netgraph/ng_hub.c +++ b/sys/netgraph/ng_hub.c @@ -86,9 +86,7 @@ ng_hub_constructor(node_p node) priv_p priv; /* Allocate and initialize private info */ - priv = malloc(sizeof(*priv), M_NETGRAPH_HUB, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_HUB, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); return (0); diff --git a/sys/netgraph/ng_iface.c b/sys/netgraph/ng_iface.c index 6e3c39d..a08595f 100644 --- a/sys/netgraph/ng_iface.c +++ b/sys/netgraph/ng_iface.c @@ -531,9 +531,7 @@ ng_iface_constructor(node_p node) priv_p priv; /* Allocate node and interface private structures */ - priv = malloc(sizeof(*priv), M_NETGRAPH_IFACE, M_NOWAIT|M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_IFACE, M_WAITOK | M_ZERO); ifp = if_alloc(IFT_PROPVIRTUAL); if (ifp == NULL) { free(priv, M_NETGRAPH_IFACE); diff --git a/sys/netgraph/ng_ksocket.c b/sys/netgraph/ng_ksocket.c index f6cbccc..6caade0 100644 --- a/sys/netgraph/ng_ksocket.c +++ b/sys/netgraph/ng_ksocket.c @@ -523,10 +523,7 @@ ng_ksocket_constructor(node_p node) priv_p priv; /* Allocate private structure */ - priv = malloc(sizeof(*priv), - M_NETGRAPH_KSOCKET, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_KSOCKET, M_WAITOK | M_ZERO); LIST_INIT(&priv->embryos); /* cross link them */ diff --git a/sys/netgraph/ng_l2tp.c b/sys/netgraph/ng_l2tp.c index ce15bc1..201fc45 100644 --- a/sys/netgraph/ng_l2tp.c +++ b/sys/netgraph/ng_l2tp.c @@ -361,9 +361,7 @@ ng_l2tp_constructor(node_p node) int i; /* Allocate private structure */ - priv = malloc(sizeof(*priv), M_NETGRAPH_L2TP, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_L2TP, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); priv->node = node; diff --git a/sys/netgraph/ng_lmi.c b/sys/netgraph/ng_lmi.c index 9de32ce..2624e06 100644 --- a/sys/netgraph/ng_lmi.c +++ b/sys/netgraph/ng_lmi.c @@ -185,9 +185,7 @@ nglmi_constructor(node_p node) { sc_p sc; - sc = malloc(sizeof(*sc), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (sc == NULL) - return (ENOMEM); + sc = malloc(sizeof(*sc), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, sc); sc->node = node; diff --git a/sys/netgraph/ng_mppc.c b/sys/netgraph/ng_mppc.c index 074c018..75194e8 100644 --- a/sys/netgraph/ng_mppc.c +++ b/sys/netgraph/ng_mppc.c @@ -201,9 +201,7 @@ ng_mppc_constructor(node_p node) priv_p priv; /* Allocate private structure */ - priv = malloc(sizeof(*priv), M_NETGRAPH_MPPC, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_MPPC, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); diff --git a/sys/netgraph/ng_nat.c b/sys/netgraph/ng_nat.c index cc69b14..6481699 100644 --- a/sys/netgraph/ng_nat.c +++ b/sys/netgraph/ng_nat.c @@ -272,10 +272,7 @@ ng_nat_constructor(node_p node) priv_p priv; /* Initialize private descriptor. */ - priv = malloc(sizeof(*priv), M_NETGRAPH, - M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); /* Init aliasing engine. */ priv->lib = LibAliasInit(NULL); diff --git a/sys/netgraph/ng_one2many.c b/sys/netgraph/ng_one2many.c index 714528d..5984279 100644 --- a/sys/netgraph/ng_one2many.c +++ b/sys/netgraph/ng_one2many.c @@ -187,9 +187,7 @@ ng_one2many_constructor(node_p node) priv_p priv; /* Allocate and initialize private info */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); priv->conf.xmitAlg = NG_ONE2MANY_XMIT_ROUNDROBIN; priv->conf.failAlg = NG_ONE2MANY_FAIL_MANUAL; diff --git a/sys/netgraph/ng_pipe.c b/sys/netgraph/ng_pipe.c index 3efd6d9..b5bab3c 100644 --- a/sys/netgraph/ng_pipe.c +++ b/sys/netgraph/ng_pipe.c @@ -253,9 +253,7 @@ ngp_constructor(node_p node) { priv_p priv; - priv = malloc(sizeof(*priv), M_NG_PIPE, M_ZERO | M_NOWAIT); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NG_PIPE, M_ZERO | M_WAITOK); NG_NODE_SET_PRIVATE(node, priv); /* Mark node as single-threaded */ diff --git a/sys/netgraph/ng_ppp.c b/sys/netgraph/ng_ppp.c index e5897f3..a0a3c38 100644 --- a/sys/netgraph/ng_ppp.c +++ b/sys/netgraph/ng_ppp.c @@ -490,9 +490,7 @@ ng_ppp_constructor(node_p node) int i; /* Allocate private structure */ - priv = malloc(sizeof(*priv), M_NETGRAPH_PPP, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_PPP, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); diff --git a/sys/netgraph/ng_pppoe.c b/sys/netgraph/ng_pppoe.c index cc59b2b..4d5e24e 100644 --- a/sys/netgraph/ng_pppoe.c +++ b/sys/netgraph/ng_pppoe.c @@ -611,9 +611,7 @@ ng_pppoe_constructor(node_p node) int i; /* Initialize private descriptor. */ - privp = malloc(sizeof(*privp), M_NETGRAPH_PPPOE, M_NOWAIT | M_ZERO); - if (privp == NULL) - return (ENOMEM); + privp = malloc(sizeof(*privp), M_NETGRAPH_PPPOE, M_WAITOK | M_ZERO); /* Link structs together; this counts as our one reference to *node. */ NG_NODE_SET_PRIVATE(node, privp); diff --git a/sys/netgraph/ng_pptpgre.c b/sys/netgraph/ng_pptpgre.c index 606d278..a640b6e 100644 --- a/sys/netgraph/ng_pptpgre.c +++ b/sys/netgraph/ng_pptpgre.c @@ -281,9 +281,7 @@ ng_pptpgre_constructor(node_p node) int i; /* Allocate private structure */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); diff --git a/sys/netgraph/ng_rfc1490.c b/sys/netgraph/ng_rfc1490.c index fff7c28..2dc5123 100644 --- a/sys/netgraph/ng_rfc1490.c +++ b/sys/netgraph/ng_rfc1490.c @@ -165,9 +165,7 @@ ng_rfc1490_constructor(node_p node) priv_p priv; /* Allocate private structure */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); /* Initialize to default encapsulation method - ietf-ip */ priv->enc = ng_rfc1490_encaps; diff --git a/sys/netgraph/ng_sample.c b/sys/netgraph/ng_sample.c index 38b86d3..3d644db 100644 --- a/sys/netgraph/ng_sample.c +++ b/sys/netgraph/ng_sample.c @@ -154,10 +154,7 @@ ng_xxx_constructor(node_p node) int i; /* Initialize private descriptor */ - privdata = malloc(sizeof(*privdata), M_NETGRAPH, - M_NOWAIT | M_ZERO); - if (privdata == NULL) - return (ENOMEM); + privdata = malloc(sizeof(*privdata), M_NETGRAPH, M_WAITOK | M_ZERO); for (i = 0; i < XXX_NUM_DLCIS; i++) { privdata->channel[i].dlci = -2; privdata->channel[i].channel = i; diff --git a/sys/netgraph/ng_source.c b/sys/netgraph/ng_source.c index e1828ac..9309559 100644 --- a/sys/netgraph/ng_source.c +++ b/sys/netgraph/ng_source.c @@ -273,9 +273,7 @@ ng_source_constructor(node_p node) { sc_p sc; - sc = malloc(sizeof(*sc), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (sc == NULL) - return (ENOMEM); + sc = malloc(sizeof(*sc), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, sc); sc->node = node; diff --git a/sys/netgraph/ng_split.c b/sys/netgraph/ng_split.c index b4f2ee0..87a2cbc 100644 --- a/sys/netgraph/ng_split.c +++ b/sys/netgraph/ng_split.c @@ -85,9 +85,7 @@ ng_split_constructor(node_p node) priv_p priv; /* Allocate node */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_ZERO | M_NOWAIT); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_ZERO | M_WAITOK); /* Link together node and private info */ NG_NODE_SET_PRIVATE(node, priv); diff --git a/sys/netgraph/ng_sppp.c b/sys/netgraph/ng_sppp.c index 6b30ed6..216ecbe 100644 --- a/sys/netgraph/ng_sppp.c +++ b/sys/netgraph/ng_sppp.c @@ -248,9 +248,7 @@ ng_sppp_constructor (node_p node) int error = 0; /* Allocate node and interface private structures */ - priv = malloc (sizeof(*priv), M_NETGRAPH_SPPP, M_NOWAIT|M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH_SPPP, M_WAITOK | M_ZERO); ifp = if_alloc(IFT_PPP); if (ifp == NULL) { diff --git a/sys/netgraph/ng_tee.c b/sys/netgraph/ng_tee.c index 6ea1a4b..4d85666 100644 --- a/sys/netgraph/ng_tee.c +++ b/sys/netgraph/ng_tee.c @@ -153,9 +153,7 @@ ng_tee_constructor(node_p node) { sc_p privdata; - privdata = malloc(sizeof(*privdata), M_NETGRAPH, M_NOWAIT|M_ZERO); - if (privdata == NULL) - return (ENOMEM); + privdata = malloc(sizeof(*privdata), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, privdata); return (0); diff --git a/sys/netgraph/ng_tty.c b/sys/netgraph/ng_tty.c index 492d21b..34105b2 100644 --- a/sys/netgraph/ng_tty.c +++ b/sys/netgraph/ng_tty.c @@ -156,9 +156,7 @@ ngt_constructor(node_p node) sc_p sc; /* Allocate private structure */ - sc = malloc(sizeof(*sc), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (sc == NULL) - return (ENOMEM); + sc = malloc(sizeof(*sc), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, sc); sc->node = node; diff --git a/sys/netgraph/ng_vjc.c b/sys/netgraph/ng_vjc.c index 797b995..97801f2 100644 --- a/sys/netgraph/ng_vjc.c +++ b/sys/netgraph/ng_vjc.c @@ -243,9 +243,7 @@ ng_vjc_constructor(node_p node) priv_p priv; /* Allocate private structure */ - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); NG_NODE_SET_PRIVATE(node, priv); diff --git a/sys/netgraph/ng_vlan.c b/sys/netgraph/ng_vlan.c index bd29929..f9de94f 100644 --- a/sys/netgraph/ng_vlan.c +++ b/sys/netgraph/ng_vlan.c @@ -161,9 +161,7 @@ ng_vlan_constructor(node_p node) priv_p priv; int i; - priv = malloc(sizeof(*priv), M_NETGRAPH, M_NOWAIT | M_ZERO); - if (priv == NULL) - return (ENOMEM); + priv = malloc(sizeof(*priv), M_NETGRAPH, M_WAITOK | M_ZERO); for (i = 0; i < HASHSIZE; i++) LIST_INIT(&priv->hashtable[i]); NG_NODE_SET_PRIVATE(node, priv); -- cgit v1.1