-
Notifications
You must be signed in to change notification settings - Fork 0
Independent object counting from GC #4
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
base: test/shen-aggregate-closure
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks mostly good.
Left a few comments.
Sorry for the delay in getting to this.
| "0 will allow back to back young collections to run during old " \ | ||
| "collections.") \ | ||
| \ | ||
| product(uintx, ShenandoahObjectCountInterval, 0, \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"JFR" somewhere in the name so it's clear.
I wouldn't use "0" to mean disabled, since the enabling or disabling is controlled separately by whether the event is enabled or disabled.
In any case, I suppose your plan is to get this value from the JFR event in the future, so may be this semantics can be ignored (i.e. kept) for now. I wouldn't spend time on it.
| * & vm.opt.ExplicitGCInvokesConcurrent != false | ||
| * @library /test/lib /test/jdk | ||
| * @run main/othervm -XX:+UnlockExperimentalVMOptions -XX:+UseShenandoahGC -XX:+ExplicitGCInvokesConcurrent -XX:MarkSweepDeadRatio=0 -XX:-UseCompressedOops -XX:-UseCompressedClassPointers -XX:+IgnoreUnrecognizedVMOptions jdk.jfr.event.gc.objectcount.TestObjectCountEventWithShenandoah | ||
| * @run main/othervm -XX:+UnlockExperimentalVMOptions -XX:+UseShenandoahGC -XX:ShenandoahObjectCountInterval=300 -XX:+ExplicitGCInvokesConcurrent -XX:MarkSweepDeadRatio=0 -XX:-UseCompressedOops -XX:-UseCompressedClassPointers -XX:+IgnoreUnrecognizedVMOptions jdk.jfr.event.gc.objectcount.TestObjectCountEventWithShenandoah |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In order to get these events, you might need to sleep for a sufficient period of time so that the (non-gc) object count event is produced.
…ntGC, and redefined final mark
No description provided.