pmem / pmemfile

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

antool: print all syscalls to analysis log by default #398

Closed ldorau closed 7 years ago

ldorau commented 7 years ago

Only syscalls with a path or a file descriptor among arguments were printed to the analysis log so far. This patch fixes that and makes that all syscalls are printed by default now.

However it has to be turned off in some tests that use threads, because during threads' joining syscalls appear randomly, so they cannot be checked using match files.


This change is Reviewable

codecov[bot] commented 7 years ago

Codecov Report

Merging #398 into master will decrease coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #398      +/-   ##
==========================================
- Coverage   77.02%   77.01%   -0.02%     
==========================================
  Files          67       67              
  Lines        8330     8335       +5     
  Branches     1678     1679       +1     
==========================================
+ Hits         6416     6419       +3     
- Misses       1496     1497       +1     
- Partials      418      419       +1
Flag Coverage Δ
#ltp_tests 46.43% <ø> (-0.02%) :arrow_down:
#sqlite_tests 45.16% <ø> (+0.01%) :arrow_up:
#tests_antool 15.29% <100%> (+0.05%) :arrow_up:
#tests_posix_multi_threaded 24.97% <ø> (-0.09%) :arrow_down:
#tests_posix_single_threaded 54.21% <ø> (ø) :arrow_up:
#tests_preload 44.68% <ø> (ø) :arrow_up:
Impacted Files Coverage Δ
src/tools/antool/antool.py 100% <100%> (ø) :arrow_up:
src/tools/antool/listsyscalls.py 99.15% <100%> (ø) :arrow_up:
src/libpmemfile-posix/data.c 94.42% <0%> (-0.4%) :arrow_down:
src/libpmemfile-posix/file.c 74.44% <0%> (-0.28%) :arrow_down:

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...284eaa8. Read the comment docs.