On Tue, Nov 22, 2016 at 10:44 PM, Fengguang Wu <fengguang.wu@xxxxxxxxx> wrote:
On Tue, Nov 22, 2016 at 02:04:42PM -0800, Linus Torvalds wrote:
I also noticed that the kernel test robot had screwed up the
participants list for some reason, and had
"Acked-by: Alexander Duyck <alexander.h.duyck@xxxxxxxxx>, David S.
as one of the participants. So there's some odd commit parsing issue
there somewhere. But Alexander seems to have seen this report despite
that, it just never went anywhere that I can tell.
Yeah the robot will CC all "Acked-by" people in the bug reports.
Shall we limit it to the below TO/CC list?
No. We do want to keep the Acked-by's on the cc.
But you missed the real problem.
It *didn't* cc the acked-by. Look closer. What happened was that it cc'd this:
"Acked-by: Alexander Duyck <alexander.h.duyck@xxxxxxxxx>, David S. Miller"
ie there is only _one_ email address (that of davem@xxxxxxxxxxxxx),
and the whole "Acked-by: Alexander Duyck <...>" part is quoted as the
_name_ of that email address.
At least that's what the headers look like for me in the original report:
From: kernel test robot <xiaolong.ye@xxxxxxxxx>
To: Eric Dumazet <edumazet@xxxxxxxxxx>
Cc: lkp@xxxxxx, Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>,
LKML <linux-kernel@xxxxxxxxxxxxxxx>, Alexei Starovoitov
<ast@xxxxxxxxxx>, Willem de Bruijn <willemb@xxxxxxxxxx>, "Acked-by:
Alexander Duyck <alexander.h.duyck@xxxxxxxxx>, David S. Miller"
Notice the quoting of that last "name".