Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add memory consuption results to benchmarks #42

Open
GoogleCodeExporter opened this issue Mar 24, 2015 · 7 comments
Open

Add memory consuption results to benchmarks #42

GoogleCodeExporter opened this issue Mar 24, 2015 · 7 comments

Comments

@GoogleCodeExporter
Copy link

Apart from measuring time, we should also measure memory usage. This would be 
really good for the journal version of the paper also.

Original issue reported on code.google.com by marianopeck on 16 Jun 2011 at 2:36

@GoogleCodeExporter
Copy link
Author

And file size!!! :)

Original comment by marianopeck on 16 Jun 2011 at 6:00

@GoogleCodeExporter
Copy link
Author

file size --> http://code.google.com/p/fuel/issues/detail?id=43

Original comment by tinchod...@gmail.com on 16 Jun 2011 at 6:03

@GoogleCodeExporter
Copy link
Author

For memory usage we need to take into account GC....it is complicated

Original comment by marianopeck on 18 Jun 2011 at 6:58

1 similar comment
@GoogleCodeExporter
Copy link
Author

For memory usage we need to take into account GC....it is complicated

Original comment by marianopeck on 18 Jun 2011 at 6:58

@GoogleCodeExporter
Copy link
Author


We could add methods to MessageTally to access gc information, since now it 
seems only accessible through printing them with:


reportGCStatsOn: str
    | oldSpaceEnd youngSpaceEnd memoryEnd fullGCs fullGCTime incrGCs incrGCTime tenureCount upTime rootOverflows |
    upTime := time.
    oldSpaceEnd         := gcStats at: 1.
    youngSpaceEnd       := gcStats at: 2.
    memoryEnd           := gcStats at: 3.
    fullGCs             := gcStats at: 7.
    fullGCTime          := gcStats at: 8.
    incrGCs             := gcStats at: 9.
    incrGCTime          := gcStats at: 10.
    tenureCount         := gcStats at: 11.
    rootOverflows       := gcStats at: 22.

    str cr.
    str nextPutAll: '**Memory**'; cr.
    str nextPutAll: '   old         ';
        nextPutAll: oldSpaceEnd asStringWithCommasSigned; nextPutAll: ' bytes'; cr.
    str nextPutAll: '   young       ';
        nextPutAll: (youngSpaceEnd - oldSpaceEnd) asStringWithCommasSigned; nextPutAll: ' bytes'; cr.
    str nextPutAll: '   used        ';
        nextPutAll: youngSpaceEnd asStringWithCommasSigned; nextPutAll: ' bytes'; cr.
    str nextPutAll: '   free        ';
        nextPutAll: (memoryEnd - youngSpaceEnd) asStringWithCommasSigned; nextPutAll: ' bytes'; cr.

    str cr.
    str nextPutAll: '**GCs**'; cr.
    str nextPutAll: '   full            ';
        print: fullGCs; nextPutAll: ' totalling '; nextPutAll: fullGCTime asStringWithCommas; nextPutAll: 'ms (';
        print: ((fullGCTime / upTime * 100) roundTo: 1.0);
        nextPutAll: '% uptime)'.
    fullGCs = 0 ifFalse:
        [str    nextPutAll: ', avg '; print: ((fullGCTime / fullGCs) roundTo: 1.0); nextPutAll: 'ms'].
    str cr.
    str nextPutAll: '   incr        ';
        print: incrGCs; nextPutAll: ' totalling '; nextPutAll: incrGCTime asStringWithCommas; nextPutAll: 'ms (';
        print: ((incrGCTime / upTime * 100) roundTo: 1.0);
        nextPutAll: '% uptime)'.
    incrGCs = 0 ifFalse:
        [str nextPutAll:', avg '; print: ((incrGCTime / incrGCs) roundTo: 1.0); nextPutAll: 'ms'].
    str cr.
    str nextPutAll: '   tenures     ';
        nextPutAll: tenureCount asStringWithCommas.
    tenureCount = 0 ifFalse:
        [str nextPutAll: ' (avg '; print: (incrGCs / tenureCount) asInteger; nextPutAll: ' GCs/tenure)'].
    str cr.
    str nextPutAll: '   root table  ';
        nextPutAll: rootOverflows asStringWithCommas; nextPutAll:' overflows'.
    str cr.

Original comment by tinchod...@gmail.com on 14 May 2012 at 5:48

@GoogleCodeExporter
Copy link
Author

After fixing http://code.google.com/p/fuel/issues/detail?id=155 , we should 
also calculate median and deviation of GC stats.

Original comment by tinchod...@gmail.com on 14 May 2012 at 6:06

@stale
Copy link

stale bot commented May 18, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will remain open but will probably not come into focus. If you still think this should receive some attention, leave a comment. Thank you for your contributions.

@stale stale bot added the stale label May 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant