Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New element is added in vgc:mem-info #7386

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

LinHu2016
Copy link
Contributor

@LinHu2016 LinHu2016 commented Jun 18, 2024

new element "vgc:continuation-objects" is added in vgc:mem-info,
the element with two integer addributes "total" and "started"

Signed-off-by: hulin linhu@ca.ibm.com

@LinHu2016
Copy link
Contributor Author

@dmitripivkine @amicic please review the changes, Thanks

@@ -452,6 +454,11 @@ xmlns:vgc="http://www.ibm.com/j9/verbosegc">
<attribute name="classloader" type="integer" use="required" />
</complexType>

<complexType name="continuation-objects">
<attribute name="alive" type="integer" use="required" />
Copy link
Contributor

Choose a reason for hiding this comment

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

it's almost given that we are talking about live ones (especially since this will be at the end of GC)
perhaps simpler is just 'total'

@dmitripivkine, opinion?

Copy link
Contributor

Choose a reason for hiding this comment

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

Update the comment, too. Still refers to 'alive'.

new element "vgc:continuation-objects" is added in vgc:mem-info,
the element with two integer addributes "total" and "started"

Signed-off-by: hulin <linhu@ca.ibm.com>
@LinHu2016 LinHu2016 force-pushed the livecontinuationInVerbosegc branch from 40b2f62 to 32aabf4 Compare June 19, 2024 15:28
@@ -452,6 +454,11 @@ xmlns:vgc="http://www.ibm.com/j9/verbosegc">
<attribute name="classloader" type="integer" use="required" />
</complexType>

<complexType name="continuation-objects">
<attribute name="total" type="integer" use="required" />
<attribute name="started" type="integer" use="required" />
Copy link
Contributor

Choose a reason for hiding this comment

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

started probably means only active (thus excluding those not only started by also ended).
not sure if active is the right term though (as per specs)
@babsingh @fengxue-IS, are you ok with 'started' or have some other preference?

Copy link
Contributor

@babsingh babsingh Jun 19, 2024

Choose a reason for hiding this comment

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

Yes. In the context of Continuations, STARTED looks good; it corresponds to the Continuation state where a stack is always populated.

@amicic
Copy link
Contributor

amicic commented Jun 19, 2024

@babsingh please, proceed with the final review

@babsingh
Copy link
Contributor

jenkins build all

@amicic
Copy link
Contributor

amicic commented Jun 21, 2024

just a frendly reminder - the testing finished

@babsingh babsingh merged commit 5d2a72e into eclipse:master Jun 21, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants