From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from psmtp.com (na3sys010amx163.postini.com [74.125.245.163]) by kanga.kvack.org (Postfix) with SMTP id 0A1A26B0036 for ; Thu, 11 Apr 2013 16:05:20 -0400 (EDT) Received: by mail-pb0-f42.google.com with SMTP id up7so1034482pbc.1 for ; Thu, 11 Apr 2013 13:05:20 -0700 (PDT) Date: Thu, 11 Apr 2013 13:05:18 -0700 From: Greg Kroah-Hartman Subject: Re: [PATCH 02/10] staging: zcache: remove zcache_freeze Message-ID: <20130411200518.GA5268@kroah.com> References: <1365553560-32258-1-git-send-email-liwanp@linux.vnet.ibm.com> <1365553560-32258-3-git-send-email-liwanp@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: owner-linux-mm@kvack.org List-ID: To: Dan Magenheimer Cc: Wanpeng Li , Seth Jennings , Konrad Rzeszutek Wilk , Minchan Kim , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Andrew Morton , Bob Liu On Thu, Apr 11, 2013 at 10:13:42AM -0700, Dan Magenheimer wrote: > > From: Wanpeng Li [mailto:liwanp@linux.vnet.ibm.com] > > Subject: [PATCH 02/10] staging: zcache: remove zcache_freeze > > > > The default value of zcache_freeze is false and it won't be modified by > > other codes. Remove zcache_freeze since no routine can disable zcache > > during system running. > > > > Signed-off-by: Wanpeng Li > > I'd prefer to leave this code in place as it may be very useful > if/when zcache becomes more tightly integrated into the MM subsystem > and the rest of the kernel. And the subtleties for temporarily disabling > zcache (which is what zcache_freeze does) are non-obvious and > may cause data loss so if someone wants to add this functionality > back in later and don't have this piece of code, it may take > a lot of pain to get it working. > > Usage example: All CPUs are fully saturated so it is questionable > whether spending CPU cycles for compression is wise. Kernel > could disable zcache using zcache_freeze. (Yes, a new entry point > would need to be added to enable/disable zcache_freeze.) > > My two cents... others are welcome to override. I will not override, and did not take this patch. greg k-h -- 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