Re: [PATCH 09/10] lustre: Add MAINTAINERS entry

[Oleg Drokin <green@linuxhacker.ru>]

On Aug 17, 2014, at 12:37 PM, Greg Kroah-Hartman wrote:

> On Fri, Aug 15, 2014 at 12:48:14PM -0400, Oleg Drokin wrote:
>> Just add the entry with some info.
>> 
>> Signed-off-by: Oleg Drokin <green@linuxhacker.ru>
>> ---
>> MAINTAINERS | 8 ++++++++
>> 1 file changed, 8 insertions(+)
>> 
>> diff --git a/MAINTAINERS b/MAINTAINERS
>> index 7e2eb4c..369183b 100644
>> --- a/MAINTAINERS
>> +++ b/MAINTAINERS
>> @@ -8646,6 +8646,14 @@ W:	http://www.lirc.org/
>> S:	Odd Fixes
>> F:	drivers/staging/media/lirc/
>> 
>> +STAGING - LUSTRE PARALLEL FILESYSTEM
>> +M:	Oleg Drokin <oleg.drokin@intel.com>
>> +M:	Andreas Dilger <andreas.dilger@intel.com>
>> +L:	HPDD-discuss@lists.01.org (moderated for non-subscribers)
> 
> I _really_ hate moderated mailing lists for kernel development stuff.
> Why not just use the driverdev mailing list instead?

I guess this could be done if we really need to, but ...

> I note that you didn't even cc: this patch series to that list...

The instructions in the beginning of the file say:
"L: Mailing list that is relevant to this area"
This is the mailing list that is relevant to the area, but we don't really send patches in there,
and because it's premoderated it's not really convenient anyway as you rightly notice.
People can ask other questions they might have bout their problems there, though, and there's
a higher chance that somebody familiar with lustre will read and answer them.

the 'M:' is the send patches to according to the instructions?

The patches are CCed to linux-kernel by most everybody anyway, so do we really need to
spell this out explicitly with an 'L:' line or in some other way?