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. -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>