pmem / pmemfile

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

preload: unbreak writev #309

Closed marcinslusarz closed 7 years ago

marcinslusarz commented 7 years ago

Regression from 46e9ebf88e937be60805206790f95a433b6da6a0.


This change is Reviewable

codecov-io commented 7 years ago

Codecov Report

Merging #309 into master will decrease coverage by 0.03%. The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   79.32%   79.29%   -0.04%     
==========================================
  Files          81       81              
  Lines       12058    12068      +10     
  Branches     1628     1629       +1     
==========================================
+ Hits         9565     9569       +4     
- Misses       1886     1892       +6     
  Partials      607      607
Impacted Files Coverage Δ
src/libpmemfile/preload.c 44.57% <0%> (+0.06%) :arrow_up:
src/libpmemfile/syscall_early_filter.c 60% <0%> (-20%) :arrow_down:
src/libpmemfile/libpmemfile-posix-fd_first.h 50% <0%> (-2.95%) :arrow_down:
src/libpmemfile/libpmemfile-posix-wrappers.h 47.42% <0%> (-1.69%) :arrow_down:
src/libpmemfile-posix/rename.c 91.12% <0%> (-0.34%) :arrow_down:
src/libpmemfile-posix/file.c 66.39% <0%> (+0.18%) :arrow_up:
tests/posix/mt/mt.cpp 86.45% <0%> (+1.56%) :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 c6f297d...345b49b. Read the comment docs.

krzycz commented 7 years ago
:lgtm:

Reviewed 1 of 1 files at r1. Review status: all files reviewed at latest revision, all discussions resolved.


Comments from Reviewable