Summary: | WARNING: unhandled amd64-linux syscall: 313 (finit_module) | ||
---|---|---|---|
Product: | [Developer tools] valgrind | Reporter: | Bartosz Golaszewski <bartekgola> |
Component: | memcheck | Assignee: | Julian Seward <jseward> |
Status: | CONFIRMED --- | ||
Severity: | minor | CC: | bartekgola, ivosh, mark, tom |
Priority: | NOR | ||
Version: | 3.13 SVN | ||
Target Milestone: | --- | ||
Platform: | Debian stable | ||
OS: | Linux | ||
Latest Commit: | r16395 | Version Fixed In: | |
Attachments: | add wrapper for finit_module() |
Description
Bartosz Golaszewski
2017-05-18 08:58:58 UTC
That is the finit_module which loads a kernel module from a file descriptor. Right, I checked the wrong file. It's in /usr/include/x86_64-linux-gnu/asm/unistd_64.h on Debian. I'll try to create a wrapper for this syscall. Sure there is - it's finit_module. Hmm... it's already there in ./coregrind/m_syswrap/syswrap-amd64-linux.c but it's commented out: 835 // LIN__(__NR_finit_module, sys_ni_syscall), // 313 Does it mean that it just doesn't have a wrapper in valgrind or some other reason? It means the syscall wrapper needs a careful review w.r.t. kernel sources and ideally also a test case. Created attachment 105623 [details]
add wrapper for finit_module()
There was no wrapper - only the basic definitions. I wrote the wrapper and it fixes the issue on my setup. Patch attached.
(In reply to Bartosz Golaszewski from comment #6) Committed, r16403. Thanks for the patch. |