Skip to content
Permalink
Browse files

8247762: [aarch64] Timeout in .../HeapDumpTestWithActiveProcess.java …

…due to inf. loop in AARCH64CurrentFrameGuess.run()

Reviewed-by: cjplummer, aph
  • Loading branch information
Patric Hedlin
Patric Hedlin committed Jun 17, 2020
1 parent d7b097f commit 1d5ec8f36333d578ba9ff92a9fe886db3ace7188
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2003, 2019, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2003, 2020, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2015, 2019, Red Hat Inc.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
@@ -138,7 +138,16 @@ public boolean run(long regionInBytesToSearch) {
setValues(curSP, null, pc);
return true;
}
Frame oldFrame = frame;
frame = frame.sender(map);
if (frame.getSP().lessThanOrEqual(oldFrame.getSP())) {
// Frame points to itself or to a location in the wrong direction.
// Break the loop and move on to next offset.
if (DEBUG) {
System.out.println("CurrentFrameGuess: frame <= oldFrame: " + frame);
}
break;
}
}
} catch (Exception e) {
if (DEBUG) {

0 comments on commit 1d5ec8f

Please sign in to comment.
You can’t perform that action at this time.