[Bridge] [PATCH net-next v2] net: bridge: vlan: include stats in dumps if requested
David Miller
davem at davemloft.net
Fri Mar 20 03:22:06 UTC 2020
From: Nikolay Aleksandrov <nikolay at cumulusnetworks.com>
Date: Thu, 19 Mar 2020 12:14:14 +0200
> This patch adds support for vlan stats to be included when dumping vlan
> information. We have to dump them only when explicitly requested (thus the
> flag below) because that disables the vlan range compression and will make
> the dump significantly larger. In order to request the stats to be
> included we add a new dump attribute called BRIDGE_VLANDB_DUMP_FLAGS which
> can affect dumps with the following first flag:
> - BRIDGE_VLANDB_DUMPF_STATS
> The stats are intentionally nested and put into separate attributes to make
> it easier for extending later since we plan to add per-vlan mcast stats,
> drop stats and possibly STP stats. This is the last missing piece from the
> new vlan API which makes the dumped vlan information complete.
>
> A dump request which should include stats looks like:
> [BRIDGE_VLANDB_DUMP_FLAGS] |= BRIDGE_VLANDB_DUMPF_STATS
>
> A vlandb entry attribute with stats looks like:
> [BRIDGE_VLANDB_ENTRY] = {
> [BRIDGE_VLANDB_ENTRY_STATS] = {
> [BRIDGE_VLANDB_STATS_RX_BYTES]
> [BRIDGE_VLANDB_STATS_RX_PACKETS]
> ...
> }
> }
>
> Signed-off-by: Nikolay Aleksandrov <nikolay at cumulusnetworks.com>
> ---
> v2: Use a separate dump attribute for the flags instead of a reserved
> field to avoid uapi breakage as noted by DaveM.
> Rebased and retested on the latest net-next.
Thanks for reworking UAPI.
Applied, thanks.
More information about the Bridge
mailing list