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

tests: add ioworker test #132

Merged
merged 2 commits into from
Sep 1, 2022
Merged

tests: add ioworker test #132

merged 2 commits into from
Sep 1, 2022

Conversation

karlowich
Copy link
Contributor

@karlowich karlowich commented Jul 20, 2022

Add ioworker read write test to relevant backends.

This is in preparation for the ramdisk enhancements, where the ioworker test is likely to be expanded.

@karlowich karlowich added the pr-ready-for-test PR is ready for functional tests label Jul 20, 2022
@karlowich karlowich force-pushed the tests/ioworker branch 5 times, most recently from 104c9df to 3f80d9e Compare July 28, 2022 13:44
@karlowich karlowich mentioned this pull request Jul 28, 2022
1 task
Copy link
Member

@safl safl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good. I little clean/squash of the history would be nice, but really nice to see the expansion of the ioworker and testcases using it. Great!

Signed-off-by: Karl Bonde Torp <k.torp@samsung.com>
- Rewrite `ioworker` to support running sync with command `ioworker verify-sync`
- Add testcases for this

This is in preparation for supporting iovecs in the ramdisk backend

Signed-off-by: Karl Bonde Torp <k.torp@samsung.com>
@safl safl merged commit 2ef4099 into xnvme:next Sep 1, 2022
@safl safl mentioned this pull request Sep 1, 2022
27 tasks
@karlowich karlowich deleted the tests/ioworker branch November 29, 2023 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-ready-for-test PR is ready for functional tests
Development

Successfully merging this pull request may close these issues.

2 participants