Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754744AbbHFK7M (ORCPT ); Thu, 6 Aug 2015 06:59:12 -0400 Received: from mail-pa0-f52.google.com ([209.85.220.52]:34544 "EHLO mail-pa0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753059AbbHFK7L (ORCPT ); Thu, 6 Aug 2015 06:59:11 -0400 Date: Thu, 6 Aug 2015 19:59:45 +0900 From: Sergey Senozhatsky To: Dan Streetman Cc: Sergey Senozhatsky , Seth Jennings , Andrew Morton , Linux-MM , linux-kernel Subject: Re: [PATCH 3/3] zswap: change zpool/compressor at runtime Message-ID: <20150806105945.GA566@swordfish> References: <1438782403-29496-1-git-send-email-ddstreet@ieee.org> <1438782403-29496-4-git-send-email-ddstreet@ieee.org> <20150806000843.GA3927@swordfish> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23+102 (2ca89bed6448) (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3300 Lines: 66 On (08/06/15 06:20), Dan Streetman wrote: > > On (08/05/15 09:46), Dan Streetman wrote: > >> Update the zpool and compressor parameters to be changeable at runtime. > >> When changed, a new pool is created with the requested zpool/compressor, > >> and added as the current pool at the front of the pool list. Previous > >> pools remain in the list only to remove existing compressed pages from. > >> The old pool(s) are removed once they become empty. > >> > > > > Sorry, just curious, is this functionality/complication really > > necessary? > > Well you could ask the same question about many other module params; > can't people just configure everything using boot params? ;-) > > > How often do you expect people to do that? The way I > > see it -- a static configuration works just fine: boot, test, > > re-configure, boot test; compare the results and done. > > Sure a static configuration will work (it has since Seth wrote zswap), > but that doesn't guarantee everyone will want to do it that way. > Certainly for testing/development/benchmarking avoiding a reboot is > helpful. And for long-running and/or critical systems that need to > change their zpool or compressor, for whatever reason, forcing a > reboot isn't desirable. Sorry, I didn't have time to read the patches carefully/attentively (will do); so my email may be a complete nonsense. > Why would someone want to change their compressor or zpool? A simple > exampe comes to mind - maybe they have 1000's of systems and a bug was > found in the current level of compressor or zpool - they would then > have to either reboot all the systems to change to a different > zpool/compressor, or leave it using the known-buggy one. Well, if that buggy compressor is being used by other modules then rebooting is sort of inevitable. But you still preserve pages compressed with the old compressor and let user access them, right? Thus read operation possibly will hit the bug regardless of current 'front' pool. > In addition, a static boot-time configuration requires adding params > to the bootloader configuration, *and* rebuilding the initramfs to > include both the required zpool and compressor. So even for static > configurations, it's simpler to be able to set the zpool and > compressor immediately after boot, instead of at boot time. I mean, it just feels that this is a way too big change for no particular use case (no offense). It doesn't take much time to figure out (a simple google request does the trick here) which one of the available compressors gives best ratio in general or which one has better read/write (compress/decompress) speeds. A buggy compressor is a good use case, I agree (with the exception that reboot is still very much possible). But if someone changes compressing backend because he or she estimates a better compression ratio or performance then there will be no immediate benefit -- pages compressed with the old compressor are still there and it will take some unpredictable amount of time to drain old pools and to remove them. -ss -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/