Re: [PATCH bpf-next] selftests/bpf: Remove "&>" usage in the selftests

[Yonghong Song <yonghong.song@linux.dev>]

On 1/26/24 10:14 PM, Martin KaFai Lau wrote:
> On 1/26/24 8:56 PM, Yonghong Song wrote:
>>
>> On 1/26/24 6:50 PM, Martin KaFai Lau wrote:
>>> From: Martin KaFai Lau <martin.lau@kernel.org>
>>>
>>> In s390, CI reported that the sock_iter_batch selftest
>>> hits this error very often:
>>>
>>> 2024-01-26T16:56:49.3091804Z Bind /proc/self/ns/net -> 
>>> /run/netns/sock_iter_batch_netns failed: No such file or directory
>>> 2024-01-26T16:56:49.3149524Z Cannot remove namespace file 
>>> "/run/netns/sock_iter_batch_netns": No such file or directory
>>> 2024-01-26T16:56:49.3772213Z test_sock_iter_batch:FAIL:ip netns add 
>>> sock_iter_batch_netns unexpected error: 256 (errno 0)
>>>
>>> It happens very often in s390 but Manu also noticed it happens very
>>> sparsely in other arch also.
>>>
>>> It turns out the default dash shell does not recognize "&>"
>>
>> Not sure whether it is feasible or not. But is it possible
>> for all our test VMs we run '/bin/bash' before everyting else
>> so we have a uniform bash environment so we do not need to
>> worry about other shells?
>
> It was my initial thought also. I think it makes sense to use the 
> minimal shell feature such that it is more portable to different 
> developer environments.

I just proposed an alternate solution. Since you have throught about this, I am certainly okay with the current patch.