Hi Philippe,

sorry to be poking about this again. Is there any news on the issue? Can I
help to provide a fix for it? Would increasing the pool size really be a
good fix? Or should the code be re-written to dynamically allocate the
needed memory? I guess another solution could be to just truncate the
string, if it is just debug information then that would at least be better
that crashing?

Let me know how I can help

Thanks

/David

On Mon, Feb 8, 2016 at 2:19 PM, David Hallas <da...@davidhallas.dk> wrote:

> Hi Philippe,
>
> thanks a lot for your help. I have created this ticket
> https://bugs.kde.org/show_bug.cgi?id=359133 to track the issue.
>
> I was wondering, is there some kind out of out buffering when using printf
> in valgrind? I guess that could explain why I do not see the debug print?
>
> Thanks
>
> /David
>
> On Fri, Feb 5, 2016 at 9:05 PM, Philippe Waroquiers <
> philippe.waroqui...@skynet.be> wrote:
>
>> On Fri, 2016-02-05 at 11:20 +0100, David Hallas wrote:
>>
>>
>> > thanks for the reply. I tried increasing the SEGINFO_STRPOOLSIZE as
>> > you requested, and for me the magic number is 268*1024 then the assert
>> > goes away :) I also tried to add the print to the ML_(addStr) function
>> > in storage.c but for some reason I never see the print?
>> Strange. VG_(printf) prints unconditionally.
>> >
>> > I tried to play with objdump and nm to find some large symbols, but I
>> > think I do not have enough objdump/nm skills to do that. But I also
>> > tried to play with strings and I found one string (a symbol name) that
>> > is 274154 bytes long (this fits perfectly with that I see the problem
>> > if the SEGINFO_STRPOOLSIZE is 267*1024 but not with 268*1024). The
>> > code that I am debugging is using a lot of templates, so go figure ;)
>> > So I think it sounds very plausible that it is this string that is
>> > causing the problem?
>> A symbol name of 268 Kb is quite impressive,
>> I hope you never have to print this variable in gdb :).
>> >
>> > Please let me know if there is any other information that you need? Or
>> > if you have some other things to test.
>> The information is clear, I think the problem is identified now.
>> The best is to file a bug entry in bugzilla, I hope to fix
>> this problem in the coming weeks or so.
>>
>> Thanks
>>
>> Philippe
>>
>>
>>
>>
>
------------------------------------------------------------------------------
Site24x7 APM Insight: Get Deep Visibility into Application Performance
APM + Mobile APM + RUM: Monitor 3 App instances at just $35/Month
Monitor end-to-end web transactions and take corrective actions now
Troubleshoot faster and improve end-user experience. Signup Now!
http://pubads.g.doubleclick.net/gampad/clk?id=272487151&iu=/4140
_______________________________________________
Valgrind-users mailing list
Valgrind-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/valgrind-users

Reply via email to