RFR: 8154343: Make SATB related code available to other GCs

Jon Masamitsu jon.masamitsu at oracle.com
Mon Apr 18 17:50:19 UTC 2016



On 04/18/2016 04:50 AM, Roman Kennke wrote:
> Am Freitag, den 15.04.2016, 14:30 -0700 schrieb Jon Masamitsu:
>> On 4/15/2016 12:17 PM, Roman Kennke wrote:
>>> I just noticed that I made the patch against the jdk9 repository,
>>> instead of jdk9-dev. There was a small change to make to
>>> require_marking()/retain_entry(). The updated webrev is here:
>>>
>>> http://cr.openjdk.java.net/~rkennke/jdk-8154343/webrev.01/
>>>
>>> And I also forgot to post a link to the jira entry:
>>> https://bugs.openjdk.java.net/browse/JDK-8154343
>>>
>>> Please let me know what you think, and if I can commit it. Thanks!
>> Roman,
>>
>> You're aware of jprt?  The tool that Oracle uses to build hotspot
>> patches on all platforms,
>> do a minimum amount of testing, and then does the commit?  For shared
>> code
>> such as in your patch, it is always used.  Sorry if I'm stating old
>> news
>> but I wasn't
>> sure what to make of your request to commit (yourself).
> Yes, I am aware, but wasn't sure if it's still used or if external
> contributors need to worry about it, because I heard conflicting
> stories about the possibility to commit patches ourself.

If a change is to platform specific code and not shared (I'm thinking
Zero and Shark), then the pushes can done by the external contributor.
Please don't quote that as definitive.   It was stated more precisely
somewhere.  But your change is to share code so does not qualify.

>
> So how would I go about getting a sponsor for a patch like this?

When it's been reviewed (for anything significant by 2 reviewers, one
of which needs to have Reviewer status as defined by the Bylaws), ask
for a sponsor.

Jon
>
> Roman
>



More information about the hotspot-gc-dev mailing list