Don't merge LTP: Fixed get_robust_list01 testcase

lsds / ltp

Linux Test Project http://linux-test-project.github.io/

https://lists.linux.it/listinfo/ltp

GNU General Public License v2.0

0 stars 4 forks source link

Closed vamsikrishna935 closed 4 years ago

vamsikrishna935 commented 4 years ago

Patch Description: One of sub tests (EFAULT) is failing because of https://github.com/lsds/sgx-lkl/issues/297 Issue 297: [Tests] lkl_access_ok() should return -1 on invalid access
Work around: Commented the test related 297 issue and added a todo comment

davidchisnall commented 4 years ago

Please can you hold off on this one? @KenGordon has a fix for lsds/sgx-lkl#297 that should eliminate the need for this.

SeanTAllen commented 4 years ago

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.

hukoyu commented 4 years ago

After https://github.com/lsds/sgx-lkl/pull/727 is merged, we will not need this patch.

hukoyu commented 4 years ago

Submitted PR https://github.com/lsds/sgx-lkl/pull/765 and this test is passing as is. This patch not needed. Closing.