From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933299AbcIFIWZ (ORCPT ); Tue, 6 Sep 2016 04:22:25 -0400 Received: from rhlx01.hs-esslingen.de ([129.143.116.10]:35722 "EHLO rhlx01.hs-esslingen.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932461AbcIFIWW (ORCPT ); Tue, 6 Sep 2016 04:22:22 -0400 Date: Tue, 6 Sep 2016 10:22:20 +0200 From: Andreas Mohr To: Minchan Kim Cc: Andrew Morton , linux-kernel@vger.kernel.org, Sergey Senozhatsky Subject: Re: [RFC] zram: support page-based parallel write Message-ID: <20160906082220.GA2751@rhlx01.hs-esslingen.de> References: <1473146657-4402-1-git-send-email-minchan@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1473146657-4402-1-git-send-email-minchan@kernel.org> X-Priority: none User-Agent: Mutt/1.7.0 (2016-08-17) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Sep 06, 2016 at 04:24:17PM +0900, Minchan Kim wrote: > @@ -1464,6 +1908,9 @@ static int __init zram_init(void) > num_devices--; > } > > + if (create_workers()) > + goto out_error; > + > return 0; > > out_error: > @@ -1474,6 +1921,7 @@ static int __init zram_init(void) > static void __exit zram_exit(void) > { > destroy_devices(); > + destroy_workers(); > } Asymmetry --> "BUG". ...right? (I have to admit that current implementation structure is not easy to follow, thus I'm not fully sure) Thanks for working in this important area! HTH, Andreas Mohr