Revert "Avoid JNI upcall for ctx_Dup"

This reverts commit 95898f3.

Author: fangerer

graalpython/com.oracle.graal.python.jni/src/hpy_jni.c

@@ -251,24 +251,7 @@ static void ctx_Close_jni(HPyContext *ctx, HPy h) {
 }
 
 static HPy ctx_Dup_jni(HPyContext *ctx, HPy h) {
-    jobject hpyContext = CONTEXT_INSTANCE(ctx);
-    if (unclosedHandleTop > 0) {
-        jobjectArray hpy_handles = (jobjectArray)(*jniEnv)->GetObjectField(jniEnv, hpyContext, jniField_hpyHandleTable);
-        if (hpy_handles == NULL) {
-            LOGS("hpy handle table is NULL")
-            return HPy_NULL;
-        }
-
-        uint64_t recycled = toBits(unclosedHandles[--unclosedHandleTop]);
-        assert(recycled < INT32_MAX);
-        uint64_t h_bits = toBits(h);
-        assert(h_bits < INT32_MAX);
-        jobject element = (*jniEnv)->GetObjectArrayElement(jniEnv, hpy_handles, (jsize)h_bits);
-        (*jniEnv)->SetObjectArrayElement(jniEnv, hpy_handles, (jsize)recycled, element);
-        dup_data_pointer(ctx, h_bits, recycled);
-        return toPtr(boxHandle(recycled));
-    }
-    return DO_UPCALL_HPY(hpyContext, Dup, HPY_UP(h));
+    return DO_UPCALL_HPY(CONTEXT_INSTANCE(ctx), Dup, HPY_UP(h));
 }
 
 static int ctx_NumberCheck_jni(HPyContext *ctx, HPy obj) {

graalpython/com.oracle.graal.python.jni/src/hpy_native_cache.h

@@ -97,14 +97,4 @@ load_global_native_data_pointer(HPyContext *ctx, uint64_t g_bits, uint64_t h_bit
     space[HANDLE_DATAPTR_INDEX(h_bits)] = g_data_ptr;
 }
 
-/*
- * Duplicates the native data pointer of an handle in the native cache.
- */
-static inline void
-dup_data_pointer(HPyContext *ctx, uint64_t h_from, uint64_t h_to) {
-    void** space = (void**)ctx->_private;
-    space[HANDLE_DATAPTR_INDEX(h_to)] = space[HANDLE_DATAPTR_INDEX(h_from)];
-    LOG("%llu %llu %p", h_from, h_to, space[HANDLE_DATAPTR_INDEX(h_from)])
-}
-
 #endif /* SRC_HPY_NATIVE_CACHE_H_ */