On Mon, 9 Jun 2025 16:19:34 -0700 Stanislav Fomichev wrote: > On 06/09, Jakub Kicinski wrote: > > On Mon, 9 Jun 2025 09:25:37 -0700 Stanislav Fomichev wrote: > > > Recently bnxt had to grow back a bunch of rtnl dependencies because > > > of udp_tunnel's infra. Add separate (global) mutext to protect > > > udp_tunnel state. > > > > Appears to break the selftest, unfortunately: > > https://netdev.bots.linux.dev/contest.html?test=udp-tunnel-nic-sh&branch=net-next-2025-06-09--21-00 > > > > Argh, should have run it locally first :-( > Looks like there is a test that sets up pretty high sleep time (1 sec) > and expects entry to not appear during next 'ethtool --show-tunnels' run. > > Gonna double check and remove the case if my understanding is correct. > Don't think there is much value in keeping the debugfs knob just for the > sake of this test? LMK if you disagree; otherwise gonna repost tomorrow.
Hm, I see you partially deleted the sleep support in patch 3. Maybe it's easier to keep it since we now always sleep? No strong preference, tho.
