On Mon, Dec 05, 2022 at 01:55:05PM +0200, Nikolay Aleksandrov wrote: > One thought (not a big deal) but it would've been ideal if we could initialize the config > struct once when parsing and then pass it around as a const argument. I know that its > arguments are currently passed to functions that don't expect const, but I *think* it > could be a small change. OK, I've made the change. It was quite painful to rebase my next patchset on top of it, but it's done now :) As a result of this change, I've appended one small patch to v2 of this patchset. It is constifying the 'group' argument of br_multicast_new_port_group(). It is a dependency of the next patchset which is already close to the limit in terms of size (most patches are small). Thanks!