From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757293AbbGGPCK (ORCPT ); Tue, 7 Jul 2015 11:02:10 -0400 Received: from mail-pd0-f180.google.com ([209.85.192.180]:36859 "EHLO mail-pd0-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757449AbbGGPBv (ORCPT ); Tue, 7 Jul 2015 11:01:51 -0400 Date: Wed, 8 Jul 2015 00:01:43 +0900 From: Minchan Kim To: Sergey Senozhatsky Cc: Andrew Morton , linux-mm@kvack.org, linux-kernel@vger.kernel.org, Sergey Senozhatsky Subject: Re: [PATCH v6 7/7] zsmalloc: use shrinker to trigger auto-compaction Message-ID: <20150707150143.GC23003@blaptop> References: <1436270221-17844-1-git-send-email-sergey.senozhatsky@gmail.com> <1436270221-17844-8-git-send-email-sergey.senozhatsky@gmail.com> <20150707134445.GD3898@blaptop> <20150707144107.GC1450@swordfish> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150707144107.GC1450@swordfish> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Jul 07, 2015 at 11:41:07PM +0900, Sergey Senozhatsky wrote: > On (07/07/15 22:44), Minchan Kim wrote: > [..] > > IMO, there is no value to maintain just in case of > > failing register_shrinker in practice. > > > > Let's remove shrinker_enabled and abort pool creation if shrinker register > > is failed. > > But why would we do this? Yes, it's kinda-sorta bad -- we were not > able to register zspool shrinker, so there will be no automatic > compaction... And that's it. > > It does not affect zsmalloc/zram functionality by any means. Including > compaction itself -- user still has a way to compact zspool (manually). > And in some scenarios user will never even see automatic compaction in > action (assuming that there is a plenty of RAM available). > > Can you explain your decision? I don't think it would fail in *real practice*. Althout it might happen, what does zram could help in that cases? If it were failed, it means there is already little memory on the system so zram could not be helpful for those environment. IOW, zram should be enabled earlier. If you want it strongly, please reproduce such failing and prove that zram was helpful for the system. on that situation. > > -ss -- Kind regards, Minchan Kim