JDK-8223665 : SA: debugd options should follow jhsdb style
  • Type: Bug
  • Component: hotspot
  • Sub-Component: svc-agent
  • Affected Version: 9,10,11,12,13
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2019-05-10
  • Updated: 2020-06-24
  • Resolved: 2019-05-17
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 11 JDK 13
11.0.4Fixed 13 b22Fixed
Related Reports
Blocks :  
JDK-8163805 - hotspot/test/serviceability/sa/sadebugd/SADebugDTest.java failed with timed out
Blocks :  
JDK-8247945 - SA tools not providing option to connect to debug server
CSR :  
JDK-8223666 - SA: debugd options should follow jhsdb style
Relates :  
JDK-8207048 - jhsdb debugd cannot specify a port number
Relates :  
JDK-8160817 - Add jsadebugd functionality to jhsdb
Description
Proposed by Osamu Sakamoto <sakamoto.osamu@nttcom.co.jp>
http://mail.openjdk.java.net/pipermail/serviceability-dev/2019-May/027975.html

I want to use `jhsdb debugd` on my laptop.
However debugd mode has different options from other modes.
I think debugd should have same options like other modes.

For example, `jhsdb debugd <pid>` should be `jhsdb debugd --pid <pid>`.
Also I added `--serverid` option for serverid.

I attached a patch for this enhancement.
This patch passes serviceability/sa jtreg tests.

Testcase for debugd is available as serviceability/sa/sadebugd/SADebugDTest.java .
However it has been disabled by JDK-8163805.
It will be fixed by Yasumasa Suenaga (ysuenaga) after my proposal has been merged.
Comments
Fix Request for 11ui: `jhsdb debugd` introduced in JDK 9. However it makes confusing the user because its command line options are different from the others. This change can be tested with serviceability/sa/sadebugd/SADebugDTest, but we need it to merge JDK-8163805 at the same time. So I also send fix request of JDK-8163805 for 11u.
18-05-2019
Review thread: http://mail.openjdk.java.net/pipermail/serviceability-dev/2019-May/027975.html
10-05-2019