Re: [PATCH bpf-next v2 1/3] selftests/bpf: do not disable /dev/null device access in cgroup dev test

2024-07-30 Thread Alexis Lothoré
On 7/30/24 10:16, Alan Maguire wrote: > On 29/07/2024 18:30, Alexis Lothoré wrote: >> Hello Alan, [...] >>> Not a big deal, but I found it a bit confusing that this file was >>> modified then deleted in patch 2. Would it work having patch 1 stop >>> building the standalone test/remove it and .git

Re: [PATCH bpf-next v2 1/3] selftests/bpf: do not disable /dev/null device access in cgroup dev test

2024-07-30 Thread Alan Maguire
On 29/07/2024 18:30, Alexis Lothoré wrote: > Hello Alan, > > On 7/29/24 18:59, Alan Maguire wrote: >> On 29/07/2024 09:20, Alexis Lothoré (eBPF Foundation) wrote: >>> test_dev_cgroup currently loads a small bpf program allowing any access on >>> urandom and zero devices, disabling access to any ot

Re: [PATCH bpf-next v2 1/3] selftests/bpf: do not disable /dev/null device access in cgroup dev test

2024-07-29 Thread Alexis Lothoré
Hello Alan, On 7/29/24 18:59, Alan Maguire wrote: > On 29/07/2024 09:20, Alexis Lothoré (eBPF Foundation) wrote: >> test_dev_cgroup currently loads a small bpf program allowing any access on >> urandom and zero devices, disabling access to any other device. It makes >> migrating this test to test_

Re: [PATCH bpf-next v2 1/3] selftests/bpf: do not disable /dev/null device access in cgroup dev test

2024-07-29 Thread Alan Maguire
On 29/07/2024 09:20, Alexis Lothoré (eBPF Foundation) wrote: > test_dev_cgroup currently loads a small bpf program allowing any access on > urandom and zero devices, disabling access to any other device. It makes > migrating this test to test_progs impossible, since this one manipulates > extensive

[PATCH bpf-next v2 1/3] selftests/bpf: do not disable /dev/null device access in cgroup dev test

2024-07-29 Thread eBPF Foundation
test_dev_cgroup currently loads a small bpf program allowing any access on urandom and zero devices, disabling access to any other device. It makes migrating this test to test_progs impossible, since this one manipulates extensively /dev/null. Allow /dev/null manipulation in dev_cgroup program to