Bug 21297 - support needed for new statx syscall
Summary: support needed for new statx syscall
Status: RESOLVED FIXED
Alias: None
Product: systemtap
Classification: Unclassified
Component: tapsets (show other bugs)
Version: unspecified
: P2 normal
Target Milestone: ---
Assignee: Martin Cermak
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-03-23 20:59 UTC by David Smith
Modified: 2017-04-04 08:29 UTC (History)
2 users (show)

See Also:
Host:
Target:
Build:
Last reconfirmed:

Attachments
possible patch (2.56 KB, patch)
2017-03-31 14:49 UTC, Martin Cermak 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Smith 2017-03-23 20:59:50 UTC 
The new statx() syscall has been added in the 4.11 kernel. Support needs to be added to the syscall tapset.

Some more information:

https://lwn.net/Articles/685519/
Comment 1 Martin Cermak 2017-03-31 14:49:40 UTC 
Created attachment 9959 [details]
possible patch
Comment 2 David Smith 2017-03-31 16:30:18 UTC 
Comment on attachment 9959 [details]
possible patch

The patch looks reasonable. I do have a question about the changes to testsuite/systemtap.syscall/nd_sys.stp. Why the change from 'probe nd_syscall.*' to 'probe syscall.getcwd ?, syscall.*open* ?, syscall.*stat* ?, syscall.*time* ?, syscall.creat ?'?
Comment 3 Martin Cermak 2017-03-31 16:37:49 UTC 
Err, that was a temporary tweak speeding up the module compilation during the patch development.  I've mistakenly left that in place.  Thanks a lot for spotting that.
Comment 4 Martin Cermak 2017-04-04 08:29:03 UTC 
Fixed in commit 4ae155ae612963b95d9a9a05273ab4baade1cf83