[PATCH] MAINTAINERS: add zpool

[PATCH] MAINTAINERS: add zpool

[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

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] MAINTAINERS: add zpool

[Joe Perches]

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/ ?


--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] MAINTAINERS: add zpool

[Dan Streetman]

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?

>
>

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] MAINTAINERS: add zpool

[Joe Perches]

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

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] MAINTAINERS: add zpool

[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

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

Re: [PATCH] MAINTAINERS: add zpool

[Dan Streetman]

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

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>