RFR (S): 8230611: infinite loop in LogOutputList::wait_until_no_readers()

Kim Barrett kim.barrett at oracle.com
Thu Nov 21 20:37:29 UTC 2019


> On Nov 21, 2019, at 3:24 PM, David Buck <david.buck at oracle.com> wrote:
> 
> Hi Kim!
> 
> Thanks again for helping me to get this fix in shape.
> 
> Once again, I agree with everything you said. I have returned the copy ctor to my first version (using an initializer list), and have modified the copy assignment so that it now exactly matches your number 2 suggestion from your previous response.
> 
> Here is a new (and hopefully final) webrev:
> 
> http://cr.openjdk.java.net/~dbuck/8230611_ver02/
> 
> I am still running the sanity check builds / tests in the background. But seeing how the copy ctor is simply a reversion back to the already thoroughly-tested version from earlier, and that the copy assignment override is dead code in our builds, I do not expect any surprises.
> 
> Does the latest webrev look good? Any other thoughts or feedback?

Looks good.



More information about the hotspot-runtime-dev mailing list