diff options
| author | Conrad Meyer <cem@FreeBSD.org> | 2019-10-17 16:23:03 +0000 |
|---|---|---|
| committer | Conrad Meyer <cem@FreeBSD.org> | 2019-10-17 16:23:03 +0000 |
| commit | 7790c8c1996ad89a22b8bd194a230cf23ee67f4b (patch) | |
| tree | 78e6de7914d8cdd8021ca4912f6223b1c30c0d32 /sys/conf/files | |
| parent | 756368b68b98ad0d95e6dc1754ab09b55e4d567f (diff) | |
| download | src-7790c8c1996ad89a22b8bd194a230cf23ee67f4b.tar.gz src-7790c8c1996ad89a22b8bd194a230cf23ee67f4b.zip | |
Split out a more generic debugnet(4) from netdump(4)
Debugnet is a simplistic and specialized panic- or debug-time reliable
datagram transport. It can drive a single connection at a time and is
currently unidirectional (debug/panic machine transmit to remote server
only).
It is mostly a verbatim code lift from netdump(4). Netdump(4) remains
the only consumer (until the rest of this patch series lands).
The INET-specific logic has been extracted somewhat more thoroughly than
previously in netdump(4), into debugnet_inet.c. UDP-layer logic and up, as
much as possible as is protocol-independent, remains in debugnet.c. The
separation is not perfect and future improvement is welcome. Supporting
INET6 is a long-term goal.
Much of the diff is "gratuitous" renaming from 'netdump_' or 'nd_' to
'debugnet_' or 'dn_' -- sorry. I thought keeping the netdump name on the
generic module would be more confusing than the refactoring.
The only functional change here is the mbuf allocation / tracking. Instead
of initiating solely on netdump-configured interface(s) at dumpon(8)
configuration time, we watch for any debugnet-enabled NIC for link
activation and query it for mbuf parameters at that time. If they exceed
the existing high-water mark allocation, we re-allocate and track the new
high-water mark. Otherwise, we leave the pre-panic mbuf allocation alone.
In a future patch in this series, this will allow initiating netdump from
panic ddb(4) without pre-panic configuration.
No other functional change intended.
Reviewed by: markj (earlier version)
Some discussion with: emaste, jhb
Objection from: marius
Differential Revision: https://reviews.freebsd.org/D21421
Notes
Notes:
svn path=/head/; revision=353685
Diffstat (limited to 'sys/conf/files')
| -rw-r--r-- | sys/conf/files | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sys/conf/files b/sys/conf/files index 48d9fde103ca..d910a1f0ce17 100644 --- a/sys/conf/files +++ b/sys/conf/files @@ -4077,6 +4077,8 @@ net/mp_ring.c optional ether iflib net/mppcc.c optional netgraph_mppc_compression net/mppcd.c optional netgraph_mppc_compression net/netisr.c standard +net/debugnet.c optional inet debugnet +net/debugnet_inet.c optional inet debugnet net/pfil.c optional ether | inet net/radix.c standard net/radix_mpath.c standard |