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

ppc64le: arch/powerpc/tools scripts and kpatch-gcc #976

Open
joe-lawrence opened this issue Jun 12, 2019 · 2 comments
Open

ppc64le: arch/powerpc/tools scripts and kpatch-gcc #976

joe-lawrence opened this issue Jun 12, 2019 · 2 comments
Assignees

Comments

@joe-lawrence
Copy link
Contributor

While testing #973, I noticed these errors in ~/.kpatch/build.log:

./arch/powerpc/tools/relocs_check.sh: line 48: /root/kpatch/kpatch-build/kpatch-gcc objdump: No such file or directory
./arch/powerpc/tools/unrel_branch_check.sh: line 26: /root/kpatch/kpatch-build/kpatch-gcc objdump: No such file or directory
./arch/powerpc/tools/unrel_branch_check.sh: line 37: /root/kpatch/kpatch-build/kpatch-gcc objdump: No such file or directory

I'm guessing that skipping these are benign, but could there be other tools scripts that we may need to properly function in the future? I wonder if the scripts are concatentating argv[0] and argv[1] so it's looking for a file called "/root/kpatch/kpatch-build/kpatch-gcc objdump"?

@joe-lawrence
Copy link
Contributor Author

@kamalesh-babulal - have you seen these in your dev/testing travels?

@kamalesh-babulal
Copy link
Contributor

have you seen these in your dev/testing travels?

@joe-lawrence Sorry, I planned to reply long back, got busy with other things at work. Yes, I started carefully looking for them recently but these warnings should be there for long. The in kernel tools expects the arguments passed to them to be:

"/bin/sh ./arch/powerpc/tools/relocs_check.sh "objdump" "vmlinux"
/bin/sh ./arch/powerpc/tools/unrel_branch_check.sh "objdump" "vmlinux"

They warn about the wrong relocation entries and direct branch(s) from the code below __end_interrupts to the code above __end_interrupts symbol, in case of the relocatable kernel. They do not error out but warn during the kernel build, we need to fix the arguments getting passed to these tools.

@joe-lawrence joe-lawrence self-assigned this Apr 18, 2023
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

No branches or pull requests

2 participants