Re: [rvm-research] submit my patch, with a new GC insice, and some bug fixes

[Steve B. <Ste...@an...>]

Hi Da feng,

Thanks for your contribution.

Can I ask you to please do a few things?

1.   You say you are making 4 contributions (3 fixes and one new  
GC).   Please submit each of the fixes as separate patches and submit  
the GC as one (or if appropriate, more) patch.

2.  Please ensure that the code passes pre-commit before submitting.

3.  Can you please tell us about this new GC?   We very much welcome  
contributions of new GCs, but any new GC implies a significant  
maintenance burden over time, so it is very important that the new GC  
have a compelling reason for inclusion (is it much faster than  
existing ones?  does it work in some special context that other ones  
cannot?  is there something else particularly compelling about it?).    
Generally the best place for research contributions (such as a new GC)  
is the research tracker.

Thanks again for taking the time to contribute your work.

Cheers,

--Steve

On 27/03/2010, at 6:52 PM, Da Feng wrote:

> 2977410	  bug fixes and a new GC	  Open	  2010-03-27	  nobody	   
> vondart	  None	5
> It is tested on amd x86_64 platform, so some pre-commit test seems  
> to fail.
>
>
> DaFENG
> Coder
> Telecommunication && Network Industry
> Pudong
> Shanghai
> China
>
>
> ------------------------------------------------------------------------------
> Download Intel® Parallel Studio Eval
> Try the new software tools for yourself. Speed compiling, find bugs
> proactively, and fine-tune applications for parallel performance.
> See why Intel Parallel Studio got high marks during beta.
> http://p.sf.net/sfu/intel-sw-dev_______________________________________________
> Jikesrvm-researchers mailing list
> Jik...@li...
> https://lists.sourceforge.net/lists/listinfo/jikesrvm-researchers