8386312: Shenandoah: Incorrect assertion in ShenandoahAllocRate uses is_locked() instead of owned_by_self()

Reviewed-by: wkemper, ruili

Author: Xiaolong Peng

src/hotspot/share/gc/shenandoah/shenandoahAllocRate.hpp

@@ -174,7 +174,7 @@ class ShenandoahAllocRate {
   void take_sample(jlong now, jlong elapsed, size_t unsampled);
 
   double upper_bound_no_lock(const double standard_deviations) const {
-    assert(_sample_lock.is_locked(), "Caller must hold lock");
+    assert(_sample_lock.owned_by_self(), "Caller must hold lock");
     return _baseline.weighted_average() + standard_deviations * _baseline.weighted_sd();
   }
 };

src/hotspot/share/gc/shenandoah/shenandoahAllocRate.inline.hpp

@@ -114,7 +114,7 @@ void ShenandoahAllocRate<Clock>::force_update() {
 
 template<typename Clock>
 void ShenandoahAllocRate<Clock>::take_sample(jlong now, jlong elapsed, size_t unsampled) {
-  assert(_sample_lock.is_locked(), "Caller must hold lock");
+  assert(_sample_lock.owned_by_self(), "Caller must hold lock");
 
   _last_sample_time = now;