diff options
author | Pedro Alves <palves@redhat.com> | 2009-06-19 13:35:35 +0000 |
---|---|---|
committer | Pedro Alves <palves@redhat.com> | 2009-06-19 13:35:35 +0000 |
commit | cf8fd78b3890f7c0a8050f7dc6cefaa8918374f9 (patch) | |
tree | 87acb8302fad51432a54c125a357661c2a7666c3 | |
parent | 5e7c27a6076edbd535b3988449c0f24b40ac2371 (diff) | |
download | gdb-cf8fd78b3890f7c0a8050f7dc6cefaa8918374f9.zip gdb-cf8fd78b3890f7c0a8050f7dc6cefaa8918374f9.tar.gz gdb-cf8fd78b3890f7c0a8050f7dc6cefaa8918374f9.tar.bz2 |
2009-06-19 Aleksandar Ristovski <aristovski@qnx.com>
Pedro Alves <pedro@codesourcery.com>
* target.h (struct target_ops) <supports_multi_process>: New
callback.
(target_supports_multi_process): New.
* server.c (handle_query): Even if GDB reports support, only
enable multi-process if the target also supports it. Report
multi-process support only if the target backend supports it.
* linux-low.c (linux_supports_multi_process): New function.
(linux_target_ops): Install it as target_supports_multi_process
callback.
-rw-r--r-- | gdb/gdbserver/ChangeLog | 13 | ||||
-rw-r--r-- | gdb/gdbserver/linux-low.c | 7 | ||||
-rw-r--r-- | gdb/gdbserver/server.c | 11 | ||||
-rw-r--r-- | gdb/gdbserver/target.h | 7 |
4 files changed, 35 insertions, 3 deletions
diff --git a/gdb/gdbserver/ChangeLog b/gdb/gdbserver/ChangeLog index 25bf311..b973d55 100644 --- a/gdb/gdbserver/ChangeLog +++ b/gdb/gdbserver/ChangeLog @@ -1,3 +1,16 @@ +2009-06-19 Aleksandar Ristovski <aristovski@qnx.com> + Pedro Alves <pedro@codesourcery.com> + + * target.h (struct target_ops) <supports_multi_process>: New + callback. + (target_supports_multi_process): New. + * server.c (handle_query): Even if GDB reports support, only + enable multi-process if the target also supports it. Report + multi-process support only if the target backend supports it. + * linux-low.c (linux_supports_multi_process): New function. + (linux_target_ops): Install it as target_supports_multi_process + callback. + 2009-05-24 Doug Evans <dje@google.com> Global renaming of find_thread_pid to find_thread_ptid. diff --git a/gdb/gdbserver/linux-low.c b/gdb/gdbserver/linux-low.c index 9af9e6e..21bf523 100644 --- a/gdb/gdbserver/linux-low.c +++ b/gdb/gdbserver/linux-low.c @@ -3008,6 +3008,12 @@ linux_start_non_stop (int nonstop) return 0; } +static int +linux_supports_multi_process (void) +{ + return 1; +} + static struct target_ops linux_target_ops = { linux_create_inferior, linux_attach, @@ -3045,6 +3051,7 @@ static struct target_ops linux_target_ops = { linux_supports_non_stop, linux_async, linux_start_non_stop, + linux_supports_multi_process }; static void diff --git a/gdb/gdbserver/server.c b/gdb/gdbserver/server.c index 423427c..a9bf4a8 100644 --- a/gdb/gdbserver/server.c +++ b/gdb/gdbserver/server.c @@ -1074,9 +1074,13 @@ handle_query (char *own_buf, int packet_len, int *new_packet_len_p) p != NULL; p = strtok (NULL, ";")) { - /* Record if GDB knows about multiprocess support. */ if (strcmp (p, "multiprocess+") == 0) - multi_process = 1; + { + /* GDB supports and wants multi-process support if + possible. */ + if (target_supports_multi_process ()) + multi_process = 1; + } } sprintf (own_buf, "PacketSize=%x;QPassSignals+", PBUFSIZ - 1); @@ -1106,7 +1110,8 @@ handle_query (char *own_buf, int packet_len, int *new_packet_len_p) if (the_target->qxfer_osdata != NULL) strcat (own_buf, ";qXfer:osdata:read+"); - strcat (own_buf, ";multiprocess+"); + if (target_supports_multi_process ()) + strcat (own_buf, ";multiprocess+"); if (target_supports_non_stop ()) strcat (own_buf, ";QNonStop+"); diff --git a/gdb/gdbserver/target.h b/gdb/gdbserver/target.h index 7ff3849..70b5d01 100644 --- a/gdb/gdbserver/target.h +++ b/gdb/gdbserver/target.h @@ -275,6 +275,9 @@ struct target_ops /* Switch to non-stop (1) or all-stop (0) mode. Return 0 on success, -1 otherwise. */ int (*start_non_stop) (int); + + /* Returns true if the target supports multi-process debugging. */ + int (*supports_multi_process) (void); }; extern struct target_ops *the_target; @@ -311,6 +314,10 @@ void set_target_ops (struct target_ops *); #define target_async(enable) \ (the_target->async ? (*the_target->async) (enable) : 0) +#define target_supports_multi_process() \ + (the_target->supports_multi_process ? \ + (*the_target->supports_multi_process) () : 0) + /* Start non-stop mode, returns 0 on success, -1 on failure. */ int start_non_stop (int nonstop); |