Skip to content

doc: update BPF instruction limit#9727

Merged
aanm merged 1 commit into
cilium:masterfrom
florianl:BPF_COMPLEXITY_LIMIT_INSNS
Dec 16, 2019
Merged

doc: update BPF instruction limit#9727
aanm merged 1 commit into
cilium:masterfrom
florianl:BPF_COMPLEXITY_LIMIT_INSNS

Conversation

@florianl
Copy link
Copy Markdown
Contributor

@florianl florianl commented Dec 9, 2019 •
edited by tgraf
Loading

Signed-off-by: Lehner Florian dev@der-flo.net

Please ensure your pull request adheres to the following guidelines:

  • [ x] For first time contributors, read Submitting a pull request
  • [~] All code is covered by unit and/or runtime tests where feasible.
    no code was changed
  • [~] All commits contain a well written commit description including a title,
    description and a Fixes: #XXX line if the commit addresses a particular
    GitHub issue.
    no code was changed
  • All commits are signed off. See the section Developer’s Certificate of Origin.
  • Provide a title or release-note blurb suitable for the release notes.
  • Thanks for contributing!

This PR updates the documentation of the BPF instruction limit. This limit was changed in [1] from 4096 to 1M.

[1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=c04c0d2b968ac45d6ef020316808ef6c82325a82

This change is Reviewable

@florianl
doc: update BPF instruction limit
d9ae27a 
Signed-off-by: Lehner Florian <dev@der-flo.net>
@florianl florianl requested a review from borkmann as a code owner December 9, 2019 12:00
@coveralls
Copy link
Copy Markdown

coveralls commented Dec 9, 2019

Coverage Status

Coverage decreased (-0.007%) to 45.986% when pulling d9ae27a on florianl:BPF_COMPLEXITY_LIMIT_INSNS into fd3670b on cilium:master.

@aanm aanm added area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. pending-review release-note/misc This PR makes changes that have no direct user impact. labels Dec 9, 2019
@aanm
Copy link
Copy Markdown
Member

aanm commented Dec 9, 2019

test-docs-please

soumynathan
soumynathan reviewed Dec 9, 2019
View reviewed changes
Copy link
Copy Markdown

@soumynathan soumynathan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@aanm aanm merged commit 9d3c94a into cilium:master Dec 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@borkmann borkmann Awaiting requested review from borkmann

1 more reviewer

@soumynathan soumynathan soumynathan left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@borkmann borkmann

Labels

area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@florianl @coveralls @aanm @soumynathan @borkmann