Re: [PATCH v6 00/11] add performance tracing facility

2014-06-20 Thread Karsten Blees
Am 21.06.2014 00:49, schrieb Philip Oakley:
> Should there be some documentation as well? Perhaps in t/README, or in
> Documentation/howto.

I'll add Documentation/technical/api-trace.txt when I find the time.
But lets settle on the final API first.
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v6 00/11] add performance tracing facility

2014-06-20 Thread Philip Oakley

From: "Karsten Blees" 

Changes since v5:
[05/11]: GIT_TRACE_BARE=1 disables 'timestamp file:line' output for
unit tests that rely on trace output (t1510 and t5503)
[08/11]: Align original trace output at col 40
[09/11]: Dropped '(div 10e9)' from the commit message.
[10/11]: Dropped trace_performance[_since]() return value and the
respective usage example. Renamed trace_performance_vfl to
trace_performance_vprintf_fl.

The other patches are the versions from pu.

Karsten Blees (11):
 trace: move trace declarations from cache.h to new trace.h
 trace: consistently name the format parameter
 trace: remove redundant printf format attribute
 trace: factor out printing to the trace file
 trace: add infrastructure to augment trace output with additional 
info

 trace: add current timestamp to all trace output
 trace: move code around, in preparation to file:line output
 trace: add 'file:line' to all trace output
 trace: add high resolution timer function to debug performance issues
 trace: add trace_performance facility to debug performance issues
 git: add performance tracing for git's main() function to debug
   scripts

Makefile  |   7 ++
cache.h   |  13 +--
config.mak.uname  |   1 +
git-compat-util.h |   4 +
git.c |   2 +
t/t1510-repo-setup.sh |   2 +-
t/t5503-tagfollow.sh  |   8 +-
trace.c   | 313 
--

trace.h   |  90 +++
9 files changed, 387 insertions(+), 53 deletions(-)


Should there be some documentation as well? Perhaps in t/README, or in
Documentation/howto.



create mode 100644 trace.h

--
Philip 


--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v6 00/11] add performance tracing facility

2014-06-20 Thread Karsten Blees
Changes since v5:
[05/11]: GIT_TRACE_BARE=1 disables 'timestamp file:line' output for
 unit tests that rely on trace output (t1510 and t5503)
[08/11]: Align original trace output at col 40
[09/11]: Dropped '(div 10e9)' from the commit message.
[10/11]: Dropped trace_performance[_since]() return value and the
 respective usage example. Renamed trace_performance_vfl to
 trace_performance_vprintf_fl.

The other patches are the versions from pu.

Karsten Blees (11):
  trace: move trace declarations from cache.h to new trace.h
  trace: consistently name the format parameter
  trace: remove redundant printf format attribute
  trace: factor out printing to the trace file
  trace: add infrastructure to augment trace output with additional info
  trace: add current timestamp to all trace output
  trace: move code around, in preparation to file:line output
  trace: add 'file:line' to all trace output
  trace: add high resolution timer function to debug performance issues
  trace: add trace_performance facility to debug performance issues
  git: add performance tracing for git's main() function to debug
scripts

 Makefile  |   7 ++
 cache.h   |  13 +--
 config.mak.uname  |   1 +
 git-compat-util.h |   4 +
 git.c |   2 +
 t/t1510-repo-setup.sh |   2 +-
 t/t5503-tagfollow.sh  |   8 +-
 trace.c   | 313 --
 trace.h   |  90 +++
 9 files changed, 387 insertions(+), 53 deletions(-)
 create mode 100644 trace.h

-- 
2.0.0.402.g13b8b25

--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html