On Fri, 16 Nov 2018 18:50:01 +0200 Nikolay Aleksandrov <nikolay@xxxxxxxxxxxxxxxxxxx> wrote: > + if (v->priv_flags & BR_VLFLAG_PER_PORT_STATS) > free_percpu(v->stats); Why not not v->stats == NULL as a flag instead? Then the fact that free_percpu(NULL) is a Nop would mean less code in the bridge driver.