UNCONFIRMED 119565
problems encountered with fast-malloc on illumos x86_64 
https://bugs.webkit.org/show_bug.cgi?id=119565
Summary problems encountered with fast-malloc on illumos x86_64
richard
Reported 2013-08-07 23:17:02 PDT
Hello, Currently working on the upgrade for the pkgsrc port of webkitgtk from 1.10.2 to 2.0.4.. We're experiencing lately problems with both 1.10.2 and the work in progress during the build of the WebKit-1.0.gir target. WebKit was configured with the following options: Build configuration: Enable debugging (slow) : no Compile with debug symbols (slow) : no Enable GCC build optimization : yes Code coverage support : no Optimized memory allocator : yes Accelerated rendering backend : opengl (gl, glx) Features: ======= WebKit1 support : yes WebKit2 support : no Accelerated Compositing : yes Gamepad support : no Geolocation support : no HTML5 video element support : yes JIT compilation : yes Opcode stats : no SVG fonts support : yes SVG support : yes Spellcheck support : yes Credential storage support : yes Web Audio support : no WebGL : yes XSLT support : yes GTK+ configuration: GTK+ version : 2.0 GDK target : x11 Introspection support : yes Generate documentation : no An extract of the output is (BTW we've patched the WebKit: missing ':' at column type of problems): GEN WebKit-1.0.gir .. Source/WebKit/gtk/webkit/webkitwebdatabase.cpp:34: Warning: WebKit: multiple comment blocks documenting 'SECTION:webkitwebdatabase:' identifier. Source/WebKit/gtk/webkit/webkitversion.h:33: Warning: WebKit: symbol='WEBKITGTK_API_VERSION': Unknown namespace for symbol 'WEBKITGTK_API_VERSION' g-ir-scanner: compile: ...blah libtool: link: ...blah Command '['/var/tmp/pkgsrc/www/webkit-gtk/work/webkitgtk-2.0.4/tmp-introspectT3dqgo/WebKit-1.0', '--introspect-dump=/var/tmp/pkgsrc/www/webkit-gtk/work/webkitgtk-2.0.4/tmp-introspectT3dqgo/functions.txt,/var/tmp/pkgsrc/www/webkit-gtk/work/webkitgtk-2.0.4/tmp-introspectT3dqgo/dump.xml']' returned non-zero exit status -11 -11 is SIGSEGV and the following is the stack: core 'gir-core' of 6347: /var/tmp/pkgsrc/www/webkit-gtk/work/webkitgtk-2.0.4/tmp-introspectIcuk ----------------- lwp# 1 / thread# 1 -------------------- fffffd7fc2302499 _ZN3WTF17TCMalloc_PageHeap8GrowHeapEm () + d9 fffffd7fc2303c0a _ZN3WTF25TCMalloc_Central_FreeList18FetchFromSpansSafeEv () + 19a fffffd7fc2304200 _ZN3WTF25TCMalloc_Central_FreeList11RemoveRangeEPNS_11HardenedSLLES2_Pi () + 70 fffffd7fc2305634 _ZN3WTF10fastMallocEm () + 354 fffffd7fc232fc1b _ZN3WTF10StringImpl5emptyEv () + 3b fffffd7fc2318607 _ZN3WTF19initializeThreadingEv () + 27 fffffd7fc223bc2e _ZN3JSC19initializeThreadingEv () + 1e fffffd7fc34ceb83 webkitInit () + 53 fffffd7fc34c6d8f _ZL33webkit_download_class_intern_initPv () + 5f fffffd7ffa86f8a2 g_type_class_ref () + 4d2 0000000000409b75 dump_properties () + c5 000000000040f566 main () + 906 00000000004097bc _start () + 6c ----------------- lwp# 3 / thread# 3 -------------------- fffffd7ffb990e97 __lwp_park () + 17 fffffd7ffb98a49b cond_wait_queue (fffffd7fc2399408, fffffd7fc23993f0, 0) + 5b fffffd7ffb98ab53 __cond_wait (fffffd7fc2399408, fffffd7fc23993f0) + b3 fffffd7ffb98ab8a cond_wait (fffffd7fc2399408, fffffd7fc23993f0) + 2a fffffd7ffb98abc5 pthread_cond_wait (fffffd7fc2399408, fffffd7fc23993f0) + 15 fffffd7fc2302e26 _ZN3WTF17TCMalloc_PageHeap15scavengerThreadEv () + d6 fffffd7fc2302e49 ???????? () fffffd7ffb990b3a _thrp_setup (fffffd7ffeae0240) + 8a fffffd7ffb990e50 _lwp_start () I notice that Source/WTF/wtf/FastMalloc.cpp is not current with the latest gproftool's TCMalloc (notably for TCMalloc_PageHeap::GrowHeap). Currently I'm testing a build with --enable-fast-malloc=no, but would it be possible to update these bits to the latest TCMalloc code (https://code.google.com/p/gperftools/) which is the recently released 2.1?
Attachments
Add attachment proposed patch, testcase, etc.
richard
Comment 1 2013-08-08 09:42:36 PDT
Well, --enable-fast-malloc=no doesn't seem to have made a difference, wondering if it is because in http://trac.webkit.org/changeset/142179 WTF_SYSTEM_MALLOC is defined as 1, but USE_SYSTEM_MALLOC is what appears to be checked for in the source code. Perhaps this was an incomplete patchset. Trying again now with CPPFLAGS+= -DUSE_SYSTEM_MALLOC as well.
richard
Comment 2 2013-08-08 21:51:25 PDT
That seemed to do the trick. Now I only get : (process:311): GLib-GObject-CRITICAL **: g_object_class_install_property: assertion `class->set_property != NULL' failed Source/WebKit/gtk/webkit/webkitwebplugin.cpp:217: Warning: WebKit: webkit_web_plugin_get_mimetypes: Unknown type: 'WebKit.WebKitWebPluginMIMEType' Source/WebKit/gtk/webkit/webkitglobals.cpp:267: Warning: WebKit: webkit_get_text_checker: return value: Missing (transfer) annotation Source/WebKit/gtk/webkit/webkitwebframe.cpp:840: Warning: WebKit: webkit_web_frame_get_global_context: return value: Unresolved type: 'JSGlobalContextRef' Source/WebKit/gtk/webkit/webkitwebframe.cpp:1146: Warning: WebKit: webkit_web_frame_get_range_for_word_around_caret: return value: Missing (transfer) annotation Source/WebKit/gtk/webkit/webkitwebplugindatabase.h:51: Warning: WebKit: webkit_web_plugin_database_plugins_list_free: argument list: Missing (element-type) annotation GEN WebKit-1.0.typelib but the build seems successful.
Note You need to log in before you can comment on or make changes to this bug.