This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA 4/8] New port: TI C6x: Read loadmap from gdbserver
- From: Pedro Alves <pedro at codesourcery dot com>
- To: gdb-patches at sourceware dot org
- Cc: Yao Qi <yao at codesourcery dot com>, Mike Frysinger <vapier at gentoo dot org>
- Date: Mon, 8 Aug 2011 14:31:54 +0100
- Subject: Re: [RFA 4/8] New port: TI C6x: Read loadmap from gdbserver
- References: <4E263865.2070100@codesourcery.com> <201108072030.02712.vapier@gentoo.org> <4E3F4E05.6010409@codesourcery.com>
On Monday 08 August 2011 03:46:29, Yao Qi wrote:
> @@ -1182,6 +1197,7 @@ static const struct qxfer qxfer_packets[] =
> { "statictrace", handle_qxfer_statictrace },
> { "threads", handle_qxfer_threads },
> { "traceframe-info", handle_qxfer_traceframe_info },
> + { "fdpic", handle_qxfer_fdpic},
Please keep the list alpha sorted.
> + memcpy (myaddr, ((char *)data) + offset, copy_length);
memcpy (myaddr, (char *) data + offset, copy_length);
> @@ -5112,6 +5172,11 @@ static struct target_ops linux_target_ops = {
> NULL,
> #endif
> linux_core_of_thread,
> +#if defined PT_GETDSBT
> + linux_read_loadmap,
> +#else
> + NULL,
> +#endif
> linux_process_qsupported,
> linux_supports_tracepoints,
> linux_read_pc,
Looks like you may be breaking all !tic6x ports. Either
put the new method at the end, or update all
struct target_ops instances to adjust for the new field.
Otherwise, with Mark's comments addressed, this looks
fine to me.
Is there any patch in the series missing review?
Please take care to apply the patches in a sequence
where none breaks the build, to avoid breaking bisects.
--
Pedro Alves