RFR (T): 8212974: Update RS Skipped cards uses wrong enum to register to phase
Kim Barrett
kim.barrett at oracle.com
Fri Oct 26 17:37:12 UTC 2018
> On Oct 25, 2018, at 11:35 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>
> Hi all,
>
> can I have a review for this trivial change that fixes a benign typo?
>
> During initialization of the phase time records the code uses the wrong
> enum to register the "Skipped Cards" to the "Update RS" logging
> structure.
>
> I.e.
> _gc_par_phases[UpdateRS]-
>> link_thread_work_items(_update_rs_skipped_cards, ScanRSSkippedCards);
>
> which should be
>
> _gc_par_phases[UpdateRS]-
>> link_thread_work_items(_update_rs_skipped_cards,
> UpdateRSSkippedCards);
>
> Since the value of ScanRSSkippedCards equals UpdateRSSkippedCards there
> is no harm.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8212974
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8212974/webrev/
> Testing:
> local compilation; there is no way to create a test for that for above
> reasons.
>
> Thanks,
> Thomas
Looks good, and trivial.
More information about the hotspot-gc-dev
mailing list