nsSegmentedBuffer::AppendNewSegment allocs 291 4096 byte buffers on startup

RESOLVED WONTFIX

Status

()

P3
normal
RESOLVED WONTFIX
17 years ago
17 years ago

People

(Reporter: dp, Assigned: darin.moz)

Tracking

Trunk
mozilla0.9.7
x86
Windows 2000
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

17 years ago
Totally about 1.1MB of 4k allocations happen here.
(Assignee)

Updated

17 years ago
Status: NEW → ASSIGNED
Priority: -- → P3
Target Milestone: --- → mozilla0.9.7
(Assignee)

Comment 1

17 years ago
i'm not measuring any real cost to these allocations.  on my linux box, i
reduced the amount of memory linux thinks i have down to 64M.  i tested a patch
that adds a cache of buffers 4096 bytes in size.  i varied the number of cached
buffers with an environment variable.  with different settings of this
environment variable, i noticed no measurable variation in startup time.

in other words, when it comes to 4k buffer allocations, i don't think you can
beat the system allocator.  of course, it'd be interesting to try this on windows.
(Assignee)

Comment 2

17 years ago
Created attachment 60907 [details] [diff] [review]
test patch
(Reporter)

Comment 3

17 years ago
I will try the patch on windows. 
(Assignee)

Comment 4

17 years ago
looks like this patch isn't helping :(

marking WONTFIX
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.