On Tue, 10 Dec 2019 13:16:33 -0500 Vivien Didelot <vivien.didelot@xxxxxxxxx> wrote: > Hi Stephen, > > On Mon, 9 Dec 2019 16:13:45 -0800, Stephen Hemminger <stephen@xxxxxxxxxxxxxxxxxx> wrote: > > On Mon, 9 Dec 2019 18:05:22 -0500 > > Vivien Didelot <vivien.didelot@xxxxxxxxx> wrote: > > > > > Add support for the BRIDGE_XSTATS_STP xstats, as follow: > > > > > > # ip link xstats type bridge_slave dev lan5 > > > STP BPDU: > > > RX: 0 > > > TX: 39 > > > STP TCN: > > > RX: 0 > > > TX: 0 > > > STP Transitions: > > > Blocked: 0 > > > Forwarding: 1 > > > IGMP queries: > > > RX: v1 0 v2 0 v3 0 > > > TX: v1 0 v2 0 v3 0 > > > ... > > > > Might I suggest a more concise format: > > STP BPDU: RX: 0 TX: 39 > > STP TCN: RX: 0 TX:0 > > STP Transitions: Blocked: 0 Forwarding: 1 > > ... > > I don't mind if you prefer this format ;-) Just trying to reduce the long output. Which is already too long for the xstat as it is.