-
Notifications
You must be signed in to change notification settings - Fork 71
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
Unable to find symbol sys_call_table in symbol map #2
Comments
@mariusgrigaitis some systems don't allow a non-root program to read /proc/kallsyms, which meltdown-checker is built upon. Without it, we can't easily check if system is still affected by meltdown. Worth leaving this issue opened until we find another way of doing like running a process, creating a file with very specific data, make sure page cache has it cached, and then we'd search for that data in the kernel memory. |
@mariusgrigaitis i'll update the program to print an informative msg for this scenario. thanks for reporting it. i'll see what i can do to fix this issue. |
Actually i'm able to cat that file. I'm trying to run it inside docker container with full caps. |
@mariusgrigaitis could you please inform what's the output of 'cat /proc/kallsyms | grep sys_call_table'? |
|
@mariusgrigaitis that's awkward, no sys_call_table. What's your kernel version? Could u pls try the same with /boot/System.map*? If it does work on /boot/System.map*, I'll make the checker fallback on it. |
@mariusgrigaitis ed5c4b2 may help you. pull the latest changes and try again with root. |
I have the same problem, my My kernel information is here:
There is also no |
On my stock Ubuntu 17.04 system, $ cat /proc/kallsyms |grep sys_call_table
0000000000000000 R sys_call_table
0000000000000000 R ia32_sys_call_table
$ sudo cat /proc/kallsyms |grep sys_call_table
ffffffff81a00200 R sys_call_table
ffffffff81a01560 R ia32_sys_call_table
$ uname -srvmpio
Linux 4.4.0-102-generic #125-Ubuntu SMP Tue Nov 21 15:15:11 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux |
@dlenski interesting. i added code for user to be aware of it here: https://github.com/raphaelsc/Am-I-affected-by-Meltdown/blob/master/meltdown_checker.cc#L172 I'm currently looking for alternatives to remove the need for root in such systems, like creating a process, make it cache very specific data, then look for that data in page cache. |
That's a pretty cool idea, although… I suspect that most of the users who are grateful for the tool you've created here are either
So I'm thinking that the root requirement is not a big deal for many users. (Perhaps I'm overlooking a large contingient of PaaS users…?) |
@dlenski, @mariusgrigaitis, @graphitemaster: just added the below section to README that may be interesting to you all:
Please tell me if it actually does the tricky |
Required for
(typically ArchLinux/Manjaro may have this off, a seen in #2 (comment) and #2 (comment) too)
console output
|
Tried to run it:
Not sure what that means.
The text was updated successfully, but these errors were encountered: