From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752768AbcFBAOc (ORCPT ); Wed, 1 Jun 2016 20:14:32 -0400 Received: from LGEAMRELO13.lge.com ([156.147.23.53]:56643 "EHLO lgeamrelo13.lge.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752005AbcFBAO1 (ORCPT ); Wed, 1 Jun 2016 20:14:27 -0400 X-Original-SENDERIP: 156.147.1.126 X-Original-MAILFROM: minchan@kernel.org X-Original-SENDERIP: 165.244.98.204 X-Original-MAILFROM: minchan@kernel.org X-Original-SENDERIP: 10.177.223.161 X-Original-MAILFROM: minchan@kernel.org Date: Thu, 2 Jun 2016 09:15:10 +0900 From: Minchan Kim To: Andrew Morton CC: , , Sergey Senozhatsky Subject: Re: [PATCH v7 11/12] zsmalloc: page migration support Message-ID: <20160602001510.GA1736@bbox> References: <1464736881-24886-1-git-send-email-minchan@kernel.org> <1464736881-24886-12-git-send-email-minchan@kernel.org> <20160601143936.a7ad8eec093514e3ee54cc62@linux-foundation.org> MIME-Version: 1.0 In-Reply-To: <20160601143936.a7ad8eec093514e3ee54cc62@linux-foundation.org> User-Agent: Mutt/1.5.21 (2010-09-15) X-MIMETrack: Itemize by SMTP Server on LGEKRMHUB06/LGE/LG Group(Release 8.5.3FP6|November 21, 2013) at 2016/06/02 09:14:24, Serialize by Router on LGEKRMHUB06/LGE/LG Group(Release 8.5.3FP6|November 21, 2013) at 2016/06/02 09:14:24, Serialize complete at 2016/06/02 09:14:24 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jun 01, 2016 at 02:39:36PM -0700, Andrew Morton wrote: > On Wed, 1 Jun 2016 08:21:20 +0900 Minchan Kim wrote: > > > This patch introduces run-time migration feature for zspage. > > > > ... > > > > +static void kick_deferred_free(struct zs_pool *pool) > > +{ > > + schedule_work(&pool->free_work); > > +} > > When CONFIG_ZSMALLOC=m, what keeps all the data structures in place > during a concurrent rmmod? > The most of data structure in zram start to work by user calling zs_create_pool and user of zsmalloc should call zs_destroy_pool before trying doing rmmod where zs_unregister_migration does flush_work(&pool->free_work). If I miss something, please let me know it.