From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756713Ab1FUNdf (ORCPT ); Tue, 21 Jun 2011 09:33:35 -0400 Received: from mx1.redhat.com ([209.132.183.28]:20061 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756560Ab1FUNdd (ORCPT ); Tue, 21 Jun 2011 09:33:33 -0400 Date: Tue, 21 Jun 2011 15:32:36 +0200 From: Andrea Arcangeli To: Amerigo Wang Cc: linux-kernel@vger.kernel.org, akpm@linux-foundation.org, Hugh Dickins , Rik van Riel , Mel Gorman , linux-mm@kvack.org Subject: Re: [PATCH 4/4] mm: introduce no_ksm to disable totally KSM Message-ID: <20110621133236.GP20843@redhat.com> References: <1308643849-3325-1-git-send-email-amwang@redhat.com> <1308643849-3325-4-git-send-email-amwang@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1308643849-3325-4-git-send-email-amwang@redhat.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jun 21, 2011 at 04:10:45PM +0800, Amerigo Wang wrote: > Introduce a new kernel parameter "no_ksm" to totally disable KSM. Here as well this is the wrong approach. If you want to save memory, you should make ksmd quit when run=0 and start only when setting ksm/run=1. And move the daemon hashes and slabs initializations to the ksmd daemon start. Not registering in sysfs and crippling down the feature despite you loaded the proper .text into memory isn't good. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail144.messagelabs.com (mail144.messagelabs.com [216.82.254.51]) by kanga.kvack.org (Postfix) with SMTP id 9EA4F90013A for ; Tue, 21 Jun 2011 09:33:03 -0400 (EDT) Date: Tue, 21 Jun 2011 15:32:36 +0200 From: Andrea Arcangeli Subject: Re: [PATCH 4/4] mm: introduce no_ksm to disable totally KSM Message-ID: <20110621133236.GP20843@redhat.com> References: <1308643849-3325-1-git-send-email-amwang@redhat.com> <1308643849-3325-4-git-send-email-amwang@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1308643849-3325-4-git-send-email-amwang@redhat.com> Sender: owner-linux-mm@kvack.org List-ID: To: Amerigo Wang Cc: linux-kernel@vger.kernel.org, akpm@linux-foundation.org, Hugh Dickins , Rik van Riel , Mel Gorman , linux-mm@kvack.org On Tue, Jun 21, 2011 at 04:10:45PM +0800, Amerigo Wang wrote: > Introduce a new kernel parameter "no_ksm" to totally disable KSM. Here as well this is the wrong approach. If you want to save memory, you should make ksmd quit when run=0 and start only when setting ksm/run=1. And move the daemon hashes and slabs initializations to the ksmd daemon start. Not registering in sysfs and crippling down the feature despite you loaded the proper .text into memory isn't good. -- 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/ . Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/ Don't email: email@kvack.org