Details
-
Bug
-
Resolution: Fixed
-
P4
-
11, 13, 14
-
b07
Backports
Issue | Fix Version | Assignee | Priority | Status | Resolution | Resolved In Build |
---|---|---|---|---|---|---|
JDK-8230534 | 13.0.2 | Ralf Schmelter | P4 | Resolved | Fixed | b02 |
JDK-8246781 | 11.0.9-oracle | Ralf Schmelter | P4 | Resolved | Fixed | b01 |
JDK-8231241 | 11.0.6 | Ralf Schmelter | P4 | Resolved | Fixed | b01 |
Description
The Perf::attach() implementation on all operating systems checks, if the perf file has a sensible size (not 0 and a multiple of the page size). If this is not the case, it throws a java.lang.Exception.
This seems to be the wrong type of exception, especially because the Java method jdk.internal.perf.Perf.attach() does not declare it.
Instead a java.io.IOException message should be thrown, since it is declared to be thrown by the Java attach() method already and it makes sense semantically.
Note that this is usually only a temporary situation, after the JVM created the perf file. Barring an unfortunate crash, it will get the correct size shortly afterwards. But it might be one cause for the sporadic hangs of the MonitorVmStartTerminate jtreg test.
This seems to be the wrong type of exception, especially because the Java method jdk.internal.perf.Perf.attach() does not declare it.
Instead a java.io.IOException message should be thrown, since it is declared to be thrown by the Java attach() method already and it makes sense semantically.
Note that this is usually only a temporary situation, after the JVM created the perf file. Barring an unfortunate crash, it will get the correct size shortly afterwards. But it might be one cause for the sporadic hangs of the MonitorVmStartTerminate jtreg test.
Attachments
Issue Links
- backported by
-
JDK-8230534 Perf::attach() should not throw a java.lang.Exception
-
- Resolved
-
-
JDK-8231241 Perf::attach() should not throw a java.lang.Exception
-
- Resolved
-
-
JDK-8246781 Perf::attach() should not throw a java.lang.Exception
-
- Resolved
-