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
Add missing module_param for zfs_per_txg_dirty_frees_percent #5750
Conversation
@dpquigl, thanks for your PR! By analyzing the history of the files in this pull request, we identified @behlendorf, @ahrens and @FransUrbo to be potential reviewers. |
man/man5/zfs-module-parameters.5
Outdated
@@ -1441,6 +1441,22 @@ Default value: \fB52,428,800\fR. | |||
.sp | |||
.ne 2 | |||
.na | |||
\fBzfs_per_txg_dirty_frees_percent \fR (int) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean ulong
here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. I'll fix.
module_param(zfs_mdcomp_disable, int, 0644); | ||
MODULE_PARM_DESC(zfs_mdcomp_disable, "Disable meta data compression"); | ||
|
||
module_param(zfs_nopwrite_enabled, int, 0644); | ||
MODULE_PARM_DESC(zfs_nopwrite_enabled, "Enable NOP writes"); | ||
|
||
module_param(zfs_per_txg_dirty_frees_percent, ulong, 0644); | ||
MODULE_PARM_DESC(zfs_per_txg_dirty_frees_percent, | ||
"percentage of dirtied blocks from frees in one TXG"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing /* END CSTYLED */
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do
When the code was added this tunable was not exposed via module params. Also it was not documented. This patch changes the type from a uint32 to a ulong as done with other percentage tunables and also documents it in the zfs-module-parameters man page.
Changes made and new branch pushed out. |
When the code was added this tunable was not exposed via module params. Also it was not documented. This patch changes the type from a uint32 to a ulong as done with other percentage tunables and also documents it in the zfs-module-parameters man page. Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: David Quigley <david.quigley@intel.com> Closes openzfs#5750
When the code was added this tunable was not exposed via module params. Also it was not documented. This patch changes the type from a uint32 to a ulong as done with other percentage tunables and also documents it in the zfs-module-parameters man page. Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov> Signed-off-by: David Quigley <david.quigley@intel.com> Closes openzfs#5750
When the code was added this tunable was not exposed via module params. Also it
was not documented. This patch changes the type from a uint32 to a ulong as
done with other percentage tunables and also documents it in the
zfs-module-parameters man page.