From: Maurycy Pawłowski-Wieroński <maurycy@maurycy.com>
Date: Fri, 8 May 2026 08:25:49 +0000 (+0200)
Subject: gh-149521: Do not update `last_profiled_frame` if it's not changed (#149522)
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=d36e5b86442bcf3b06876c91ff631c1f62632506;p=thirdparty%2FPython%2Fcpython.git

gh-149521: Do not update `last_profiled_frame` if it's not changed (#149522)
---

diff --git a/Modules/_remote_debugging/threads.c b/Modules/_remote_debugging/threads.c
index d775234b8d78..4daa5e5f92bc 100644
--- a/Modules/_remote_debugging/threads.c
+++ b/Modules/_remote_debugging/threads.c
@@ -450,12 +450,14 @@ unwind_stack_for_thread(
             set_exception_cause(unwinder, PyExc_RuntimeError, "Failed to collect frames");
             goto error;
         }
-        // Update last_profiled_frame for next sample
-        uintptr_t lpf_addr =
-            *current_tstate + (uintptr_t)unwinder->debug_offsets.thread_state.last_profiled_frame;
-        if (_Py_RemoteDebug_WriteRemoteMemory(&unwinder->handle, lpf_addr,
-                                              sizeof(uintptr_t), &frame_addr) < 0) {
-            PyErr_Clear();  // Non-fatal
+        // Update last_profiled_frame for next sample if it changed
+        if (frame_addr != ctx.last_profiled_frame) {
+            uintptr_t lpf_addr =
+                *current_tstate + (uintptr_t)unwinder->debug_offsets.thread_state.last_profiled_frame;
+            if (_Py_RemoteDebug_WriteRemoteMemory(&unwinder->handle, lpf_addr,
+                                                  sizeof(uintptr_t), &frame_addr) < 0) {
+                PyErr_Clear();  // Non-fatal
+            }
         }
     } else {
         // No caching - process entire frame chain with base_frame validation