From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail143.messagelabs.com (mail143.messagelabs.com [216.82.254.35]) by kanga.kvack.org (Postfix) with SMTP id AD6C16B0055 for ; Mon, 20 Jul 2009 07:44:19 -0400 (EDT) Message-ID: <4A645974.3020801@redhat.com> Date: Mon, 20 Jul 2009 14:48:04 +0300 From: Izik Eidus MIME-Version: 1.0 Subject: Re: [PATCH 10/10] ksm: change ksm nice level to be 5 References: <1247851850-4298-2-git-send-email-ieidus@redhat.com> <1247851850-4298-3-git-send-email-ieidus@redhat.com> <1247851850-4298-4-git-send-email-ieidus@redhat.com> <1247851850-4298-5-git-send-email-ieidus@redhat.com> <1247851850-4298-6-git-send-email-ieidus@redhat.com> <1247851850-4298-7-git-send-email-ieidus@redhat.com> <1247851850-4298-8-git-send-email-ieidus@redhat.com> <1247851850-4298-9-git-send-email-ieidus@redhat.com> <1247851850-4298-10-git-send-email-ieidus@redhat.com> <1247851850-4298-11-git-send-email-ieidus@redhat.com> <20090720045037.GA24157@balbir.in.ibm.com> In-Reply-To: <20090720045037.GA24157@balbir.in.ibm.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: owner-linux-mm@kvack.org To: balbir@linux.vnet.ibm.com Cc: akpm@linux-foundation.org, hugh.dickins@tiscali.co.uk, aarcange@redhat.com, chrisw@redhat.com, avi@redhat.com, riel@redhat.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, nickpiggin@yahoo.com.au List-ID: Balbir Singh wrote: > * Izik Eidus [2009-07-17 20:30:50]: > > >> From: Izik Eidus >> >> ksm should try not to disturb other tasks as much as possible. >> >> Signed-off-by: Izik Eidus >> --- >> mm/ksm.c | 2 +- >> 1 files changed, 1 insertions(+), 1 deletions(-) >> >> diff --git a/mm/ksm.c b/mm/ksm.c >> index 75d7802..4afe345 100644 >> --- a/mm/ksm.c >> +++ b/mm/ksm.c >> @@ -1270,7 +1270,7 @@ static void ksm_do_scan(unsigned int scan_npages) >> >> static int ksm_scan_thread(void *nothing) >> { >> - set_user_nice(current, 0); >> + set_user_nice(current, 5); >> > > Is the 5 arbitrary? Why not +19? What is the intention of this change > - to run when no other task is ready to run? > Hey Balbir, I thought about giving it the lowest priority of nice before I did this patch, but then I came into understanding that it isn't right, Although ksm should not distrub other tasks while they are running, it does need to run while they are running, most of the use cases for ksm is to find identical pages in real time while they are changing in the application, so giving it the lowest priority doesn't seems right to me, But my understanding of how the nice prioritys are working is just my intuition, so if you know better and think that for the use case i described above other nice priority is better fit, tell me and I wont have any problem to change. Thanks. > >> while (!kthread_should_stop()) { >> if (ksm_run & KSM_RUN_MERGE) { >> > > -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@kvack.org. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: email@kvack.org