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.

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

LTP:fixed fallocate01 tests #45

Merged
merged 6 commits into from
Aug 11, 2020
Merged

LTP:fixed fallocate01 tests #45

merged 6 commits into from
Aug 11, 2020

Conversation

vamsikrishna935
Copy link

  • Patch Description:
    Test Failure reason in SGX-LKL:
    [[ SGX-LKL ]] libc_start_main_stage2(): Calling app main: /ltp/testcases/kernel/syscalls/fallocate/fallocate01
    fallocate01 1 TCONF : fallocate01.c:233: fallocate system call is not implemented
    Test is happening on temporary file created under /tmp, on which fallocate system call is failing.
  • Workaround to fix the issue:
    fallocate system call is failing on temporary file created under /tmp directory.
    So modified the tests to create directory in root filesystem.

@davidchisnall
Copy link

This need some root cause analysis. Why does fallocate not work in the tmp directory?

@hukoyu
Copy link
Collaborator

hukoyu commented Aug 5, 2020

@davidchisnall , @SeanTAllen submitted a fix for /tmp file create issue: lsds/sgx-lkl#723
I also beleive this change will not be needed.

@SeanTAllen
Copy link

This needs a corresponding PR on sgx-lkl so that CI is run with this change. Please speak to @hukoyu if you aren't familiar with how to do that.

@vamsikrishna935
Copy link
Author

vamsikrishna935 commented Aug 6, 2020

@hukoyu : In the latest code changes the test is failing on /tmp directory after pr lsds/sgx-lkl#723 merged
So logged an issue and updated the pr. Issue: lsds/sgx-lkl#734

@hukoyu hukoyu requested review from prp and letmaik August 6, 2020 20:36
@vamsikrishna935
Copy link
Author

@SeanTAllen Addressed comments. Could you please review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants