[m-rev.] For review: Implement support for threadscope profiling.

Paul Bone pbone at csse.unimelb.edu.au
Fri Dec 11 09:37:36 AEDT 2009


On Fri, Dec 11, 2009 at 12:18:16AM +1100, Zoltan Somogyi wrote:
> On 10-Dec-2009, Paul Bone <pbone at csse.unimelb.edu.au> wrote:
> > For garbage collection.  We have gc, mps and agc (and none).  I know the
> > compiler won't bootstrap without garbage collection, so I won't test that.  The
> > documentation says that agc isn't supported in low level C grades.  asside from
> > that can I expect agc and mps to work?
> 
> No.
> 
>  The user guide doesn't cover mps, it's
> > not even their in a commented out state.
> 
> That is because Fergus tried it, found it was far slower than Boehm, and we never
> used it since.
> 
> > There appear to be a number of experimental and seldom-used options including
> > --record-term-sizes-as-{words|cells}, rbmm, ssdebug, minimal model grades,
> > ll_debug, (--extend-stacks-when-needed).  Should these be tested.
> 
> extend stacks when needed, yes. The others, no.
> 
> > So far I have these grades to test:
> 
> That is far too many.
> 
> You should test this list
> 
> >     asm_fast.gc
> >     asm_fast.gc.par
> >     asm_fast.gc.par.threadscope
> >     asm_fast.gc.memprof
> >     asm_fast.gc.profdeep
> >     asm_fast.gc.trseg
> >     asm_fast.gc.decldebug
> >     asm_fast.gc.stseg
> >     asm_fast.gc.par.stseg
> >     asm_fast.gc.par.threadscope.stseg
> >     hlc.gc
> >     hlc.gc.par
> >     hlc.gc.par.threadscope
> >     hlc.gc.trseg
> >     hlc.gc.prof

I've added:
    asm_fast.gc.exts
    asm_fast.gc.par.exts
    asm_fast.gc.par.threadscope.exts

and started the tests on goofy, they're running with nice -10.  I'll go do
something else while they run. :-)

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 489 bytes
Desc: Digital signature
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20091211/1513e02a/attachment.sig>


More information about the reviews mailing list