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

SSV-24185: Fix the number of zvol I/O threads to align with 50% of the maximum available logical CPUs. #98

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

arun-kv
Copy link

@arun-kv arun-kv commented Oct 30, 2024

SSV-24185

Issue: CPU stuck at 100% is related to ZFS when we use dedup & comp, never able to reproduce the issue with Compression only enable.

Fix: The number of zvol I/O threads to align with 50% of the maximum available logical CPUs if the 'zvol_io_threads' value in 'Computer\HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\Services\ZFSin' is not specified. If the 'zvol_io_threads' value is provided, use this value for creating the zvol task queue thread pool.

Use kstat.exe zfs:0:tunable to check the current value of zvol_io_threads. This can be set only by updating the registry, and also require a reboot.

If zvol_io_threads set to '0', then the default zvol io threads will
be 50% of number of  logical processors.
@arun-kv arun-kv changed the title SSV-24185: Fix the number of zvol I/O threads to align with the maximum available CPUs. SSV-24185: Fix the number of zvol I/O threads to align with 50% of the maximum available logical CPUs. Oct 30, 2024
@arun-kv arun-kv requested a review from hrudaya21 October 30, 2024 14:28
Copy link

@datacore-rbassett datacore-rbassett left a comment

Choose a reason for hiding this comment

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

Approved. I assume we will address the failed checks; e.g. "zfs-tests-sanity / tests (push)" and others.

@arun-kv
Copy link
Author

arun-kv commented Nov 4, 2024

Approved. I assume we will address the failed checks; e.g. "zfs-tests-sanity / tests (push)" and others.

These tests are not executed for our commits. We have our own test running in a Jenkin portal Here is the link http://10.200.2.48:8080/job/OpenZFS-Test-Suite/118/

@arun-kv arun-kv merged commit ae82c38 into rel-10psp18 Nov 4, 2024
0 of 8 checks passed
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