RESOLVED FIXED 79962
The JIT should not crash the entire process just because there is not enough executable memory, if the LLInt is enabled 
https://bugs.webkit.org/show_bug.cgi?id=79962
Summary The JIT should not crash the entire process just because there is not enough ...
Filip Pizlo
Reported 2012-02-29 17:29:49 PST
Patch forthcoming. <rdar://problem/10922215>
Attachments
the patch (42.92 KB, patch)
2012-02-29 17:33 PST, Filip Pizlo 		no flags
DetailsFormatted DiffDiff
the patch for qt (1.66 KB, patch)
2012-03-01 14:49 PST, Filip Pizlo 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Filip Pizlo
Comment 1 2012-02-29 17:33:26 PST
Created attachment 129562 [details] the patch
Gavin Barraclough
Comment 2 2012-02-29 17:47:31 PST
Comment on attachment 129562 [details] the patch r=me, minus the debug settings.
Csaba Osztrogonác
Comment 4 2012-03-01 14:43:11 PST
Reopen, because it broke Qt-ARM and Qt-SH4 builds: Qt-SH4 build error log: ------------------------- ../../../../Source/JavaScriptCore/assembler/SH4Assembler.h: In member function 'WTF::PassRefPtr<WTF::MetaAllocatorHandle> JSC::SH4Assembler::executableCopy(JSC::JSGlobalData&, void*, JSC::JITCompilationEffort)': ../../../../Source/JavaScriptCore/assembler/SH4Assembler.h:1520:68: error: no matching function for call to 'JSC::AssemblerBufferWithConstantPool<512, 4, 2, JSC::SH4Assembler>::executableCopy(JSC::JSGlobalData&, void*&, JSC::JITCompilationEffort&)'In file included from ../../../../Source/JavaScriptCore/assembler/MacroAssemblerSH4.h:32:0, ARM build error log: ----------------------- ../../../../Source/JavaScriptCore/assembler/ARMAssembler.cpp: In member function 'WTF::PassRefPtr<WTF::MetaAllocatorHandle> JSC::ARMAssembler::executableCopy(JSC::JSGlobalData&, void*, JSC::JITCompilationEffort)': ../../../../Source/JavaScriptCore/assembler/ARMAssembler.cpp:354:97: error: no matching function for call to 'JSC::AssemblerBufferWithConstantPool<2048, 4, 4, JSC::ARMAssembler>::executableCopy(JSC::JSGlobalData&, void*&, JSC::JITCompilationEffort&)' ../../../../Source/JavaScriptCore/assembler/AssemblerBufferWithConstantPool.h:198:40: note: candidate is: WTF::PassRefPtr<WTF::MetaAllocatorHandle> JSC::AssemblerBufferWithConstantPool<maxPoolSize, barrierSize, maxInstructionSize, AssemblerType>::executableCopy(JSC::JSGlobalData&, void*) [with int maxPoolSize = 2048, int barrierSize = 4, int maxInstructionSize = 4, AssemblerType = JSC::ARMAssembler]
Filip Pizlo
Comment 5 2012-03-01 14:46:17 PST
(In reply to comment #4) > Reopen, because it broke Qt-ARM and Qt-SH4 builds: > > Qt-SH4 build error log: > ------------------------- > ../../../../Source/JavaScriptCore/assembler/SH4Assembler.h: In member function 'WTF::PassRefPtr<WTF::MetaAllocatorHandle> JSC::SH4Assembler::executableCopy(JSC::JSGlobalData&, void*, JSC::JITCompilationEffort)': > ../../../../Source/JavaScriptCore/assembler/SH4Assembler.h:1520:68: error: no matching function for call to 'JSC::AssemblerBufferWithConstantPool<512, 4, 2, JSC::SH4Assembler>::executableCopy(JSC::JSGlobalData&, void*&, JSC::JITCompilationEffort&)'In file included from ../../../../Source/JavaScriptCore/assembler/MacroAssemblerSH4.h:32:0, > > ARM build error log: > ----------------------- > ../../../../Source/JavaScriptCore/assembler/ARMAssembler.cpp: In member function 'WTF::PassRefPtr<WTF::MetaAllocatorHandle> JSC::ARMAssembler::executableCopy(JSC::JSGlobalData&, void*, JSC::JITCompilationEffort)': > ../../../../Source/JavaScriptCore/assembler/ARMAssembler.cpp:354:97: error: no matching function for call to 'JSC::AssemblerBufferWithConstantPool<2048, 4, 4, JSC::ARMAssembler>::executableCopy(JSC::JSGlobalData&, void*&, JSC::JITCompilationEffort&)' > ../../../../Source/JavaScriptCore/assembler/AssemblerBufferWithConstantPool.h:198:40: note: candidate is: WTF::PassRefPtr<WTF::MetaAllocatorHandle> JSC::AssemblerBufferWithConstantPool<maxPoolSize, barrierSize, maxInstructionSize, AssemblerType>::executableCopy(JSC::JSGlobalData&, void*) [with int maxPoolSize = 2048, int barrierSize = 4, int maxInstructionSize = 4, AssemblerType = JSC::ARMAssembler] Looks like I forgot about threading JITCompilationEffort through AssemblerBufferWithConstantPool::executableCopy().
Filip Pizlo
Comment 6 2012-03-01 14:49:23 PST
Created attachment 129751 [details] the patch for qt
Csaba Osztrogonác
Comment 7 2012-03-01 21:43:04 PST
Comment on attachment 129751 [details] the patch for qt rs=me, thanks for the fix.
Csaba Osztrogonác
Comment 8 2012-03-01 22:28:59 PST
Comment on attachment 129751 [details] the patch for qt Clearing flags on attachment: 129751 Committed r109511: <http://trac.webkit.org/changeset/109511>
Csaba Osztrogonác
Comment 9 2012-03-01 22:29:08 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.