Hi Alex,
It looks good in general.
I have a couple of minor comments.
http://cr.openjdk.java.net/~amenkov/jdk15/RemovingUnixDomainSocket/webrev/test/lib/jdk/test/lib/apps/LingeredApp.java.udiff.html
+// if for a reason the app hangs, we don't
Hi,
I think it would be very convenient for app developers if JVM were able to
create intermediate directories to gc.log file if they do not exist.
I.e.
$ if [ -f logs/gc.log ]; then echo "log file exists"; else echo "log file
does not exist"; fi
log file does not exist
$ if [ -d logs ]; t
Hi Chris,
Please review a new version of the fix[1] that also updates jdk/sun/tools
tests.
Testing: Mach5 tier1-tier7 tests successfully passed.
[1] http://cr.openjdk.java.net/~dtitov/8242009/webrev.03/
[2] ] https://bugs.openjdk.java.net/browse/JDK-8242009
Thank you,
Daniil
On 5/8/20, 12:
Hi Alex,
228 // if for a reason the app hangs, we don't want to
wait test timeout
229 if
(!appProcess.waitFor(Utils.adjustTimeout(appWaitTime), TimeUnit.SECONDS)) {
Can you fix the comment. Maybe:
228 // If the app hangs, we don't want to wait for the t
Hi Daniil,
Looks good.
thanks,
Chris
On 5/11/20 9:43 AM, Daniil Titov wrote:
Hi Chris,
Please review a new version of the fix[1] that also updates jdk/sun/tools
tests.
Testing: Mach5 tier1-tier7 tests successfully passed.
[1] http://cr.openjdk.java.net/~dtitov/8242009/webrev.03/
[2] ] h
Hi Daniil,
Overall looks good. Just one minor thing. In ClassTypeImpl.c and
ObjectReferenceImpl.c I think the following would be more readable with
an if/else:
79 return JNI_FUNC_PTR(env,ExceptionOccurred)(env) ?
AGENT_ERROR_JNI_EXCEPTION
80 : JVMTI_ERROR_NONE;
Also
> On May 8, 2020, at 12:48 PM, Daniel D. Daugherty
> wrote:
>
> On 5/7/20 1:35 AM, Mikael Vidstedt wrote:
>> New webrev here:
>>
>> webrev:
>> http://cr.openjdk.java.net/~mikael/webrevs/8244224/webrev.01/hotspot/open/webrev/
>
> This pretty much says it all:
>
> > Summary of changes:9
Hi Alex,
There is no need to repeat this:
"deploy applications that package an agent with the application,
or use tools that load agents into a
running
application"
I'd suggest to rephrase it to something like:
Hi Daniil,
It looks pretty good in general.
A couple of nits below.
http://cr.openjdk.java.net/~dtitov/8241080/webrev.01/src/jdk.jdwp.agent/share/native/libjdwp/invoker.c.udiff.html
+void *cursor;
+jbyte argumentTag;
+jint argIndex = 0;
+jv
On 11/05/2020 19:21, serguei.spit...@oracle.com wrote:
Hi Alex,
There is no need to repeat this:
"deploy applications thatpackage an agent with the application,
or use tools that load agents into a running application"
I'd suggest to rephrase it to something like:
"Agents can transform
I do not see my message reached the
serviceability-dev mailing list.
Resending...
On 5/11/20 00:31, serguei.spit...@oracle.com wrote:
Hi Alex,
It looks good in general.
I have a couple of minor comments.
Hi Daniil,
In the grand scheme of things, could servicability use the signature
parsing support in HotSpot?
Thanks, Roger
On 5/9/20 12:29 PM, Daniil Titov wrote:
Please review a change[1] that centralizes the signature processing in
serviceability tools to make it capable of being easily e
Hi Yasumasa, Serguei, Chris,
Thank you for the feedback
updated webrev (all suggestions are applied):
http://cr.openjdk.java.net/~amenkov/jdk15/RemovingUnixDomainSocket/webrev.2/
On 05/11/2020 00:31, serguei.spit...@oracle.com wrote:
Hi Alex,
It looks good in general.
I have a couple of minor
BTW, I just found out that due to a fix for JDK-8220295 early last year,
the jdk/sun/tools tests are not run in parallel, so at least atm are not
contributing to any host memory related issues. Still it is best to fix
them now.
Chris
On 5/11/20 10:54 AM, Chris Plummer wrote:
Hi Daniil,
Look
228 // If the app hangs, we don't want to wait for the to
test timeout.
Sorry, there was a typo in my suggestion. Should be "test to", not "to
test".
thanks,
Chris
On 5/11/20 12:53 PM, Alex Menkov wrote:
Hi Yasumasa, Serguei, Chris,
Thank you for the feedback
updated webre
Please hold on with the review.
webrev.2 causes LingeredApp crash.
Looks like need to decrease ThreadBlockInVM scope.
Will send new version after completing test run.
On 05/11/2020 13:52, Chris Plummer wrote:
228 // If the app hangs, we don't want to wait for the to
test timeout
Hi Serguei, Alan,
Updated webrev:
http://cr.openjdk.java.net/~amenkov/jdk15/java_instrument_spec/webrev.2/
--alex
On 05/11/2020 11:52, Alan Bateman wrote:
On 11/05/2020 19:21, serguei.spit...@oracle.com wrote:
Hi Alex,
There is no need to repeat this:
"deploy applications thatpackage an ag
Hi Alex,
LGTM
Thank you for the update!
Serguei
On 5/11/20 14:14, Alex Menkov wrote:
Hi Serguei, Alan,
Updated webrev:
http://cr.openjdk.java.net/~amenkov/jdk15/java_instrument_spec/webrev.2/
--alex
On 05/11/2020 11:52, Alan Bateman wrote:
On 11/05/2020 19:21, serguei.spit...@oracle.com w
Hi Alex,
I'm not sure this update suggested by Yasumasa is right:
536 // avoid deadlock if AttachListener thread is blocked at safepoint
537 ThreadBlockInVM tbivm(JavaThread::current());
538 while (AttachListener::transit_state(AL_INITIALIZING,
539
Hi Serguei,
If I understand correctly, this also should work - when we mark a thread
"_thread_blocked" VM does not need to suspend it at safepoint.
--alex
On 05/11/2020 15:36, serguei.spit...@oracle.com wrote:
Hi Alex,
I'm not sure this update suggested by Yasumasa is right:
536 // avoid d
Hi Alex,
I see the point, thanks.
Serguei
On 5/11/20 17:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a
thread "_thread_blocked" VM does not need to suspend it at safepoint.
--alex
On 05/11/2020 15:36, serguei.spit...@oracle.com wrote:
On 2020/05/12 9:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a thread
"_thread_blocked" VM does not need to suspend it at safepoint.
I think so, but if state would be transit to the other in is_init_trigger(), it
might be affect to incorr
Updated webrev:
http://cr.openjdk.java.net/~amenkov/jdk15/RemovingUnixDomainSocket/webrev.3/
--alex
On 05/11/2020 14:12, Alex Menkov wrote:
Please hold on with the review.
webrev.2 causes LingeredApp crash.
Looks like need to decrease ThreadBlockInVM scope.
Will send new version after complet
Hi Yasumasa,
On 5/11/20 17:08, Yasumasa Suenaga wrote:
On 2020/05/12 9:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a
thread "_thread_blocked" VM does not need to suspend it at safepoint.
I think so, but if state would be transit to the
On 5/11/20 17:17, Alex Menkov wrote:
On 05/11/2020 17:08, Yasumasa Suenaga wrote:
On 2020/05/12 9:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a
thread "_thread_blocked" VM does not need to suspend it at safepoint.
I think so, but if s
On 05/11/2020 17:08, Yasumasa Suenaga wrote:
On 2020/05/12 9:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a
thread "_thread_blocked" VM does not need to suspend it at safepoint.
I think so, but if state would be transit to the other in
On 2020/05/12 9:17, serguei.spit...@oracle.com wrote:
Hi Yasumasa,
On 5/11/20 17:08, Yasumasa Suenaga wrote:
On 2020/05/12 9:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a thread
"_thread_blocked" VM does not need to suspend it at safepo
Hi Alex,
On 2020/05/12 9:17, Alex Menkov wrote:
On 05/11/2020 17:08, Yasumasa Suenaga wrote:
On 2020/05/12 9:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a thread
"_thread_blocked" VM does not need to suspend it at safepoint.
I think
On 5/11/20 17:33, Yasumasa Suenaga wrote:
On 2020/05/12 9:17, serguei.spit...@oracle.com wrote:
Hi Yasumasa,
On 5/11/20 17:08, Yasumasa Suenaga wrote:
On 2020/05/12 9:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a
thread "_thread_blocke
On 05/11/2020 17:21, serguei.spit...@oracle.com wrote:
On 5/11/20 17:17, Alex Menkov wrote:
On 05/11/2020 17:08, Yasumasa Suenaga wrote:
On 2020/05/12 9:01, Alex Menkov wrote:
Hi Serguei,
If I understand correctly, this also should work - when we mark a
thread "_thread_blocked" VM does
This test recently starting running with -Xcheck:jni applied and is
failing, so we want to problem-list it until the issue can be investigated
diff -r bbdcc6741915 test/hotspot/jtreg/ProblemList.txt
--- a/test/hotspot/jtreg/ProblemList.txt
+++ b/test/hotspot/jtreg/ProblemList.txt
@@ -108,6 +108,
LGTM
-- Igor
> On May 11, 2020, at 9:28 PM, David Holmes wrote:
>
> This test recently starting running with -Xcheck:jni applied and is failing,
> so we want to problem-list it until the issue can be investigated
>
> diff -r bbdcc6741915 test/hotspot/jtreg/ProblemList.txt
> --- a/test/hotspot/
Thanks Igor!
David
On 12/05/2020 2:38 pm, Igor Ignatyev wrote:
LGTM
-- Igor
On May 11, 2020, at 9:28 PM, David Holmes wrote:
This test recently starting running with -Xcheck:jni applied and is failing, so
we want to problem-list it until the issue can be investigated
diff -r bbdcc6741915
LGTM++ and trivial.
Thanks,
Serguei
On 5/11/20 21:38, Igor Ignatyev wrote:
LGTM
-- Igor
On May 11, 2020, at 9:28 PM, David Holmes wrote:
This test recently starting running with -Xcheck:jni applied and is failing, so
we want to problem-list it until the issue can be investigated
diff -r b
Thanks Serguei. Already pushed :)
David
On 12/05/2020 3:16 pm, serguei.spit...@oracle.com wrote:
LGTM++ and trivial.
Thanks,
Serguei
On 5/11/20 21:38, Igor Ignatyev wrote:
LGTM
-- Igor
On May 11, 2020, at 9:28 PM, David Holmes
wrote:
This test recently starting running with -Xcheck:jni
35 matches
Mail list logo