[Webkit-unassigned] [Bug 27051] Use fastMalloc when neither MMAP nor VIRTUALALLOC are enabled

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Wed Aug 5 11:18:03 PDT 2009


https://bugs.webkit.org/show_bug.cgi?id=27051





--- Comment #2 from Norbert Leser <norbert.leser at nokia.com>  2009-08-05 11:18:02 PDT ---
(In reply to comment #1)
> (From update of attachment 32410 [details])
> I'll have to check into whether this works - I don't remember whether there are
> special alignment or lazy commit requirements.
> 
> But I also noticed that PLATFORM(SYMBIAN) uses fastMalloc in place of VM
> allocation inside Collector.cpp. That's definitely bugus - the GC counts on the
> guaranteed alignment provided by vm_map, VirtualAlloc and posix_memalign.
> Failing to meet the alignment requirements will lead to random crashes. If
> Symbian has no way to get aligned memory chunks, it needs to do a trick to fix
> up the alignment, like the mmap() code path.

Maciej, did you have a chance, in the meantime, to verify whether there are
special requirements for alignment, etc? I could not find any just by looking
at the code, and we are running my patch here for a while already without
seeing any impact. 

It would be great if we can resolve this issue soon.
If you can point me to alignment issues, I could easily submit my initial code
that mimics alignment via fastMalloc() but otherwise it would certainly be
cleaner to leave that overhead out and just implement my patch as is.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.



More information about the webkit-unassigned mailing list