10 RFR: 8175221: Cleanup DirtyCardQueueSet::concatenate_log
Kim Barrett
kim.barrett at oracle.com
Fri Mar 3 09:58:46 UTC 2017
Need a second reviewer.
> On Feb 19, 2017, at 4:38 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
>
> Please review this simplification of concatenate_log and removal of
> some thereby unused public functions from DirtyCardQueue.
>
> We change concatenate_log to call flush, rather than inlining more or
> less equivalent code. The flush is conditional on the queue
> containing any data, as we prefer to leave an empty buffer in place in
> the queue when concatenating.
>
> We also changed flush to support this; flush was a nop when the queue
> is "permanent", which is not what we want for concatenate_log. This
> behavior was because flush was called by the queue destructor, and
> performing a flush when destroying the shared queue for a set was
> problematic. However, JDK-8048949 changed things so that only the
> DirtyCardQueue destructor called flush, and only if the queue is
> non-permanent. So the permanent check in flush is no longer needed,
> and removing it makes it suitable for use by concatenate_log.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8175221
>
> Webrev:
> http://cr.openjdk.java.net/~kbarrett/8175221/hotspot.00/
>
> Testing:
> jprt
More information about the hotspot-gc-dev
mailing list