Imagelibrary/binutils-gdb
1
0
Fork 1
mirror of https://github.com/bminor/binutils-gdb.git synced 2025-12-05 15:15:42 +00:00
Code Issues Packages Projects Releases Wiki Activity

gdbserver: remove handling of the 'L' tracepoint action

Browse Source 
Now that static tracepoint support is removed from gdbserver, it makes
sense to remove handling of the 'L' tracepoint action, too.  The code
that checks received actions already has a default case that tolerates
unrecognized actions:

        default:
          trace_debug ("unknown trace action '%c', ignoring...", *act);

In case 'L' is unexpectedly received, we would at least be able to see
this in the logs.

Approved-By: Simon Marchi <simon.marchi@efficios.com>
This commit is contained in:
Tankut Baris Aktemur
2025-01-14 10:10:31 +01:00
parent a00b56ff16
commit 1956ad8d66
1 changed files with 0 additions and 46 deletions
Download Patch File Download Diff File Expand all files Collapse all files

46
gdbserver/tracepoint.cc
View File

@@ -435,12 +435,6 @@ struct eval_expr_action
struct agent_expr *expr;
};
/* An 'L' (collect static trace data) action. */
struct collect_static_trace_data_action
{
struct tracepoint_action base;
};
#ifndef IN_PROCESS_AGENT
static CORE_ADDR
m_tracepoint_action_download (const struct tracepoint_action *action)
@@ -532,24 +526,6 @@ x_tracepoint_action_send ( char *buffer, const struct tracepoint_action *action)
return agent_expr_send (buffer, eaction->expr);
}
static CORE_ADDR
l_tracepoint_action_download (const struct tracepoint_action *action)
{
CORE_ADDR ipa_action
= target_malloc (sizeof (struct collect_static_trace_data_action));
target_write_memory (ipa_action, (unsigned char *) action,
sizeof (struct collect_static_trace_data_action));
return ipa_action;
}
static char *
l_tracepoint_action_send (char *buffer, const struct tracepoint_action *action)
{
return buffer;
}
static char *
tracepoint_action_send (char *buffer, const struct tracepoint_action *action)
{
@@ -561,8 +537,6 @@ tracepoint_action_send (char *buffer, const struct tracepoint_action *action)
return r_tracepoint_action_send (buffer, action);
case 'X':
return x_tracepoint_action_send (buffer, action);
case 'L':
return l_tracepoint_action_send (buffer, action);
}
error ("Unknown trace action '%c'.", action->type);
}
@@ -578,8 +552,6 @@ tracepoint_action_download (const struct tracepoint_action *action)
return r_tracepoint_action_download (action);
case 'X':
return x_tracepoint_action_download (action);
case 'L':
return l_tracepoint_action_download (action);
}
error ("Unknown trace action '%c'.", action->type);
}
@@ -1863,18 +1835,6 @@ add_tracepoint_action (struct tracepoint *tpoint, const char *packet)
++act;
break;
}
case 'L':
{
struct collect_static_trace_data_action *raction =
XNEW (struct collect_static_trace_data_action);
raction->base.type = *act;
action = &raction->base;
trace_debug ("Want to collect static trace data");
++act;
break;
}
case 'S':
trace_debug ("Unexpected step action, ignoring");
++act;
@@ -4547,12 +4507,6 @@ do_action_at_tracepoint (struct tracepoint_hit_ctx *ctx,
}
}
break;
case 'L':
{
trace_debug ("warning: collecting static trace data, "
"but static tracepoints are not supported");
}
break;
default:
trace_debug ("unknown trace action '%c', ignoring", taction->type);
break;