Quantcast

RFR (S): 8176885: Clear Claimed Marks log message at wrong location in the log

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

RFR (S): 8176885: Clear Claimed Marks log message at wrong location in the log

Thomas Schatzl
Hi all,

  can I have quick reviews for this change to the order of the "Clear
Claimed Marks" log message?

Two issues: it's at the wrong location, printed in the "Post Evacuate
Collection Set" section as it is executed during the "Evacuate
Collection Set" phase, and in the middle of the "(Parallel) Redirty"
log output.

Second, this task should actually be part of the "Pre Evacuate
Collection Set" phase, as the "Evacuate Collection Set" phase should
only contain the bare (parallel) evacuation itself.

I moved the log message and the actual code to the "pre evacuate
collection set" phase.

CR:
https://bugs.openjdk.java.net/browse/JDK-8176885
Webrev:
http://cr.openjdk.java.net/~tschatzl/8176885/webrev/
Testing:
local testing, inspecting log

Thanks,
  Thomas

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: RFR (S): 8176885: Clear Claimed Marks log message at wrong location in the log

Thomas Schatzl
Hi,

  could someone have a look at this "move some code around" change?

Thanks,
  Thomas

On Thu, 2017-03-16 at 18:53 +0100, Thomas Schatzl wrote:

> Hi all,
>
>   can I have quick reviews for this change to the order of the "Clear
> Claimed Marks" log message?
>
> Two issues: it's at the wrong location, printed in the "Post Evacuate
> Collection Set" section as it is executed during the "Evacuate
> Collection Set" phase, and in the middle of the "(Parallel) Redirty"
> log output.
>
> Second, this task should actually be part of the "Pre Evacuate
> Collection Set" phase, as the "Evacuate Collection Set" phase should
> only contain the bare (parallel) evacuation itself.
>
> I moved the log message and the actual code to the "pre evacuate
> collection set" phase.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8176885
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8176885/webrev/
> Testing:
> local testing, inspecting log
>
> Thanks,
>   Thomas
>
Loading...