Change in vdsm[ovirt-3.6]: Limit sleep_millisecs to minimum 10

fromani at redhat.com fromani at redhat.com
Thu Oct 29 10:27:41 UTC 2015


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: http://bugzilla.redhat.com/1256949
Change-Id: I7412ca576089f1dae2441ec39f72e7a94b3c8fc9
Signed-off-by: Amador Pahim <apahim at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/45332
Continuous-Integration: Jenkins CI
Reviewed-by: Martin Sivák <msivak at redhat.com>
Reviewed-by: Douglas Schilling Landgraf <dougsland at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
(cherry picked from commit 1cd2b7b36e550afa56e8773e68153665d3bbd2a0)
Reviewed-on: https://gerrit.ovirt.org/47835
Tested-by: Martin Sivák <msivak at redhat.com>
Reviewed-by: Francesco Romani <fromani at redhat.com>
---
M vdsm/mom.d/03-ksm.policy
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Martin Sivák: Verified
  Jenkins CI: Passed CI tests
  Francesco Romani: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/47835
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7412ca576089f1dae2441ec39f72e7a94b3c8fc9
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.6
Gerrit-Owner: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list