On Wed, Feb 22, 2023 at 03:44:17PM -0800, Martin KaFai Lau wrote: > > + system("ip netns del " NS_TEST " >& /dev/null"); > > It needs to be "&>", like the fix in commit 98e13848cf43 ("selftests/bpf: > Fix decap_sanity_ns cleanup"). :Shame, Didn't notice this when do copy/paste... > > Since it needs to respin, could you help and take this chance to put the > above SYS() macro into the test_progs.h. Other selftests are doing similar > thing also. If possible, it may be easier to have a configurable > "goto_label" as the first arg. OK, I will fix it. Thanks Hangbin