Re: patch [Re: [fpc-devel] time for dumping stacktrace, when using dwarf.]

2009-11-14 Thread Michael Van Canneyt
On Sat, 14 Nov 2009, Martin wrote: Martin wrote: I'll have to withdraw it. I have no idea how I compiled an exe that actually did not crash when dumping a stack (yes I had one.). but the patch doesn't work. I had that right,, just the debugger confused me the way it displayed the

Re: patch [Re: [fpc-devel] time for dumping stacktrace, when using dwarf.]

2009-11-14 Thread Jonas Maebe
On 14 Nov 2009, at 11:16, Michael Van Canneyt wrote: I don't think that the thread-safety is something to worry about at this time; As you remark, the code was already not thread-safe. But it is something to keep in mind. At the very least this bug should be documented, and a {$warning

Re: patch [Re: [fpc-devel] time for dumping stacktrace, when using dwarf.]

2009-11-14 Thread Michael Van Canneyt
On Sat, 14 Nov 2009, Jonas Maebe wrote: On 14 Nov 2009, at 11:16, Michael Van Canneyt wrote: I don't think that the thread-safety is something to worry about at this time; As you remark, the code was already not thread-safe. But it is something to keep in mind. At the very least this

Re: patch [Re: [fpc-devel] time for dumping stacktrace, when using dwarf.]

2009-11-14 Thread Jonas Maebe
On 14 Nov 2009, at 11:30, Michael Van Canneyt wrote: On Sat, 14 Nov 2009, Jonas Maebe wrote: At the very least this bug should be documented, and a {$warning ...} should be added at the appropriate place in the source code so that someone interested in fixing it can do that right away

Re: patch [Re: [fpc-devel] time for dumping stacktrace, when using dwarf.]

2009-11-14 Thread Jonas Maebe
On 14 Nov 2009, at 12:19, Michael Van Canneyt wrote: On Sat, 14 Nov 2009, Jonas Maebe wrote: I can't follow here. There was an undocumented problem in a unit, and now that it has been identified and sort of made worse, there is no need to tell our users about it? Well, IMHO without

Re: patch [Re: [fpc-devel] time for dumping stacktrace, when using dwarf.]

2009-11-14 Thread Michael Van Canneyt
On Sat, 14 Nov 2009, Jonas Maebe wrote: On 14 Nov 2009, at 12:19, Michael Van Canneyt wrote: On Sat, 14 Nov 2009, Jonas Maebe wrote: I can't follow here. There was an undocumented problem in a unit, and now that it has been identified and sort of made worse, there is no need to tell our

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-13 Thread Martin
Michael Van Canneyt wrote: On Tue, 10 Nov 2009, Schatzl Thomas wrote: The main reason for this slowness is that that implementation (of mine) reads debug info byte-by-byte from the input stream using blockread(). I think this I/O the main performance bottleneck. At that time I did not

argh [Re: [fpc-devel] time for dumping stacktrace, when using dwarf.]

2009-11-13 Thread Martin
I'll have to withdraw it. I have no idea how I compiled an exe that actually did not crash when dumping a stack (yes I had one.). but the patch doesn't work. Problem I have is how to deal with the open var param. I need to be able to address the first (or other) byte of it function

patch [Re: [fpc-devel] time for dumping stacktrace, when using dwarf.]

2009-11-13 Thread Martin
Martin wrote: I'll have to withdraw it. I have no idea how I compiled an exe that actually did not crash when dumping a stack (yes I had one.). but the patch doesn't work. I had that right,, just the debugger confused me the way it displayed the values. Attached patch works here, and

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Schatzl Thomas
Hello, Von: Martin laza...@mfriebe.de Just to make sure I don't to anything silly. When an application crashes, it usually (at least in Lazarus) dumps a stacktrace to the console. I used to use stabs, and the stacktrace was dumped in less than a second. Now I have switched to dwarf

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Michael Van Canneyt
On Tue, 10 Nov 2009, Schatzl Thomas wrote: Hello, Von: Martin laza...@mfriebe.de Just to make sure I don't to anything silly. When an application crashes, it usually (at least in Lazarus) dumps a stacktrace to the console. I used to use stabs, and the stacktrace was dumped in less than

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Jonas Maebe
On 10 Nov 2009, at 11:32, Michael Van Canneyt wrote: By making the buffer a threadvar, this should not be an issue ? It will waste memory for each thread you start though, for functionality that you hopefully won't need anyway. Jonas ___

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Michael Van Canneyt
On Tue, 10 Nov 2009, Jonas Maebe wrote: On 10 Nov 2009, at 11:32, Michael Van Canneyt wrote: By making the buffer a threadvar, this should not be an issue ? It will waste memory for each thread you start though, for functionality that you hopefully won't need anyway. Correct. But

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Micha Nelissen
Schatzl Thomas wrote: Improving upon that should be trivial, all reading from the debug input has been encapsuled in the two ReadNext() methods in the file mentioned. It should be easy to make them to read from a (static?) buffer that is filled blockwise; note that a static buffer may give

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Vincent Snijders
Micha Nelissen schreef: Schatzl Thomas wrote: Improving upon that should be trivial, all reading from the debug input has been encapsuled in the two ReadNext() methods in the file mentioned. It should be easy to make them to read from a (static?) buffer that is filled blockwise; note that a

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Jonas Maebe
On 10 Nov 2009, at 11:57, Michael Van Canneyt wrote: But let's start by improving efficiency, the MT issue can follow later :-) I disagree. I think correctness should always be more important than efficiency, especially for RTL functions (and in particular when it's about debugging

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Martin
Vincent Snijders wrote: Micha Nelissen schreef: Schatzl Thomas wrote: Improving upon that should be trivial, all reading from the debug input has been encapsuled in the two ReadNext() methods in the file mentioned. It should be easy to make them to read from a (static?) buffer that is filled

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Vinzent Höfler
Vincent Snijders vsnijd...@vodafonevast.nl: Micha Nelissen schreef: Allocating that buffer on the stack isn't sufficient? That's not easy when the exception is raised in case of a stack overflow. In case of a stack overflow *anything* using the stack is doomed. But if we're talking

Re: [fpc-devel] time for dumping stacktrace, when using dwarf.

2009-11-10 Thread Martin
Vinzent Höfler wrote: Vincent Snijders vsnijd...@vodafonevast.nl: Micha Nelissen schreef: Allocating that buffer on the stack isn't sufficient? That's not easy when the exception is raised in case of a stack overflow. In case of a stack overflow *anything* using the