Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

mtest06_3 do not using custom tmp ltp folder #72

Closed
olerem opened this issue Jun 23, 2016 · 2 comments
Closed

mtest06_3 do not using custom tmp ltp folder #72

olerem opened this issue Jun 23, 2016 · 2 comments

Comments

@olerem
Copy link

olerem commented Jun 23, 2016

mtest06_3 failed with this log:

2016-06-23T10:51:53 Test: mtest06_3 Testparm: mmap3 -x 0.002 -p
mkfile(): mkstemp(): Read-only file system
main(): mkfile(): Failed to create temp file.
mkfile(): mkstemp(): Read-only file system

I run tests on readonly remote machine, for more info see:
#57

looks like mmap3 is ignoring LTPTMP env variable.
@metan-ucw
Copy link
Member

It looks like the test wasn't really ported to LTP, it does not use any of the LTP API at all.

It should be fixed to at least use the LTP temp directory and ideally it should use LTP result reporting as well. I will look into this.

@olerem
Copy link
Author

olerem commented Jun 23, 2016

Thank you :)

@olerem olerem mentioned this issue Jul 7, 2016
Closed
metan-ucw added a commit that referenced this issue Aug 4, 2016
@metan-ucw
mtest06/mmap3: Rewrite to use new library.
68aaa8e 
* Make use of SAFE_MACROS()

* Do not call printf() and exit() in signal handler

* Make use of test temporary directory (fixes #72)

* Properly clean up after the test

Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@metan-ucw @olerem