Except for `trunkproto' wich his trunk(4) specific, aggr(4) has the same
options so I'd like to merge it into the same section just like TUNNEL
documents mostly identical interfaces with differences mentioned in at
specific options.

The wording "trunk" seems clear under OpenBSD, but other vendors give it
a different meaning (e.g. Cisco), so I slightly enhanced the section
name as well.

Feedback? Objections? OK?

Another thing: Note that `lacpmode' and `lacptimeout' are missing from
the existing TRUNK synopsis - I hesitate to add them because it feels
like too much duplication.  Would it perhaps be worthwile to drop each
interface type section's synopsis all together?  Most options are under
DESCRIPTION and the second biggest section is BRIDGE which does not have
a synopsis, either.  Less seems more here.

If at all, this seems best handled in a different commit.

NB: At least tpmr(4) is missing as well, which I'll handle soon as well.

Index: ifconfig.8
RCS file: /cvs/src/sbin/ifconfig/ifconfig.8,v
retrieving revision 1.352
diff -u -p -r1.352 ifconfig.8
--- ifconfig.8  27 Jun 2020 17:46:29 -0000      1.352
+++ ifconfig.8  26 Jul 2020 17:22:24 -0000
@@ -1759,15 +1759,17 @@ from all protected domains.
 .It Cm up
 Start the switch processing packets.
 .Nm ifconfig
 .Ar trunk-interface
 .Op Oo Fl Oc Ns Cm trunkport Ar child-iface
 .Op Cm trunkproto Ar proto
-The following options are available for a
+The following options are available for
+.Xr aggr 4
 .Xr trunk 4
 .Bl -tag -width Ds
 .It Cm lacpmode Cm active Ns | Ns Cm passive
 Set the LACP trunk mode to either
@@ -1787,7 +1789,9 @@ as a trunk port.
 Remove the trunk port
 .Ar child-iface .
 .It Cm trunkproto Ar proto
-Set the trunk protocol.
+Set the link aggregation protocol on
+.Xr trunk 4
 Refer to
 .Xr trunk 4
 for a complete list of the available protocols.

Reply via email to