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

Workaround create-diff-object issue #1405

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blitz
Copy link
Contributor

@blitz blitz commented Jul 24, 2024

We've seen this issue on Linux 6.9 and 6.10:

create-diff-object: ERROR: x86.o: kpatch_bundle_symbols: 251: symbol get_cpu_tsc_khz at offset 16 within section .text.get_cpu_tsc_khz, expected 0

We found this workaround at:
https://gitee.com/src-openeuler/kpatch/issues/I9I3EZ

We are not sure whether this fix is sane, but the result seems to work. :)

We've seen this on Linux 6.9:

create-diff-object: ERROR: x86.o: kpatch_bundle_symbols: 251: symbol get_cpu_tsc_khz at offset 16 within section .text.get_cpu_tsc_khz, expected 0

The workaround comes from:
https://gitee.com/src-openeuler/kpatch/issues/I9I3EZ

Signed-off-by: Julian Stecklina <[email protected]>
Signed-off-by: Thomas Prescher <[email protected]>
@joe-lawrence
Copy link
Contributor

Hi @blitz, thanks for the report.

Is there any chance you could provide a kernel config (maybe it's the same as #1403 ?) and an input kpatch that will reproduce this error. This looks very familiar in my KERNEL_IBT travels (the aborted #1350 perhaps?)

@blitz
Copy link
Contributor Author

blitz commented Jul 29, 2024

Hi @blitz, thanks for the report.

Is there any chance you could provide a kernel config (maybe it's the same as #1403 ?) and an input kpatch that will reproduce this error. This looks very familiar in my KERNEL_IBT travels (the aborted #1350 perhaps?)

You should be able to reproduce it with the config and patch from here: #1404

I'll try without KERNEL_IBT.

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