[Webkit-unassigned] [Bug 41790] Make FastMalloc more portable

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Thu Jul 8 02:32:36 PDT 2010


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


Zoltan Herczeg <zherczeg at webkit.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |zherczeg at webkit.org




--- Comment #8 from Zoltan Herczeg <zherczeg at webkit.org>  2010-07-08 02:32:36 PST ---
If we use threading.h, and wtf based mutexes, we should ALSO create the thread by wtf based method.

runScavengerThread calls the scavengerThread()

1492    void TCMalloc_PageHeap::initializeScavenger()
1493    {
1494      m_scavengeThreadActive = true;
1495      pthread_t thread;
1496      pthread_create(&thread, 0, runScavengerThread, this);
1497    }

pthread_create - should be replaced by its threading.h variant

I think Qt depends on some thread specific variables, which are not created for the thread by pthread_create

I suggest to rollout this patch until we fix and test this issue

-- 
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