neethu-prasad

/covered

> It seems fine to me that the GC backends are responsible for checking if the nmethod is disarmed outside the lock. However, we have some callers that now check...

@fisk I've addressed the feedback. Can you take a look? I did not remove the check [here](https://github.com/openjdk/jdk/blob/5dcb7a627e1cfb360719a25722588180e5de9d09/src/hotspot/share/gc/shared/barrierSetNMethod.cpp#L175-L177). Removing this check resulted in time out when `-XX:+DeoptimizeNMethodBarriersALot` flag set as it...

@fisk I just merged the latest changes. Do I need approval on the merge commit or can I integrate?

Thanks for the review & approval. I've created follow up bug - https://bugs.openjdk.org/browse/JDK-8334890 /integrate

@phohensee @vnkozlov Thanks for the review & approval. /integrate

Thanks for the review & approval. /integrate