Francesco Romani has submitted this change and it was merged. Change subject: Limit sleep_millisecs to minimum 10 ......................................................................
Limit sleep_millisecs to minimum 10 According to the former KSM controller, ksmtuned, 10 should be the minimum value for sleep_millisecs: 48 sleep=$[KSM_SLEEP_MSEC * 16 * 1024 * 1024 / total] 49 [ $sleep -le 10 ] && sleep=10 <---- 10 is the minimum 50 debug sleep $sleep Systems with a low sleep_millisecs number will run ksmd daemon without interruption, causing a very high cpu time consumption. This patch limits sleep_millisecs to the minimum of 10. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1261507 Change-Id: I7412ca576089f1dae2441ec39f72e7a94b3c8fc9 Signed-off-by: Amador Pahim <[email protected]> Reviewed-on: https://gerrit.ovirt.org/45332 Continuous-Integration: Jenkins CI Reviewed-by: Martin Sivák <[email protected]> Reviewed-by: Douglas Schilling Landgraf <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> (cherry picked from commit 1cd2b7b36e550afa56e8773e68153665d3bbd2a0) Reviewed-on: https://gerrit.ovirt.org/46006 Tested-by: Martin Sivák <[email protected]> Reviewed-by: Francesco Romani <[email protected]> Continuous-Integration: Francesco Romani <[email protected]> --- M vdsm/mom.d/03-ksm.policy 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Martin Sivák: Verified Francesco Romani: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46006 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7412ca576089f1dae2441ec39f72e7a94b3c8fc9 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Martin Sivák <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Amador Pahim <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Martin Sivák <[email protected]> Gerrit-Reviewer: Piotr Kliczewski <[email protected]> Gerrit-Reviewer: [email protected] _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
