pmem / pmemfile

Userspace implementation of file APIs using persistent memory.
Other
34 stars 21 forks source link

antool: add option for printing syscalls to analysis log #401

Closed ldorau closed 6 years ago

ldorau commented 6 years ago

Add the "--print-syscalls-without-path-or-fd" option which makes it possible to print to the analysis log also syscalls without a path or a file descriptor among arguments.


This change is Reviewable

codecov[bot] commented 6 years ago

Codecov Report

Merging #401 into master will increase coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #401      +/-   ##
==========================================
+ Coverage   77.02%   77.03%   +0.01%     
==========================================
  Files          67       67              
  Lines        8330     8335       +5     
  Branches     1678     1679       +1     
==========================================
+ Hits         6416     6421       +5     
  Misses       1496     1496              
  Partials      418      418
Flag Coverage Δ
#ltp_tests 46.46% <ø> (+0.01%) :arrow_up:
#sqlite_tests 45.16% <ø> (+0.01%) :arrow_up:
#tests_antool 15.29% <100%> (+0.05%) :arrow_up:
#tests_posix_multi_threaded 25.06% <ø> (ø) :arrow_up:
#tests_posix_single_threaded 54.18% <ø> (-0.03%) :arrow_down:
#tests_preload 44.6% <ø> (-0.08%) :arrow_down:
Impacted Files Coverage Δ
src/tools/antool/antool.py 100% <100%> (ø) :arrow_up:
src/tools/antool/listsyscalls.py 99.15% <100%> (ø) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update af1a8ce...2ab4f42. Read the comment docs.

krzycz commented 6 years ago

Reviewed 2 of 4 files at r1, 2 of 2 files at r2. Review status: all files reviewed at latest revision, 1 unresolved discussion.


Comments from Reviewable

sarahjelinek commented 6 years ago
:lgtm:

Reviewed 2 of 4 files at r1, 2 of 2 files at r2. Review status: all files reviewed at latest revision, 1 unresolved discussion.


Comments from Reviewable