This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
[MI][patch] broken -target-detach
- From: Marc Khouzam <marc dot khouzam at ericsson dot com>
- To: "'gdb-patches at sourceware dot org'" <gdb-patches at sourceware dot org>
- Date: Mon, 27 Sep 2010 16:25:27 -0400
- Subject: [MI][patch] broken -target-detach
Hi,
with GDB 7.2, the MI command -target-detach is not working very well.
It still assumes that the thread-group id is a pid, instead of the
new thread-group id format which starts with an 'i'.
Also, the usage printout does not correspond to the documentation:
Usage: -target-detach [thread-group]
vs
-target-detach [ pid | gid ]
I have a patch that fixes things to parse both a pid or a thread-group.
I've added it at the bottom, but I'm not sure it is the right approach.
With the new global MI flag --thread-group, I wonder if -target-detach
should take a thread-group as a parameter anymore.
Note that although "-target-detach i1" does not work,
"-target-detach --thread-group i1" works.
I'm not sure what would happen if I did:
"-target-detach --thread-group i1 <pid or threadGroupId>"
I'm guessing the --thread-group flag would get ignored.
What should we do about this?
Thanks
Marc
You can see the error in the session below:
GNU gdb (GDB) 7.2.50.20100927-cvs
(gdb) interpreter-exec mi "-target-attach --thread-group i1 12283"
^done,frame={addr="0x00f2b422",func="__kernel_vsyscall",args=[]}
(gdb)
(gdb) interpreter-exec mi "-list-thread-groups"
^done,groups=[{id="i1",type="process",pid="12283",executable="/home/lmckhou/testing/a1",cores=["0"]}]
(gdb)
(gdb) interpreter-exec mi "-target-detach a a"
^error,msg="Usage: -target-detach [thread-group]"
(gdb)
(gdb) interpreter-exec mi "-target-detach i1"
^error,msg="Cannot parse thread group id 'i1'"
(gdb) interpreter-exec mi "-target-detach --thread-group i1"
^done
Patch:
### Eclipse Workspace Patch 1.0
#P src
Index: gdb/mi/mi-main.c
===================================================================
RCS file: /cvs/src/src/gdb/mi/mi-main.c,v
retrieving revision 1.181
diff -u -r1.181 mi-main.c
--- gdb/mi/mi-main.c 1 Sep 2010 19:03:54 -0000 1.181
+++ gdb/mi/mi-main.c 27 Sep 2010 17:18:21 -0000
@@ -418,19 +418,36 @@
mi_cmd_target_detach (char *command, char **argv, int argc)
{
if (argc != 0 && argc != 1)
- error ("Usage: -target-detach [thread-group]");
+ error ("Usage: -target-detach [pid | thread-group]");
if (argc == 1)
{
struct thread_info *tp;
char *end = argv[0];
+ /* First see if we are dealing with a pid */
int pid = strtol (argv[0], &end, 10);
if (*end != '\0')
- error (_("Cannot parse thread group id '%s'"), argv[0]);
+ {
+ int id;
+ struct inferior *inf;
+ /* We are not dealing with a pid. Check for group id */
+ if (*(argv[0]) != 'i')
+ error (_("Invalid pid or thread-group id '%s'"), argv[0]);
+
+ id = strtoul (argv[0] + 1, &end, 0);
+ if (*end != '\0')
+ error (_("Invalid syntax of group id '%s'"), argv[0]);
+
+ inf = find_inferior_id (id);
+ if (!inf)
+ error (_("Non-existent thread-group id '%d'"), id);
+
+ pid = inf->pid;
+ }
/* Pick any thread in the desired process. Current
- target_detach deteches from the parent of inferior_ptid. */
+ target_detach detaches from the parent of inferior_ptid. */
tp = iterate_over_threads (find_thread_of_process, &pid);
if (!tp)
error (_("Thread group is empty"));