Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752904AbbFCDzq (ORCPT ); Tue, 2 Jun 2015 23:55:46 -0400 Received: from mail-pd0-f173.google.com ([209.85.192.173]:34751 "EHLO mail-pd0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751673AbbFCDzp (ORCPT ); Tue, 2 Jun 2015 23:55:45 -0400 Date: Wed, 3 Jun 2015 12:56:08 +0900 From: Sergey Senozhatsky To: Joe Perches Cc: Dan Streetman , Andrew Morton , Linux-MM , linux-kernel , Minchan Kim , Nitin Gupta , Sergey Senozhatsky Subject: Re: [PATCH] MAINTAINERS: add zpool Message-ID: <20150603035608.GA1652@swordfish> References: <1433264166-31452-1-git-send-email-ddstreet@ieee.org> <1433279395.4861.100.camel@perches.com> <1433280616.4861.102.camel@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1433280616.4861.102.camel@perches.com> 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 Content-Length: 1089 Lines: 34 On (06/02/15 14:30), Joe Perches wrote: > > >> +ZPOOL COMPRESSED PAGE STORAGE API > > >> +M: Dan Streetman > > >> +L: linux-mm@kvack.org > > >> +S: Maintained > > >> +F: mm/zpool.c > > >> +F: include/linux/zpool.h > > > > > > If zpool.h is only included from files in mm/, > > > maybe zpool.h should be moved to mm/ ? > > > > It *could* be included by others, e.g. drivers/block/zram. > > > > It currently is only used by zswap though, so yeah it could be moved > > to mm/. Should I move it there, until (if ever) anyone outside of mm/ > > wants to use it? > > Up to you. > > I think include/linux is a bit overstuffed and > whatever can be include local should be. > Hi, I agree, can be local for now. if zram will ever want to use zpool then we will move zpool.h to include/linux. just my 5 cents. -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/