[MI][patch] broken -target-detach

[Marc Khouzam <marc.khouzam@ericsson.com>]

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"));