Ido Schimmel <[email protected]> writes:
> On Tue, Sep 16, 2025 at 06:25:46PM +0200, Petr Machata wrote:
>> @@ -635,6 +648,7 @@ static void bridge_print_opt(struct link_util *lu, FILE
>> *f, struct rtattr *tb[])
>>
>> if (tb[IFLA_BR_MULTI_BOOLOPT]) {
>> __u32 mofn_bit = 1 << BR_BOOLOPT_MDB_OFFLOAD_FAIL_NOTIFICATION;
>> + __u32 fdb_vlan_0_bit = 1 << BR_BOOLOPT_FDB_LOCAL_VLAN_0;
>> __u32 mcvl_bit = 1 << BR_BOOLOPT_MCAST_VLAN_SNOOPING;
>> __u32 no_ll_learn_bit = 1 << BR_BOOLOPT_NO_LL_LEARN;
>> __u32 mst_bit = 1 << BR_BOOLOPT_MST_ENABLE;
>> @@ -661,6 +675,11 @@ static void bridge_print_opt(struct link_util *lu, FILE
>> *f, struct rtattr *tb[])
>> "mdb_offload_fail_notification",
>> "mdb_offload_fail_notification %u ",
>> !!(bm->optval & mofn_bit));
>> + if (bm->optval & fdb_vlan_0_bit)
>
> Shouldn't this be 'bm->optmask' ?
It should.
