-
Notifications
You must be signed in to change notification settings - Fork 652
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add v5.14 to CI builds #462
Conversation
asm.Instruction only supports %v, not %s.
We currently have a failing test that is skipped on all CI builds, since it specifies a minimum version higher than the maximum version we test against. Prevent this from happening again by failing tests that are never executed.
@@ -66,6 +66,10 @@ func TestHaveProgType(t *testing.T) { | |||
} | |||
testutils.SkipOnOldKernel(t, minVersion, feature) | |||
|
|||
if pt == ebpf.Syscall { | |||
t.Skip("HaveProgType(Syscall) is broken on 5.14") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cc @rgo3 this needs some looking at, seems like the feature test doesn't actually work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I've opened an issue (#464) to keep track and will look into it asap (feel free to assign me). Since the program type is quite new, and bpftool
doesn't have a probe for it either I'll have to check the kernel code to figure out how to fix it. Once this PR is merged I can use the new CI kernel to debug the issue as I don't have 5.14 installed locally.
cc @mehrdadrad this will unblock your PR. |
t.Skip("Skipping due to missing InnerMap in map definition") | ||
case "test_core_autosize.o": | ||
t.Skip("Skipping since the test generates dynamic BTF") | ||
case "for_each_array_map_elem.o", "for_each_array_map_elem.linked3.o", | ||
"for_each_hash_map_elem.o", "for_each_hash_map_elem.linked3.o": | ||
t.Skip("Skipping since function pointers are not supported") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
case "test_static_linked.linked3.o": | ||
t.Skip("Skipping since .text contains 'subprog' twice") | ||
case "linked_maps.linked3.o", "linked_funcs.linked3.o": | ||
t.Skip("Skipping since weak relocations are not supported") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There are a couple of new features that break integration tests:
There is also a bunch of stuff around static linking of objects which I don't understand. Not sure if that is a bug in libbpf or a feature I don't understand.