[PATCH 0 of 4] Add support for dtrace compatible sdt probes on GNU/Linux
Coleen Phillmore
coleen.phillimore at oracle.com
Fri Sep 28 07:46:20 PDT 2012
Okay, thank for the clarification. I'll add it back. Sorry!
Coleen
On 9/28/2012 9:54 AM, Mark Wielaard wrote:
> On Fri, 2012-09-28 at 09:23 -0400, Coleen Phillimore wrote:
>> I have prepared a changeset against hotspot-rt with your patch and
>> have tested it against solaris/sparc dtrace. It's built against bsd
>> also. It seems fine. I will send out a request for review with you as
>> contributor shortly.
> Thanks a lot for running the tests!
>
>> The only process issue I think wasn't correct is that I don't think
>> you are supposed to change the copyright header to include RedHat.
> No, that is correct. The work was done for Red Hat, I don't hold
> copyright myself on the changes. The copyright header should list all
> copyright holders. That is how it is done with other contributions also.
> Just grep through the sources and you will see. You are not supposed to
> remove copyright notices only add to them:
> http://mail.openjdk.java.net/pipermail/jdk7-dev/2009-June/000716.html
>
> Thanks,
>
> Mark
More information about the hotspot-dev
mailing list