Closed Bug 558795 Opened 15 years ago Closed 15 years ago

Investigate how to speed up nsEventTargetChainItem (de)allocation

Categories

(Core :: DOM: Events, defect)

x86
All
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: smaug, Assigned: smaug)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Currently nsFixedSizeAllocator is used, but it shows up surprisingly high in the Shark profiles. Perhaps nsEventTargetChainItem could be cached in a linked list or something like that and release them only if large amount of objects has been released.
Attached patch patch (deleted) — Splinter Review
This should do it. We allocate ETCI objects from pool (so that locality should be reasonable good) and release the whole pool when needed. But usually we can just cache ETCI objects. The patch gives ~4% speed up in event *dispatch*. I posted the patch to tryserver.
Attachment #438714 - Flags: review?(jst)
Attachment #438714 - Flags: review?(jst) → review+
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: