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

compile original module for create-diff-object #4

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

Conversation

joe-lawrence
Copy link
Owner

Normally, the symbols in the freshly built object file that
create-diff-object used are the same with our original module.

But the name of the symbols may change if use --skip-gcc-check
or the influence of KCFLAGS. This patch compile original module
for create-diff-object, like original vmlinux, to detect this
problem earlier, before we need to insmod klp.ko.

Signed-off-by: Zhou Chengming <zhouchengming1@huawei.xyz>

@joe-lawrence
Copy link
Owner Author

kpatch-bot - Ubuntu 16.04.1 LTS (Xenial Xerus) - 4.4.0-57-generic
PASS build  PASS make-clean  FAIL static-checker  SKIP quick  SKIP long  

@joe-lawrence
Copy link
Owner Author

kpatch-bot - Fedora 25 (Cloud Edition) - 4.8.6-300.fc25.x86_64
PASS build  PASS make-clean  PASS static-checker  SKIP quick  SKIP long  

@joe-lawrence
Copy link
Owner Author

kpatch-bot - CentOS Linux 7 (Core) - 3.10.0-514.2.2.el7.x86_64
PASS build  PASS make-clean  PASS static-checker  SKIP quick  SKIP long  

@joe-lawrence joe-lawrence force-pushed the zhouchengming1_iss612 branch 27 times, most recently from 6425293 to fa46f3f Compare January 6, 2017 20:21
@joe-lawrence joe-lawrence force-pushed the zhouchengming1_iss612 branch 5 times, most recently from 643902b to bff8791 Compare January 6, 2017 21:53
Normally, the symbols in the freshly built object file that
create-diff-object used are the same with our original module.

But the name of the symbols may change if use --skip-gcc-check
or the influence of KCFLAGS. This patch compile original module
for create-diff-object, like original vmlinux, to detect this
problem earlier, before we need to insmod klp.ko.

Signed-off-by: Zhou Chengming <zhouchengming1@_huawei.xyz_>
@joe-lawrence joe-lawrence force-pushed the zhouchengming1_iss612 branch from bff8791 to d2c72f2 Compare January 9, 2017 14:25
@joe-lawrence
Copy link
Owner Author

kpatch-bot - Ubuntu 16.04.1 LTS (Xenial Xerus) - 4.4.0-57-generic
PASS build  PASS make-clean  FAIL static-checker  

@joe-lawrence
Copy link
Owner Author

kpatch-bot - CentOS Linux 7 (Core) - 3.10.0-514.2.2.el7.x86_64
PASS build  PASS make-clean  PASS static-checker  

@joe-lawrence
Copy link
Owner Author

kpatch-bot - Fedora 25 (Cloud Edition) - 4.8.6-300.fc25.x86_64
PASS build  PASS make-clean  PASS static-checker  

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.

1 participant