[RFR] Ignore fields annotated with @LazyInit

Man Cao manc at google.com
Wed Jun 12 23:22:56 UTC 2019


Looks good. One minor comment is the VM symbol
java_util_concurrent_atomic_annotation_LazyInit, can we remove "atomic_"
from the name?
No need for a new webrev for this change.

-Man


On Mon, Jun 10, 2019 at 5:37 PM Arthur Eubanks <aeubanks at google.com> wrote:

> webrev:
> http://cr.openjdk.java.net/~aeubanks/tsanfieldignore/webrev.00/index.html
>
> This change introduces a new annotation,
> java.util.concurrent.annotation.LazyInit, which causes TSAN to ignore races
> on that field.
> References are not simply ignored, but a release/acquire is performed on
> them. This is so that any following accesses to its member variables are
> not also reported as racy.
>
> LazyInitReferenceLoopTest.java is still failing with weird stack trace
> issues, but I'd like some feedback on this change anyway.
>


More information about the tsan-dev mailing list