SplitFS icon indicating copy to clipboard operation
SplitFS copied to clipboard

Implement SplitFS using system-call interception rather than library calls

Open vijay03 opened this issue 5 years ago • 8 comments

Use the system-call intercept library: https://github.com/pmem/syscall_intercept in SplitFS instead of intercepting glibc calls. This will make SplitFS more portable and hopefully more robust (different glibc variants will have the same system call implementation).

vijay03 avatar Jul 31 '20 03:07 vijay03

Is it possible for me to work on this?

ishank-arora avatar Jul 31 '20 14:07 ishank-arora

@vijay03 @rohankadekodi I have been working on this. Current status: POSIX (pjd) test suite integrated and passing successfully in POSIX mode SplitFS.

OmSaran avatar Jan 20 '21 17:01 OmSaran

Thats great! Can we get a pull request?

vijay03 avatar Jan 20 '21 17:01 vijay03

@vijay03 I have created a draft pull request. #47 Draft because:

  1. There are some unwanted files (Quill related) that can be removed (In progress)
  2. I wanted to integrate Sync and Strict mode also in this PR itself.

Looking to get above 2 done by 25th Jan.

I was thinking of converting to full pull request once above 2 are complete. Would that be ok?

OmSaran avatar Jan 20 '21 18:01 OmSaran

That would be great! Let us know when @rohankadekodi and I can review it.

vijay03 avatar Jan 20 '21 18:01 vijay03

Thanks a lot Om! This is great news. Yes, please let us know once the other two modes are passing on the PJD test suite and we can review it.

On Thu, 21 Jan 2021 at 00:08, Vijay Chidambaram [email protected] wrote:

That would be great! Let us know when @rohankadekodi https://github.com/rohankadekodi and I can review it.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/utsaslab/SplitFS/issues/32#issuecomment-763850773, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVQOWYA4BASLOKK7737OY3S24PKPANCNFSM4PPH4QGA .

rohankadekodi avatar Jan 21 '21 03:01 rohankadekodi

@vijay03 @rohankadekodi sync and strict integration is also complete and ready for review. I have included some details of the PR in the PR description. #47

OmSaran avatar Jan 24 '21 16:01 OmSaran

Thanks Om! I will review and test the code and get back to you.

On Sun, Jan 24, 2021, 10:29 PM OmSaran [email protected] wrote:

@vijay03 https://github.com/vijay03 @rohankadekodi https://github.com/rohankadekodi sync and strict integration is also complete and ready for review. I have included some details of the PR in the PR description. #47 https://github.com/utsaslab/SplitFS/pull/47

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/utsaslab/SplitFS/issues/32#issuecomment-766394256, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABVQOW6HYBUFONPXRXB4SDLS3RGYFANCNFSM4PPH4QGA .

rohankadekodi avatar Jan 25 '21 01:01 rohankadekodi