Bug 278057 - Increase max heap size for performance tests
Summary: Increase max heap size for performance tests
Status: RESOLVED FIXED
Alias: None
Product: Platform
Classification: Eclipse Project
Component: Releng (show other bugs)
Version: 3.5   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: 3.6 M2   Edit
Assignee: Kim Moir CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-05-27 10:54 EDT by Oleg Besedin CLA
Modified: 2009-08-07 17:27 EDT (History)
4 users (show)

See Also:


Attachments
patch (1.08 KB, patch)
2009-08-06 17:05 EDT, Kim Moir CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Oleg Besedin CLA 2009-05-27 10:54:43 EDT
Performance tests run with default values for max heap size (which is currently set to 256Mb). 

It seems that performance tests run close to this limit (200 - 250Mb on my computer). This alters results as:
- more time is spend in garbage collections;
- components that use soft references might discard their contents. For instance, extension registry cache will be freed as we get low on memory resulting in substantial slowdown in all operations that use extension registry.

If the heap size is increased to 512Mb, tests take 300 - 350Mb on my computer.

I'd recommend increasing maximum heap size for the performance tests to 512Mb.

(Not sure if this needs to be done separately for the test framework and for the inside-IDE runs.)
Comment 1 Boris Bokowski CLA 2009-05-27 15:15:04 EDT
Can we start by capturing how many times the garbage collector runs during each test (and how much time this took)?
Comment 2 Kim Moir CLA 2009-08-06 17:05:41 EDT
Created attachment 143726 [details]
patch
Comment 3 Frederic Fusier CLA 2009-08-07 03:20:02 EDT
Note that this must be done both in HEAD and baseline stream (perf_35x).

I also wonder if we should not need also increase the Xms value to 512M. I'm afraid that if we let to 256M, then we got some extra time consumed by the VM when it needs some more room for memory and then makes the concerned test time having some strange behavior...
Comment 4 Kim Moir CLA 2009-08-07 09:45:29 EDT
I've released this change to the perf_35x branch as well.
Comment 5 Kim Moir CLA 2009-08-07 17:27:00 EDT
Fixed.