Skip to content

Do not send SVM map to JITServer clients - #17593

Merged
mpirvu merged 1 commit into
eclipse-openj9:masterfrom
cjjdespres:no-map-deserialization
Jun 15, 2023
Merged

Do not send SVM map to JITServer clients#17593
mpirvu merged 1 commit into
eclipse-openj9:masterfrom
cjjdespres:no-map-deserialization

Conversation

@cjjdespres

Copy link
Copy Markdown
Contributor

The _valueToSymbolMap generated at the JITServer itself is not useful to clients. It is used in the creation of relocation records during code generation and is unused afterward, including during method relocation.

The _valueToSymbolMap generated at the JITServer itself is not useful
to clients. It is used in the creation of relocation records during code
generation and is unused afterward, including during method relocation.

Signed-off-by: Christian Despres <despresc@ibm.com>
@cjjdespres

Copy link
Copy Markdown
Contributor Author

Attn @mpirvu. My usual benchmark runs didn't turn up any problems after I removed the value-to-symbol map serialization and deserialization. This PR removes it entirely, but I could also add an option to enable or disable this serialization instead, for something more conservative.

@mpirvu mpirvu self-assigned this Jun 14, 2023
@mpirvu mpirvu added the comp:jitserver Artifacts related to JIT-as-a-Service project label Jun 14, 2023

@mpirvu mpirvu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mpirvu

mpirvu commented Jun 14, 2023

Copy link
Copy Markdown
Contributor

jenkins test sanity plinuxjit,xlinuxjit,zlinuxjit jdk17

@mpirvu
mpirvu merged commit 5bb8dc2 into eclipse-openj9:master Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp:jitserver Artifacts related to JIT-as-a-Service project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants