On Fri, 22 Jan 2021 14:21:52 GMT, Roland Westrelin wrote:
> I noticed that the SA's dumpreplaydata command fails with:
>
> java.lang.AssertionError: CLHSDB wasn't run successfully: Opening core file,
> please wait...
> hsdb> Exception in thread "main" ja
tem because core files are
> handled by systemd (I'm running some recent version of fedora). I
> suppose, the system can be configured differently but having the test
> work out the box is nice. I extended the test case to handle that.
>
> With the improved test, there are a
On Thu, 4 Feb 2021 18:25:45 GMT, Igor Ignatyev wrote:
>> Roland Westrelin has updated the pull request with a new target base due to
>> a merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request cont
tem because core files are
> handled by systemd (I'm running some recent version of fedora). I
> suppose, the system can be configured differently but having the test
> work out the box is nice. I extended the test case to handle that.
>
> With the improved test, there are a
On Wed, 3 Feb 2021 18:51:57 GMT, Igor Ignatyev wrote:
>> Thanks for reviewing this. I did not store the pid in the OutputAnalyzer
>> because it doesn't seem to belong there as it has nothing to do with the
>> text output of a test. But if you think that's ok. that's fine with me too.
>> Do you
On Wed, 3 Feb 2021 05:45:02 GMT, Igor Ignatyev wrote:
>> Roland Westrelin has updated the pull request with a new target base due to
>> a merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contain
On Fri, 22 Jan 2021 17:44:34 GMT, Vladimir Kozlov wrote:
>> Roland Westrelin has updated the pull request with a new target base due to
>> a merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contain
On Mon, 25 Jan 2021 19:43:56 GMT, Chris Plummer wrote:
>> Roland Westrelin has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> use CoreUtils
>
> test/lib/jdk/test/lib/util/CoreUtils.java line 193:
>
tem because core files are
> handled by systemd (I'm running some recent version of fedora). I
> suppose, the system can be configured differently but having the test
> work out the box is nice. I extended the test case to handle that.
>
> With the improved test, there are a
tem because core files are
> handled by systemd (I'm running some recent version of fedora). I
> suppose, the system can be configured differently but having the test
> work out the box is nice. I extended the test case to handle that.
>
> With the improved test, there are a
On Fri, 22 Jan 2021 21:28:33 GMT, Chris Plummer wrote:
>> Roland Westrelin has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> use CoreUtils
>
> src/jdk.hotspot.agent/share/classes/sun/jvm/hotspot/oops/Profile
I noticed that the SA's dumpreplaydata command fails with:
java.lang.AssertionError: CLHSDB wasn't run successfully: Opening core file,
please wait...
hsdb> Exception in thread "main" java.lang.InternalError: ciMetadata does not
appear to be polymorphic
with a simple test program. This happens
> http://cr.openjdk.java.net/~rkennke/shenandoah-upstream/05/
C2 changes look good to me.
Roland.
Thanks for the reviews, Vladimir, Dmitry and Staffan.
Roland.
http://cr.openjdk.java.net/~roland/8077832/webrev.00/
I found 3 locations where the SA code is out of sync with the hotspot code.
Roland.
> So it isn't default methods that are the problem but bridge methods?
Yes, it looks like it.
Roland.
Hi David,
> I wouldn't say it "supports default methods" rather it ignores them so they
> don't trigger exceptions. I'm still puzzling over why default methods have a
> zero _codeIndex as that seems to be the real cause of the trouble. ??
I took another look and it turns out the test on _codeIn
Hi Chris,
> Since you’re saying it only fixes it partially should we file a followup bug
> and maybe leave a comment behind?
Actually I don’t know if it’s partial or not. I don’t understand default
methods enough to be sure.
Roland.
buildreplayjars sometimes fail with an exception and I think that's
caused by the lack of support for default methods when the SA dumps
classes. This change fixes it at least partially. It may not be the
cleanest way to support default methods but it already proved useful
when investigating a comp
Forwarding to serviceability alias where this question belongs I think.
Begin forwarded message:
> From: Brendan Gregg
> Subject: system profilers and incomplete stacks
> Date: June 12, 2014 at 7:15:54 PM GMT+2
> To: hotspot-compiler-...@openjdk.java.net
>
> G'Day,
>
> Is there a way to run ho
Hi Staffan,
> Please review this fix for a problem with compiler/ciReplay/TestSA.sh. It
> fails with an exception because SA can’t find the values for some the enum
> entries in Deoptimization::DeoptReason since they are missing from
> vmStructs.cpp. The patch below adds them.
>
> bug: https:/
Thanks for the new webrev and explanations.
ciReplay.hpp
59 // Replay data file replay_pid%p.log is also created when VM crushes
when VM crashes
That looks good to me.
Roland.
On Jan 8, 2014, at 5:02 AM, Vladimir Kozlov wrote:
> Thank you, Roland
>
> On 1/7/14 2:39 AM, Roland
506 }
507 }
Roland.
On Jan 7, 2014, at 11:39 AM, Roland Westrelin
wrote:
>> http://cr.openjdk.java.net/~kvn/8028468/webrev/
>
> Should the agent/doc/cireplay.html be updated? Is there another doc on how to
> use the replay support somewhere (wiki)?
>
> ciRep
> http://cr.openjdk.java.net/~kvn/8028468/webrev/
Should the agent/doc/cireplay.html be updated? Is there another doc on how to
use the replay support somewhere (wiki)?
ciReplay.cpp
typos:
// Use pointer because we may need to retirn inline records
// Replay Inlinig
vmError.cpp
typo:
// Do not
Thanks for the review Vladimir & Christian
Roland.
em dictionary.
> + if (notice) {
> + notice_modification();
>
> Thanks,
> Vladimir
>
> On 12/9/13 9:03 AM, Roland Westrelin wrote:
>> http://cr.openjdk.java.net/~roland/8029383/webrev.00/
>>
>> When an anonymous class is defined th
It looks good to me.
Roland.
>Please give your comments of the changes about
>6830717: replay of compilations would help with debugging.
> http://monaco.sfbay.sun.com/detail.jsf?cr=6830717
It looks good to me.
Roland.
Changeset: 2fecca53a2c6
Author:roland
Date: 2011-09-07 14:15 +0200
URL: http://hg.openjdk.java.net/hsx/hotspot-rt/hotspot/rev/2fecca53a2c6
7085012: ARM: com/sun/jdi/PopSynchronousTest.java still fails
Summary: InterpreterRuntime::popframe_move_outgoing_args() is required for the
AR
29 matches
Mail list logo
