Module Name: src
Committed By: riastradh
Date: Tue Jun 29 21:19:58 UTC 2021
Modified Files:
src/sys/net: if.c if.h if_stats.c if_stats.h
Log Message:
Make if_stats_init, if_attach, if_initialize return void.
percpu_alloc can't fail.
Author: Maya Rashish <[email protected]>
Committer: Taylor R Campbell <[email protected]>
To generate a diff of this commit:
cvs rdiff -u -r1.485 -r1.486 src/sys/net/if.c
cvs rdiff -u -r1.290 -r1.291 src/sys/net/if.h
cvs rdiff -u -r1.3 -r1.4 src/sys/net/if_stats.c
cvs rdiff -u -r1.2 -r1.3 src/sys/net/if_stats.h
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/sys/net/if.c
diff -u src/sys/net/if.c:1.485 src/sys/net/if.c:1.486
--- src/sys/net/if.c:1.485 Mon May 17 04:07:43 2021
+++ src/sys/net/if.c Tue Jun 29 21:19:58 2021
@@ -1,4 +1,4 @@
-/* $NetBSD: if.c,v 1.485 2021/05/17 04:07:43 yamaguchi Exp $ */
+/* $NetBSD: if.c,v 1.486 2021/06/29 21:19:58 riastradh Exp $ */
/*-
* Copyright (c) 1999, 2000, 2001, 2008 The NetBSD Foundation, Inc.
@@ -90,7 +90,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if.c,v 1.485 2021/05/17 04:07:43 yamaguchi Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if.c,v 1.486 2021/06/29 21:19:58 riastradh Exp $");
#if defined(_KERNEL_OPT)
#include "opt_inet.h"
@@ -702,10 +702,9 @@ skip:
* ether_ifattach(ifp, enaddr);
* if_register(ifp);
*/
-int
+void
if_initialize(ifnet_t *ifp)
{
- int rv = 0;
KASSERT(if_indexlim > 0);
TAILQ_INIT(&ifp->if_addrlist);
@@ -748,25 +747,11 @@ if_initialize(ifnet_t *ifp)
psref_target_init(&ifp->if_psref, ifnet_psref_class);
ifp->if_ioctl_lock = mutex_obj_alloc(MUTEX_DEFAULT, IPL_NONE);
LIST_INIT(&ifp->if_multiaddrs);
- if ((rv = if_stats_init(ifp)) != 0) {
- goto fail;
- }
+ if_stats_init(ifp);
IFNET_GLOBAL_LOCK();
if_getindex(ifp);
IFNET_GLOBAL_UNLOCK();
-
- return 0;
-
-fail:
- IF_AFDATA_LOCK_DESTROY(ifp);
-
- pfil_run_ifhooks(if_pfil, PFIL_IFNET_DETACH, ifp);
- (void)pfil_head_destroy(ifp->if_pfil);
-
- IFQ_LOCK_DESTROY(&ifp->if_snd);
-
- return rv;
}
/*
@@ -1142,19 +1127,13 @@ if_input(struct ifnet *ifp, struct mbuf
* migrate softint-based if_input without much changes. If you don't
* want to enable it, use if_initialize instead.
*/
-int
+void
if_attach(ifnet_t *ifp)
{
- int rv;
-
- rv = if_initialize(ifp);
- if (rv != 0)
- return rv;
+ if_initialize(ifp);
ifp->if_percpuq = if_percpuq_create(ifp);
if_register(ifp);
-
- return 0;
}
void
Index: src/sys/net/if.h
diff -u src/sys/net/if.h:1.290 src/sys/net/if.h:1.291
--- src/sys/net/if.h:1.290 Mon May 17 04:07:43 2021
+++ src/sys/net/if.h Tue Jun 29 21:19:58 2021
@@ -1,4 +1,4 @@
-/* $NetBSD: if.h,v 1.290 2021/05/17 04:07:43 yamaguchi Exp $ */
+/* $NetBSD: if.h,v 1.291 2021/06/29 21:19:58 riastradh Exp $ */
/*-
* Copyright (c) 1999, 2000, 2001 The NetBSD Foundation, Inc.
@@ -1115,9 +1115,9 @@ void if_activate_sadl(struct ifnet *, st
void if_set_sadl(struct ifnet *, const void *, u_char, bool);
void if_alloc_sadl(struct ifnet *);
void if_free_sadl(struct ifnet *, int);
-int if_initialize(struct ifnet *);
+void if_initialize(struct ifnet *);
void if_register(struct ifnet *);
-int if_attach(struct ifnet *); /* Deprecated. Use if_initialize and if_register */
+void if_attach(struct ifnet *); /* Deprecated. Use if_initialize and if_register */
void if_attachdomain(void);
void if_deactivate(struct ifnet *);
bool if_is_deactivated(const struct ifnet *);
Index: src/sys/net/if_stats.c
diff -u src/sys/net/if_stats.c:1.3 src/sys/net/if_stats.c:1.4
--- src/sys/net/if_stats.c:1.3 Fri Feb 14 22:04:12 2020
+++ src/sys/net/if_stats.c Tue Jun 29 21:19:58 2021
@@ -1,4 +1,4 @@
-/* $NetBSD: if_stats.c,v 1.3 2020/02/14 22:04:12 thorpej Exp $ */
+/* $NetBSD: if_stats.c,v 1.4 2021/06/29 21:19:58 riastradh Exp $ */
/*-
* Copyright (c) 2020 The NetBSD Foundation, Inc.
@@ -30,7 +30,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: if_stats.c,v 1.3 2020/02/14 22:04:12 thorpej Exp $");
+__KERNEL_RCSID(0, "$NetBSD: if_stats.c,v 1.4 2021/06/29 21:19:58 riastradh Exp $");
#include <sys/param.h>
#include <sys/mbuf.h>
@@ -45,13 +45,10 @@ __KERNEL_RCSID(0, "$NetBSD: if_stats.c,v
* if_stats_init --
* Initialize statistics storage for a network interface.
*/
-int
+void
if_stats_init(ifnet_t * const ifp)
{
ifp->if_stats = percpu_alloc(IF_STATS_SIZE);
- if (ifp->if_stats == NULL)
- return ENOMEM;
- return 0;
}
/*
Index: src/sys/net/if_stats.h
diff -u src/sys/net/if_stats.h:1.2 src/sys/net/if_stats.h:1.3
--- src/sys/net/if_stats.h:1.2 Fri Feb 14 22:04:12 2020
+++ src/sys/net/if_stats.h Tue Jun 29 21:19:58 2021
@@ -1,4 +1,4 @@
-/* $NetBSD: if_stats.h,v 1.2 2020/02/14 22:04:12 thorpej Exp $ */
+/* $NetBSD: if_stats.h,v 1.3 2021/06/29 21:19:58 riastradh Exp $ */
/*-
* Copyright (c) 2020 The NetBSD Foundation, Inc.
@@ -115,7 +115,7 @@ if_statsub_ref(net_stat_ref_t nsr, if_st
_NET_STATSUB_REF(nsr, x, v);
}
-int if_stats_init(ifnet_t *);
+void if_stats_init(ifnet_t *);
void if_stats_fini(ifnet_t *);
void if_stats_to_if_data(ifnet_t *, struct if_data *, bool);
