[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