Thread (6 messages) 6 messages, 5 authors, 2021-12-10

Re: [PATCH] bpf: return EOPNOTSUPP when JIT is needed and not possible

flat view

From: Jakub Kicinski <kuba@kernel.org>
Date: 2021-12-10 02:23:55
Also in: bpf, lkml

On Fri, 10 Dec 2021 00:03:40 +0100 Daniel Borkmann wrote:
quoted
Similar issue was discussed in the past. See:
https://lore.kernel.org/netdev/20191204.125135.750458923752225025.davem@davemloft.net/ (local)  
With regards to ENOTSUPP exposure, if the consensus is that we should fix all
occurences over to EOPNOTSUPP even if they've been exposed for quite some time
(Jakub?), 
Did you mean me? :) In case you did - I think we should avoid it 
for new code but changing existing now seems risky. Alexei and Andrii
would know best but quick search of code bases at work reveals some
scripts looking for ENOTSUPP.

Thadeu, what motivated the change?

If we're getting those changes fixes based on checkpatch output maybe 
there is a way to mute the checkpatch warnings when it's not run on a 
diff?
we could give this patch a try maybe via bpf-next and see if anyone complains.

Thadeu, I think you also need to fix up BPF selftests as test_verifier, to mention
one example (there are also bunch of others under tools/testing/selftests/), is
checking for ENOTSUPP specifically..
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help