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

casadm: use alru max dirty ratio #1376

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

live4thee
Copy link
Contributor

This patch will enable the 'max_dirty_ratio' parameter from Open-CAS/ocf#757

With patched casadm:

$ casadm -G -n cleaning-alru -i 1
+===============================+=======+
| Parameter name                | Value |
+===============================+=======+
| Wake up time [s]              |    10 |
| Stale buffer time [s]         |   120 |
| Flush max buffers             |   256 |
| Activity threshold [ms]       |  1000 |
| Maximum dirty ratio [percent] |    75 |
+===============================+=======+

$ casadm -X -n cleaning-alru -i 1 -d 80
$ casadm -G -n cleaning-alru -i 1
+===============================+=======+
| Parameter name                | Value |
+===============================+=======+
| Wake up time [s]              |    10 |
| Stale buffer time [s]         |   120 |
| Flush max buffers             |   256 |
| Activity threshold [ms]       |  1000 |
| Maximum dirty ratio [percent] |    80 |
+===============================+=======+

The odd is, if upgrading from existing installation - the first '-G -n cleaning-alru' will report an ioctl failure. It will be OK after issuing an -X -n cleaning-alru -i 1 -d something. I guess that's due to the missing 'max_dirty_ratio' in existing metadata. If any examples that handle such issue, please guide me.

Thanks in advance.

live4thee and others added 2 commits October 9, 2022 23:07
- introduce alru: `max_dirty_ratio'

Signed-off-by: David Lee <[email protected]>
@live4thee live4thee force-pushed the casadm/use-alru-max-dirty-ratio branch from 47473cf to d335cc5 Compare October 9, 2022 15:08
@CAS-Linux-Jenkins
Copy link

Can one of the admins verify this patch?

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.

2 participants