Skip to content
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

vendor: update github.com/cilium/ebpf #12068

Merged
merged 1 commit into from Jun 16, 2020
Merged

Conversation

tklauser
Copy link
Member

@tklauser tklauser commented Jun 15, 2020

This pulls in cilium/ebpf#81 which fixes a crash when trying to
initialize BPF per ring buffers for offline CPUs:

level=fatal msg="Cannot initialise BPF perf ring buffer sockets" error="failed to create perf ring for CPU 2: can't create perf event: can't create perf event: no such device" startTime="2020-06-15 12:15:09.153912253 +0000 UTC m=+129.850487215" subsys=monitor-agent

Reported-by: Sebastian Wicki sebastian@isovalent.com
Signed-off-by: Tobias Klauser tklauser@distanz.ch

Update the Cilium eBPF library to the latest version.
Fix issue where Cilium could crash on startup with "can't create perf event: no such device".

@tklauser tklauser added release-note/minor This PR changes functionality that users may find relevant to operating Cilium. needs-backport/1.8 labels Jun 15, 2020
@tklauser tklauser requested a review from gandro June 15, 2020 13:31
@tklauser tklauser requested a review from a team as a code owner June 15, 2020 13:31
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.8.0 Jun 15, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.8.0 Jun 15, 2020
This pulls in cilium/ebpf#81 which fixes a crash when trying to
initialize BPF per ring buffers for offline CPUs:

  level=fatal msg="Cannot initialise BPF perf ring buffer sockets" error="failed to create perf ring for CPU 2: can't create perf event: can't create perf event: no such device" startTime="2020-06-15 12:15:09.153912253 +0000 UTC m=+129.850487215" subsys=monitor-agent

Reported-by: Sebastian Wicki <sebastian@isovalent.com>
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
@tklauser tklauser force-pushed the pr/tklauser/vendor-cilium-ebpf branch from 0a10d29 to cab77da Compare June 15, 2020 13:36
@tklauser
Copy link
Member Author

test-me-please

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.009%) to 37.036% when pulling cab77da on pr/tklauser/vendor-cilium-ebpf into 831c5ee on master.

@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jun 16, 2020
@borkmann borkmann merged commit 00ef71b into master Jun 16, 2020
1.8.0 automation moved this from In progress to Merged Jun 16, 2020
@borkmann borkmann deleted the pr/tklauser/vendor-cilium-ebpf branch June 16, 2020 11:41
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.0 Jun 16, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.0 Jun 16, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.0 Jun 16, 2020
@gandro
Copy link
Member

gandro commented Jul 8, 2020

We've found that #12363 has caused the reported panic to now also occur on 1.7.6. We need to backport this to 1.7.7 to avoid crashes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
No open projects
1.7.7
Backport done to v1.7
1.8.0
  
Merged
1.8.0
Backport done to v1.8
Development

Successfully merging this pull request may close these issues.

None yet

7 participants