Messages in this thread | | | Date | Wed, 3 Jun 2015 12:56:08 +0900 | From | Sergey Senozhatsky <> | Subject | Re: [PATCH] MAINTAINERS: add zpool |
| |
On (06/02/15 14:30), Joe Perches wrote: > > >> +ZPOOL COMPRESSED PAGE STORAGE API > > >> +M: Dan Streetman <ddstreet@ieee.org> > > >> +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
| |