On 06/27/2017 08:50 AM, Langer, Christoph wrote: > So, waiting for comments from other experts...
>> Webrev: http://cr.openjdk.java.net/~horii/8179527/webrev.00/ This patch is fine as stated. Without volatile, there is a benign data race on the bugLevel field, which is okay, assuming "sun.nio.cs.bugLevel" never actually changes. Doing double-checked-locking seems excessive here, because we can accept more than one invocation of privilegedGetProperty. -Aleksey
