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

io_uring support #3419

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

io_uring support #3419

wants to merge 3 commits into from

Conversation

SRIKKANTH
Copy link
Collaborator

@SRIKKANTH SRIKKANTH commented Sep 13, 2024

Added 'io_uring' support to LISA io perf tests and a new testcase 'perf_nvme_io_uring'.
io_uring is advanced alternative for libaio with low system overhead and more io depth.

lisa/tools/fio.py Outdated Show resolved Hide resolved
microsoft/testsuites/performance/common.py Outdated Show resolved Hide resolved
microsoft/testsuites/performance/nvmeperf.py Outdated Show resolved Hide resolved
microsoft/testsuites/performance/nvmeperf.py Outdated Show resolved Hide resolved
@SRIKKANTH SRIKKANTH marked this pull request as draft September 13, 2024 17:46
@SRIKKANTH SRIKKANTH force-pushed the smyakam/fio_io_uring_03_09_2024 branch 2 times, most recently from 847d358 to d401259 Compare September 13, 2024 21:37
@SRIKKANTH SRIKKANTH marked this pull request as ready for review September 13, 2024 21:49
@SRIKKANTH SRIKKANTH force-pushed the smyakam/fio_io_uring_03_09_2024 branch from d9fefcf to 2747a2b Compare September 13, 2024 22:28
@SRIKKANTH SRIKKANTH force-pushed the smyakam/fio_io_uring_03_09_2024 branch from 71564e3 to 81ae9ec Compare September 14, 2024 06:08
Added 'io_uring' support to LISA io perf tests and new testcase 'perf_nvme_io_uring'
@SRIKKANTH SRIKKANTH force-pushed the smyakam/fio_io_uring_03_09_2024 branch from 500db48 to 09959a5 Compare September 16, 2024 17:16
@@ -37,6 +38,25 @@ class NvmePerformace(TestSuite):
),
)
def perf_nvme(self, node: Node, result: TestResult) -> None:
self._perf_nvme(node, IoEngine.LIBAIO, result)
Copy link
Member

Choose a reason for hiding this comment

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

Update test case comments to specify what io engine is used.

requirement=simple_requirement(
supported_features=[Nvme],
),
)
Copy link
Collaborator

Choose a reason for hiding this comment

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

for perf_nvme, it has requirement for count of nvme disks, for perf_nvme_io_uring, by default it uses 1 nvme disk, is it expected?

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.

3 participants