RFR: 8073464 - GC workers do not have thread names

David Lindholm david.lindholm at oracle.com
Tue Mar 3 09:02:38 UTC 2015


Hi,

On 2015-02-20 16:01, Thomas Schatzl wrote:
> Hi,
>
> On Fri, 2015-02-20 at 14:39 +0100, David Lindholm wrote:
>> Hi,
>>
>> Please review this small fix in GangWorker:initialize(). This addition
>> sets the native thread names of GangWorker threads, which the GC uses
>> for worker threads. Now these names will be visible when debugging in GDB.
>>
>> Webrev: http://cr.openjdk.java.net/~stefank/0thers/david/8073464/webrev.00/
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8073464
>>
>    please use Thread::set_native_thread_name() instead of the os function
> directly.

Ok.

> Also, wouldn't it be much more useful to set the native thread name for
> all NamedThreads to the name that is passed via set_name() by default?
> This would prevent us from doing this work multiple times.

Ok, this new patch does this for all NamedThreads.

> You might also fix the names themselves for gc owned threads in this CR.
> I do not see the need to split this up in CRs per thread type with each
> CR being a one-line change.

The changes of all the thread names will be handled in a separate 
change. There is a separate bug for this.

I have also changed the copyright year in the changed files, even though 
I think it is a weird process to do this separately for every change.

New webrev: http://cr.openjdk.java.net/~brutisso/8073464/webrev.01/

I have run and passed JPRT on this change.

Thanks,
David
> Thanks,
>    Thomas
>   
>




More information about the hotspot-gc-dev mailing list