[PATCH] Linux implementation for set_native_thread_name() (original) (raw)

Roman Kennke rkennke at redhat.com
Tue Jul 24 14:10:58 PDT 2012

I think it looks fine.

Great! I posted an updated webrev for this (replaced snprint with jio_snprintf):

http://cr.openjdk.java.net/~rkennke/linux_threadname/webrev.01/

Do you have a bug number for this?

No. (I don't have access to bugster anymore).

Also, how do we verify the result?

I verified it using htop and by inspecting /proc/$PID/task/$TID/stat. Unfortuntately, I couldn't get top or ps to show the thread name, but maybe I'm just stupid (and I did not try very hard).

Also: which of the many repositories do I need to push this to?

Regards, Roman

-- - Keith

On 7/24/2012 3:42 PM, Roman Kennke wrote: > Hello, > > The following patch implements setnativethreadname() for Linux, which > was so far unimplemented. The name needs to be truncated to 15 chars, > otherwise it would not be accepted by pthreadsetnamenp() at all. > > http://cr.openjdk.java.net/~rkennke/linuxthreadname/webrev.00/ > > This is my first patch to hotspot ever, so please let me know when I did > anything fundamental wrong. I have another patch in the pipeline for > implementing the other 2 missing methods in the same source file > (distributeprocesses() and bindtoprocessor() ). > > What do you think? > > Kind regards, Roman > >

More information about the hotspot-dev mailing list