[an error occurred while processing this directive]
|
[an error occurred while processing this directive]
|
|
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <aYud8nWZ7c0RxRIx@sellars>
Date: Tue, 10 Feb 2026 22:06:58 +0100
From: Linus Lüssing <linus.luessing@...3.blue>
To: Ido Schimmel <idosch@...dia.com>
Cc: bridge@...ts.linux.dev, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org,
Nikolay Aleksandrov <razor@...ckwall.org>,
Andrew Lunn <andrew+netdev@...n.ch>,
Simon Horman <horms@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
Jakub Kicinski <kuba@...nel.org>,
Eric Dumazet <edumazet@...gle.com>,
"David S . Miller" <davem@...emloft.net>,
Kuniyuki Iwashima <kuniyu@...gle.com>,
Stanislav Fomichev <sdf@...ichev.me>,
Xiao Liang <shaw.leon@...il.com>
Subject: Re: [PATCH net-next v2 02/14] net: bridge: mcast: track active
state, adding tests
On Sun, Feb 08, 2026 at 06:00:46PM +0200, Ido Schimmel wrote:
[...]
> > +test_inactive_other_querier_norespdelay()
> > +{
> > + RET=0
> > +
> > + test_active_setup_bridge "1" "2" "3" "4" "5" "6"
> > + test_active_setup_config "1" "2" "3" ""
> > + #test_active_setup_wait
>
> Why the comment? There are more instances below
The intention was just to highlight what was skipped in comparison
to the according test_{vlan,}_active_*_querier() variants.
Do you think I should remove these lines? Or add a brief
clarification, something like "# skipped: test_active_setup_wait"?
Powered by blists - more mailing lists