On Fri, Jan 16, 2015 at 12:05:18PM +0000, Mark Brown wrote:

On Fri, Jan 16, 2015 at 10:10:34AM +0000, Catalin Marinas wrote:

quoted

Indeed, keeping the series as one thread is better. Apart from a
slightly less misleading subject, I suggest Hanjun that he passes each
patch via get_maintainer.pl and adds the corresponding Cc: lines to the
commit log. I think that's a clearer way keep track of who needs to
ack/review the patches.

I'm not saying stop sending them as part of this series, I'm saying it
might help to get people to look at them if they were also sent
disassociated from it - in the situations where I end up ignoring things
that look controversial but unexciting for my bit of it I know I won't
even bother opening the e-mails since I know it's just going to get
resent anyway quite often with revisions from whatever makes the rest of
the series controversial, I expect others will do the same.

Actually, another thing that might help would be if you and/or Will were
to prod the relevant people for review, letting them know that the
controversy isn't likely to affect their bits.

Hmm, we *could*, but having maintainers go round poking other people to
review patches feels like the wrong way round to me. The authors should
be taking ownership of this series and that's half of the problem we
had in previous versions.

It needs to be clear that Hanjun/Al/Graeme are the ones running the show,
not just now, but (even more importantly) when people start building on
top of the base support. If Catalin and I have to go around asking for
review, it sets us off on the wrong foot imo.

Will