Thread (5 messages) 5 messages, 3 authors, 2015-08-07

Re: [PATCH] perf trace: add total time column to summary.

From: Milian Wolff <hidden>
Date: 2015-08-06 09:53:44

Hey Arnaldo,

On Thursday 06 August 2015 11:24:29 Milian Wolff wrote:
<snip>
Cc: Arnaldo Carvalho de Melo <acme@kernel.org>
Signed-off-by: Milian Wolff <redacted>
I hope this patch is OK. I have some more questions:

a) Should I always send to linux-kernel and CC linux-perf-users, or is it OK 
to send it only to linux-perf-users? 

b) Should I put you explicitly into the CC, I recon you also read the mailing 
lists?

c) Should I add more people to the CC list? The get_maintainer script also 
lists Ingo Molnar and Peter Zijlstra, but your recent patch set has them not 
in the CC list.

d) My patch has some warnings according to checkpatch, because some lines are 
too long. Is this very frowned upon in the Kernel? It would make the table 
much harder to read when I split it onto multiple lines, but of course I can 
amend that if you want it.

If I messed up anything else, please tell me.

Thanks
-- 
Milian Wolff | milian.wolff@kdab.com | Software Engineer
KDAB (Deutschland) GmbH&Co KG, a KDAB Group company
Tel: +49-30-521325470
KDAB - The Qt Experts
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help