On Tue, Jun 2, 2015 at 11:56 PM, Sergey Senozhatsky <sergey.senozhatsky.work@xxxxxxxxx> wrote: > On (06/02/15 14:30), Joe Perches wrote: >> > >> +ZPOOL COMPRESSED PAGE STORAGE API >> > >> +M: Dan Streetman <ddstreet@xxxxxxxx> >> > >> +L: linux-mm@xxxxxxxxx >> > >> +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. Ok. I'll send a patch to move it from include/linux to mm/ and update the drivers there that include it. > > -ss -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx";> email@xxxxxxxxx </a>