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

docs, bpf: fix llvm-objdump --no-show-raw-insn options #16848

Merged
merged 1 commit into from Jul 12, 2021

Conversation

claudiajkang
Copy link
Contributor

On LLVM section at "BPF and XDP Reference Guide"
there are typo in llvm-objdump, which is
-no-show-raw-insn. On llvm-objdump man page,
this option declared "--no-show-raw-insn" option.
Thus, this commit fix --no-show-raw-insn.

Signed-off-by: Claudia J. Kang claudiajkang@gmail.com

On LLVM section at "BPF and XDP Reference Guide"
there are typo in llvm-objdump, which is
-no-show-raw-insn. On llvm-objdump man page,
this option declared "--no-show-raw-insn" option.
Thus, this commit fix --no-show-raw-insn.

Signed-off-by: Claudia J. Kang <claudiajkang@gmail.com>
@claudiajkang claudiajkang requested a review from a team July 10, 2021 10:16
@claudiajkang claudiajkang requested a review from a team as a code owner July 10, 2021 10:16
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 10, 2021
Copy link
Contributor

@kkourt kkourt left a comment

Choose a reason for hiding this comment

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

Thank you!

@kkourt kkourt added the release-note/misc This PR makes changes that have no direct user impact. label Jul 12, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 12, 2021
@kkourt
Copy link
Contributor

kkourt commented Jul 12, 2021

Just a documentation change, no need for CI. Merging.

@kkourt kkourt merged commit 8876594 into cilium:master Jul 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants