* [PATCH] MAINTAINERS: add zpool @ 2015-06-02 16:56 Dan Streetman 2015-06-02 21:09 ` Joe Perches 0 siblings, 1 reply; 6+ messages in thread From: Dan Streetman @ 2015-06-02 16:56 UTC (permalink / raw) To: Andrew Morton; +Cc: linux-mm, linux-kernel, Dan Streetman Add entry for zpool to MAINTAINERS file. Signed-off-by: Dan Streetman <ddstreet@ieee.org> --- MAINTAINERS | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index e308718..5c0f13b 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -11056,6 +11056,13 @@ L: zd1211-devs@lists.sourceforge.net (subscribers-only) S: Maintained F: drivers/net/wireless/zd1211rw/ +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 + ZR36067 VIDEO FOR LINUX DRIVER L: mjpeg-users@lists.sourceforge.net L: linux-media@vger.kernel.org -- 2.1.0 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add zpool 2015-06-02 16:56 [PATCH] MAINTAINERS: add zpool Dan Streetman @ 2015-06-02 21:09 ` Joe Perches 2015-06-02 21:19 ` Dan Streetman 0 siblings, 1 reply; 6+ messages in thread From: Joe Perches @ 2015-06-02 21:09 UTC (permalink / raw) To: Dan Streetman; +Cc: Andrew Morton, linux-mm, linux-kernel On Tue, 2015-06-02 at 12:56 -0400, Dan Streetman wrote: > Add entry for zpool to MAINTAINERS file. [] > diff --git a/MAINTAINERS b/MAINTAINERS [] > @@ -11056,6 +11056,13 @@ L: zd1211-devs@lists.sourceforge.net (subscribers-only) > S: Maintained > F: drivers/net/wireless/zd1211rw/ > > +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/ ? ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add zpool 2015-06-02 21:09 ` Joe Perches @ 2015-06-02 21:19 ` Dan Streetman 2015-06-02 21:30 ` Joe Perches 0 siblings, 1 reply; 6+ messages in thread From: Dan Streetman @ 2015-06-02 21:19 UTC (permalink / raw) To: Joe Perches Cc: Andrew Morton, Linux-MM, linux-kernel, Minchan Kim, Nitin Gupta, Sergey Senozhatsky On Tue, Jun 2, 2015 at 5:09 PM, Joe Perches <joe@perches.com> wrote: > On Tue, 2015-06-02 at 12:56 -0400, Dan Streetman wrote: >> Add entry for zpool to MAINTAINERS file. > [] >> diff --git a/MAINTAINERS b/MAINTAINERS > [] >> @@ -11056,6 +11056,13 @@ L: zd1211-devs@lists.sourceforge.net (subscribers-only) >> S: Maintained >> F: drivers/net/wireless/zd1211rw/ >> >> +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? > > ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add zpool 2015-06-02 21:19 ` Dan Streetman @ 2015-06-02 21:30 ` Joe Perches 2015-06-03 3:56 ` Sergey Senozhatsky 0 siblings, 1 reply; 6+ messages in thread From: Joe Perches @ 2015-06-02 21:30 UTC (permalink / raw) To: Dan Streetman Cc: Andrew Morton, Linux-MM, linux-kernel, Minchan Kim, Nitin Gupta, Sergey Senozhatsky On Tue, 2015-06-02 at 17:19 -0400, Dan Streetman wrote: > On Tue, Jun 2, 2015 at 5:09 PM, Joe Perches <joe@perches.com> wrote: > > On Tue, 2015-06-02 at 12:56 -0400, Dan Streetman wrote: > >> Add entry for zpool to MAINTAINERS file. > > [] > >> diff --git a/MAINTAINERS b/MAINTAINERS > > [] > >> @@ -11056,6 +11056,13 @@ L: zd1211-devs@lists.sourceforge.net (subscribers-only) > >> S: Maintained > >> F: drivers/net/wireless/zd1211rw/ > >> > >> +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. cheers, Joe ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add zpool 2015-06-02 21:30 ` Joe Perches @ 2015-06-03 3:56 ` Sergey Senozhatsky 2015-06-03 12:55 ` Dan Streetman 0 siblings, 1 reply; 6+ messages in thread From: Sergey Senozhatsky @ 2015-06-03 3:56 UTC (permalink / raw) To: Joe Perches Cc: Dan Streetman, Andrew Morton, Linux-MM, linux-kernel, Minchan Kim, Nitin Gupta, Sergey Senozhatsky 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] MAINTAINERS: add zpool 2015-06-03 3:56 ` Sergey Senozhatsky @ 2015-06-03 12:55 ` Dan Streetman 0 siblings, 0 replies; 6+ messages in thread From: Dan Streetman @ 2015-06-03 12:55 UTC (permalink / raw) To: Sergey Senozhatsky Cc: Joe Perches, Andrew Morton, Linux-MM, linux-kernel, Minchan Kim, Nitin Gupta On Tue, Jun 2, 2015 at 11:56 PM, Sergey Senozhatsky <sergey.senozhatsky.work@gmail.com> wrote: > 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. Ok. I'll send a patch to move it from include/linux to mm/ and update the drivers there that include it. > > -ss ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-06-03 12:55 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-06-02 16:56 [PATCH] MAINTAINERS: add zpool Dan Streetman 2015-06-02 21:09 ` Joe Perches 2015-06-02 21:19 ` Dan Streetman 2015-06-02 21:30 ` Joe Perches 2015-06-03 3:56 ` Sergey Senozhatsky 2015-06-03 12:55 ` Dan Streetman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox