On Thu, Oct 31, 2019 at 5:51 PM Jakub Kicinski <jakub.kicinski@xxxxxxxxxxxxx> wrote: > > This reverts commit 5bc60de50dfe ("selftests: bpf: Don't try to read > files without read permission"). > > Quoted commit does not work at all, and was never tested. > Script requires root permissions (and tests for them) > and os.access() will always return true for root. > > The correct fix is needed in the bpf tree, so let's just > revert and save ourselves the merge conflict. > > Signed-off-by: Jakub Kicinski <jakub.kicinski@xxxxxxxxxxxxx> Acked-by: Alexei Starovoitov <ast@xxxxxxxxxx> Since original commit is broken may be apply directly to net-next ? I'm fine whichever way. I would wait for Jiri to reply first though.