Hi,
Could you please review this very light fix:
bug: https://bugs.openjdk.java.net/browse/JDK-8131783
webrev: http://cr.openjdk.java.net/~olagneau/8131783/webrev.00/
We first quarantine the test.
We will then add tracing information to help diagnosing the problem
Hey folks,
os::current_thread_id on Linux now maps to pthread_self. The problem with
pthread_self is that it only makes sense in the context of the running
process. When it is written out to the log (as it is in several places),
there really isn't a way (AFAICT) for the user to map it back to
Summary: Need to get source_file_name from the_class's constant pool not
previous version constant pool
open webrev at http://cr.openjdk.java.net/~coleenp/8087315.01/
bug link https://bugs.openjdk.java.net/browse/JDK-8087315
Tested with added test (yay!), RBT (remote build and test),
Hi,
Could I please have a review of this fix.
bug: https://bugs.openjdk.java.net/browse/JDK-8132094
webrev: http://cr.openjdk.java.net/~ykantser/8132094/webrev.00
Thanks,
Katja
Example of where this would help:
Current output of SIGQUIT:
C1 CompilerThread3 #8 daemon prio=9 os_prio=0 tid=0x7f0890247800
nid=0x2e3d waiting on condition [0x]
java.lang.Thread.State: RUNNABLE
C2 CompilerThread2 #7 daemon prio=9 os_prio=0 tid=0x7f0890245000
Jeremy,
Please take in account that gettid() returns a value that unique for
the *running threads* across a system.
I.e. for long running java process gettid() can return a value that was
previously assigned to one of dead threads.
-Dmitry
On 2015-07-22 21:22, Jeremy Manson wrote:
Hey folks,
On 23/07/2015 8:01 AM, Jeremy Manson wrote:
Based on the feedback, this seems to be a good idea, approximately.
Coleen would have sponsored, but she's going on vacation. Anyone else
feel like sponsoring?
Hold up a minute! :) There are different notions of native thread id
that exist. First
Dan, Thank you again for the quick code review.
On 7/22/15 8:57 PM, Daniel D. Daugherty wrote:
On 7/22/15 11:22 AM, Coleen Phillimore wrote:
Summary: Need to get source_file_name from the_class's constant pool
not previous version constant pool
open webrev at
Hey David,
Thanks for the offer of sponsorship. My goal here is really to make the
log output on Linux usable. I want to be able to map the log output back
to an actual thread. I don't think it really matters to users if the API
consistently means kernel thread ID or threading API thread ID,
On 23/07/2015 3:15 PM, Jeremy Manson wrote:
Hey David,
Thanks for the offer of sponsorship. My goal here is really to make the
log output on Linux usable. I want to be able to map the log output
back to an actual thread. I don't think it really matters to users if
the API consistently means
Based on the feedback, this seems to be a good idea, approximately. Coleen
would have sponsored, but she's going on vacation. Anyone else feel like
sponsoring?
Jeremy
On Wed, Jul 22, 2015 at 11:22 AM, Jeremy Manson jeremyman...@google.com
wrote:
Hey folks,
os::current_thread_id on Linux
Am 22.07.2015 um 20:52 schrieb Jeremy Manson:
Example of where this would help:
Current output of SIGQUIT:
C1 CompilerThread3 #8 daemon prio=9 os_prio=0 tid=0x7f0890247800
nid=0x2e3d waiting on condition [0x]
java.lang.Thread.State: RUNNABLE
C2 CompilerThread2 #7
It would be nice if even internal hotspot APIs had clear specs. All we have
now is:
// thread id on Linux/64bit is 64bit, on Windows and Solaris, it's 32bit
static intx current_thread_id();
pthread_self() is documented as being an opaque value that cannot
necessarily be stored in a native
Hi Olivier,
The fix is good.
Thanks,
Serguei
On 7/22/15 9:25 AM, olivier.lagn...@oracle.com wrote:
Hi,
Could you please review this very light fix:
bug: https://bugs.openjdk.java.net/browse/JDK-8131783
webrev: http://cr.openjdk.java.net/~olagneau/8131783/webrev.00/
We first quarantine the
On 7/22/15 11:22 AM, Coleen Phillimore wrote:
Summary: Need to get source_file_name from the_class's constant pool
not previous version constant pool
open webrev at http://cr.openjdk.java.net/~coleenp/8087315.01/
src/share/vm/classfile/javaClasses.cpp
So in the new code, you avoid the
15 matches
Mail list logo