Add !walkj9hashtable DDR interactive command#23526
Add !walkj9hashtable DDR interactive command#23526keithc-ca merged 1 commit intoeclipse-openj9:masterfrom
Conversation
9f76b38 to
5eedba1
Compare
|
Look at https://github.com/eclipse-openj9/openj9/blob/master/CONTRIBUTING.md for how to format commit messages. The PR description should match the commit message. You can add comments to the PR for other things like questions, concerns, etc. |
|
One thing to make this more usable is to add an optional type parameter. I assume you are just printing out something like: But it would be better if the type was included, for example: |
|
Another thing is to test that the data outputted is actually correct. So writing a test that dumps the classes in a classloader ( |
or
|
5eedba1 to
36148cf
Compare
|
Addressed the following in the latest commit:
When Sample output - 1: Inline structs (e.g. Sample output - 2: Pointer entries (e.g. Sample output - 3: No type argument specified:
|
ec75eef to
6b040fd
Compare
keithc-ca
left a comment
There was a problem hiding this comment.
Sorry, I just noticed that my comments were still pending.
e6e789f to
b1f5693
Compare
|
Grinder Tests: Set JDK 8: https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/59348/ |
|
@keithc-ca do you have any further comments |
b1f5693 to
625d7f6
Compare
keithc-ca
left a comment
There was a problem hiding this comment.
I don't see the necessary fixes to HashTable_V1 to properly handle non-inline entries.
This comment was marked as outdated.
This comment was marked as outdated.
625d7f6 to
51b00a0
Compare
de5d87e to
9991e04
Compare
Adds a new DDR extension command that walks all entries of a J9HashTable and prints each entry as a runnable DDR command. The command accepts a single hex address and an optional type argument: !walkj9hashtable <address> [type] Signed-off-by: Tomal Majumder <tomal.majumder@ibm.com>
9991e04 to
670c83e
Compare
|
Jenkins test extended.functional amac jdk25 |
Add !walkj9hashtable DDR command
Adds a new DDR extension command that walks all entries of a
J9HashTable and prints each entry as a runnable DDR command.
The command accepts a single hex address and an optional type
argument:
!walkj9hashtable <address> [type]Issue: #23452