File 5011-dbg-Disarm-the-stop-0-footgun.patch of Package erlang

Overview Repositories Revisions Requests Users Attributes Meta

File 5011-dbg-Disarm-the-stop-0-footgun.patch of Package erlang (Revision 2ace71662e6798e9ff6c2a98c08c5d61)

Currently displaying revision 2ace71662e6798e9ff6c2a98c08c5d61 , Show latest

From c211ae25e8a6719d2b10aac3a861cdbd6fe5881f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?John=20H=C3=B6gberg?= <john@erlang.org>
Date: Wed, 2 Feb 2022 14:51:45 +0100
Subject: [PATCH] dbg: Disarm the stop/0 footgun

`dbg:stop/0` explicitly leaves global function tracing untouched,
which can lead to some very confusing situations:

1. As it clears the trace flags of all processes, it often
   *looks* like all global function trace patterns have been
   cleared, leaving a fun surprise for the next poor sod that
   tries to trace anything.
2. It makes it really annoying to clear global function traces
   afterwards, as we've discarded all information on what nodes
   we've been tracing on. The user more or less has to call
   `erlang:trace_pattern/3` themselves on all nodes they've
   traced on.

Since clearing the trace flags of all processes means that `dbg`
can't coexist with other trace tools to begin with, there is
very little point in keeping global trace patterns around after
stop/0. This commit makes stop/0 behave like stop_clear/0,
ensuring that *ALL* tracing is turned off on *ALL* associated
nodes before shutting down.
---
 lib/runtime_tools/doc/src/dbg.xml | 16 ++++------------
 lib/runtime_tools/src/dbg.erl     | 16 +++++++++++-----
 2 files changed, 15 insertions(+), 17 deletions(-)

diff --git a/lib/runtime_tools/doc/src/dbg.xml b/lib/runtime_tools/doc/src/dbg.xml
index 7532ec692d..1f6bc713f4 100644
--- a/lib/runtime_tools/doc/src/dbg.xml
+++ b/lib/runtime_tools/doc/src/dbg.xml
@@ -1156,20 +1156,12 @@ hello</pre>
 </func>
 <func>
 <name since="">stop() -> ok</name>
- <fsummary>Stop the <c>dbg</c>server and the tracing of all processes.</fsummary>
- <desc>
- Stops the <c>dbg</c> server and clears all trace flags for
- all processes and all local trace patterns for all functions. Also
- shuts down all trace clients and closes all trace ports.
- Note that no global trace patterns are affected by this
- function.
- </desc>
- </func>
- <func>
- <name since="">stop_clear() -> ok</name>
 <fsummary>Stop the <c>dbg</c>server and the tracing of all processes, and clears trace patterns.</fsummary>
 <desc>
- Same as stop/0, but also clears all trace patterns on global functions calls.
+ Stops the <c>dbg</c> server, clears all trace flags for
+ all processes, clears all trace patterns for all functions,
+ clears trace patterns for send/receive, shuts down all trace clients,
+ and closes all trace ports.
 </desc>
 </func>
 </funcs>
diff --git a/lib/runtime_tools/src/dbg.erl b/lib/runtime_tools/src/dbg.erl
index 7dd3c452e5..ca0f6e689e 100644
--- a/lib/runtime_tools/src/dbg.erl
+++ b/lib/runtime_tools/src/dbg.erl
@@ -595,17 +595,23 @@ c(Parent, M, F, A, Flags) ->
 
 -spec stop() -> ok.
 stop() ->
+ {ok, _} = ctp(),
+ {ok, _} = ctpe('receive'),
+ {ok, _} = ctpe('send'),
+
 Mref = erlang:monitor(process, dbg),
 catch dbg ! {self(),stop},
+
 receive
-	{'DOWN',Mref,_,_,_} ->
-	 ok
+ {'DOWN',Mref,_,_,_} -> ok
 end.
 
+%% This is a vestigial function that used to be documented as a variant of
+%% `stop/0` that also clears global function traces. Since `stop/0` now clears
+%% all tracing as the user would expect it to, we've removed this from the
+%% documentation but keep it around for backwards compatibility, much like
+%% `queue:lait`.
 stop_clear() ->
- {ok, _} = ctp(),
- {ok, _} = ctpe('receive'),
- {ok, _} = ctpe('send'),
 stop().
 
 %%% Calling the server.
-- 
2.34.1

Places

File 5011-dbg-Disarm-the-stop-0-footgun.patch of Package erlang (Revision 2ace71662e6798e9ff6c2a98c08c5d61)

Places