Skip to content
This repository has been archived by the owner on Jan 6, 2025. It is now read-only.

Syscall format test fix #78

Merged
merged 2 commits into from
Apr 9, 2018
Merged

Syscall format test fix #78

merged 2 commits into from
Apr 9, 2018

Conversation

GBuella
Copy link
Contributor

@GBuella GBuella commented Apr 7, 2018

Because of #76

Whether the kernel support mlock2 or not, whether libc gives a constant SYS_mlock2 or not, has no effect on being able to intercept syscalls.
The syscall logging is an "extra feature" anyways, shouldn't stop people from using syscall_intercept.


This change is Reviewable

GBuella added 2 commits April 8, 2018 00:16
Make some lines in the match file optional, those
which were made optional in bb10939
@codecov-io
Copy link

Codecov Report

Merging #78 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #78   +/-   ##
=======================================
  Coverage   91.95%   91.95%           
=======================================
  Files          11       11           
  Lines         957      957           
=======================================
  Hits          880      880           
  Misses         77       77
Flag Coverage Δ
#pmemfile_tests 86.35% <ø> (ø) ⬆️
#regular_tests 90.8% <ø> (ø) ⬆️

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 ef0cbce...e85b98a. Read the comment docs.

@krzycz
Copy link
Contributor

krzycz commented Apr 9, 2018

:lgtm:


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


Comments from Reviewable

@marcinslusarz
Copy link
Contributor

:lgtm:

Something similar should be done for O_TMPFILE.


Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.


Comments from Reviewable

@GBuella GBuella merged commit a744b72 into pmem:master Apr 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants