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

tools: Fix BCC bio tools with recent kernel change #3748

Merged
merged 1 commit into from
Dec 15, 2021

Conversation

chenhengqi
Copy link
Collaborator

Several BCC bio tools are broken due to kernel change (0).
blk_account_io_{start, done} were renamed to _blk_account_io{start, done},
and the symbols gone from /proc/kallsyms. Fix them by checking symbol existence.

Signed-off-by: Hengqi Chen chenhengqi@outlook.com

Several BCC bio tools are broken due to kernel change ([0]).
blk_account_io_{start, done} were renamed to __blk_account_io_{start, done},
and the symbols gone from /proc/kallsyms. Fix them by checking symbol existence.

  [0]: torvalds/linux@be6bfe3

Signed-off-by: Hengqi Chen <chenhengqi@outlook.com>
@davemarchevsky davemarchevsky merged commit 7913ce0 into iovisor:master Dec 15, 2021
chenhengqi added a commit to chenhengqi/bcc that referenced this pull request Feb 24, 2022
Kernel commit 24b83deb29b ("block: move struct request to blk-mq.h")
and be6bfe36db17 ("block: inline hot paths of blk_account_io_*()")
introduce changes which break disksnoop.

Like iovisor#3748 and iovisor#3877 but for disksnoop, this commit fixes those issues.

Closes iovisor#3825.

Signed-off-by: Hengqi Chen <chenhengqi@outlook.com>
yonghong-song pushed a commit that referenced this pull request Feb 24, 2022
Kernel commit 24b83deb29b ("block: move struct request to blk-mq.h")
and be6bfe36db17 ("block: inline hot paths of blk_account_io_*()")
introduce changes which break disksnoop.

Like #3748 and #3877 but for disksnoop, this commit fixes those issues.

Closes #3825.

Signed-off-by: Hengqi Chen <chenhengqi@outlook.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants