This is the mail archive of the cygwin-patches mailing list for the Cygwin project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] gprof profiling of multi-threaded Cygwin programs, ver 2

From: Jon Turney <jon dot turney at dronecode dot org dot uk>
To: cygwin-patches at cygwin dot com
Date: Mon, 22 Feb 2016 16:14:41 +0000
Subject: Re: [PATCH] gprof profiling of multi-threaded Cygwin programs, ver 2
Authentication-results: sourceware.org; auth=none
References: <56C820D8 dot 4010203 at maxrnd dot com> <56CAF4A3 dot 5060806 at dronecode dot org dot uk> <20160222142502 dot GA26624 at calimero dot vinschen dot de>

On 22/02/2016 14:25, Corinna Vinschen wrote:

On Feb 22 11:44, Jon Turney wrote:

Thanks for this.  A few comments inline.
[...]
On 20/02/2016 08:16, Mark Geisert wrote:

+      // record profiling samples for other pthreads, if any
+      cygwin_internal (CW_CYGHEAP_PROFTHR_ALL, profthr_byhandle);
+


Hmm.. so why isn't this written as cygheap_profthr_all (profthr_byhandle) ?


I asked for it:
https://cygwin.com/ml/cygwin-patches/2016-q1/msg00028.html

Ok. I guess I don't understand why it's exported at all in that versionof the patch.

Follow-Ups:
- Re: [PATCH] gprof profiling of multi-threaded Cygwin programs, ver 2
  - From: Corinna Vinschen

References:
- [PATCH] gprof profiling of multi-threaded Cygwin programs, ver 2
  - From: Mark Geisert
- Re: [PATCH] gprof profiling of multi-threaded Cygwin programs, ver 2
  - From: Jon Turney
- Re: [PATCH] gprof profiling of multi-threaded Cygwin programs, ver 2
  - From: Corinna Vinschen

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]