netgraph/ng_vlan_rotate: IEEE 802.1ad VLAN manipulation netgraph node type (new type)

[lutz_donnerhacke.de]

Summary

This node is part of an A10-NSP development.

Carrier networks tend to stack three or more tags for internal purposes and therefore hiding the service tags deep inside of the stack. When decomposing such an access network frame, the processing order is typically reversed: First distinguish by service, than by other means.

This new netgragh node allows to bring the relevant VLAN in front (to the out-most position). This way other netgraph nodes (like ng_vlan) can operate on this specific type.

Test Plan

The example section of the man page contains two examples, how to use this node.
It also contains the sniffer insight of the manipulation process.

[brueffer]

Minor notes on the manpage, that one looks good otherwise. Can't comment on the code.

share/man/man4/ng_vlan_rotate.4

76

stray comma

93

stray comma

97

on -> on the

157

stray comma

253

This can simply be ".Fx 12.1", we don't count prereleases, betas etc here.

[lutz_donnerhacke.de]

Fixed manpage bugs.

[lutz_donnerhacke.de]

Update inline notes.

[kevans]

Just some style nits, some of which are applicable in places other than just where I left them. Functionally, it appears to be OK, but adding Julian while I'm still diving into netgraph for bluetooth bits.

sys/netgraph/ng_vlan_rotate.c

224

Spurious whitespace on blank lines in a couple spots

228

The various case lines should line up with the switch

246

Space between if and opening paren, error assignments should move to the next line

253

Indentation on this guy should be tabs all the way

362

We don't traditionally space assignments out like this, but I don't know that anyone's going to hassle over it.

415

Opening comment marker should rest on its own line

473

Operator belongs to the line before it, with the wrapping starting just after.

[lutz_donnerhacke.de]

Run indent(1) on the source files.
Reduced comments from the boilerplate.
Changed hard coded ethertypes by global definitions introduced by D21846.

[lutz_donnerhacke.de]

Rebase to r357812

[lutz_donnerhacke.de]

rebase to r358008.

[lutz_donnerhacke.de]

Updated to revision 358170.

[lutz_donnerhacke.de]

Updated to revision 358500.

[lutz_donnerhacke.de]

We are going to hard production now.
Any interest in reviewing it?

[lutz_donnerhacke.de]

@brueffer Are your concerns resolved?

[lutz_donnerhacke.de]

Rebase to r366170.

[gbe]

LGTM

[lutz_donnerhacke.de]

Updated to revision 367040.

[gbe]

LGTM.

share/man/man4/ng_vlan_rotate.4

2

The "All rights reserved." should be dropped.

sys/netgraph/ng_vlan_rotate.c

5

The "All rights reserved." should be dropped.

sys/netgraph/ng_vlan_rotate.h

5

The "All rights reserved." should be dropped.

[afedorov]

share/man/man4/ng_vlan_rotate.4

2

While we're here, maybe it should be added: "SPDX-License-Identifier: BSD-2-Clause-FreeBSD"

[lutz_donnerhacke.de]

Updated to latest revision and fixed copyright annotations.

[lutz_donnerhacke.de]

Mark annotations as done.

[lutz_donnerhacke.de]

Updated to revision 368146.