diff options
| author |   dlg <dlg@openbsd.org> | 2020-04-12 07:02:43 +0000 |
|---|---|---|
| committer | dlg <dlg@openbsd.org> | 2020-04-12 07:02:43 +0000 |
| commit | 351e2860c25c8ccf6243c57011e94a4cc87f241d (patch) | |
| tree | 9257e22f14547a561d464d6d5cfebb24b10895c2 | |
| parent | take NET_LOCK in aggr_clone_destroy() before calling aggr_p_dtor() (diff) | |
| download | wireguard-openbsd-351e2860c25c8ccf6243c57011e94a4cc87f241d.tar.xz wireguard-openbsd-351e2860c25c8ccf6243c57011e94a4cc87f241d.zip | |
say if_pcount needs NET_LOCK instead of the kernel lock.
if_pcount is only touched in ifpromisc(), and ifpromisc() needs
NET_LOCK anyway because it also modifies if_flags.
suggested by mpi@
ok visa@
| -rw-r--r-- | sys/net/if_var.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sys/net/if_var.h b/sys/net/if_var.h index 7f4eeb2192a..6225dbd9a9e 100644 --- a/sys/net/if_var.h +++ b/sys/net/if_var.h @@ -1,4 +1,4 @@ -/* $OpenBSD: if_var.h,v 1.103 2019/11/08 07:16:29 dlg Exp $ */ +/* $OpenBSD: if_var.h,v 1.104 2020/04/12 07:02:43 dlg Exp $ */ /* $NetBSD: if.h,v 1.23 1996/05/07 02:40:27 thorpej Exp $ */ /* @@ -130,7 +130,7 @@ struct ifnet { /* and the entries */ /* [I] check or clean routes (+ or -)'d */ void (*if_rtrequest)(struct ifnet *, int, struct rtentry *); char if_xname[IFNAMSIZ]; /* [I] external name (name + unit) */ - int if_pcount; /* [k] # of promiscuous listeners */ + int if_pcount; /* [N] # of promiscuous listeners */ unsigned int if_bridgeidx; /* [k] used by bridge ports */ caddr_t if_bpf; /* packet filter structure */ caddr_t if_switchport; /* used by switch ports */ |