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>, Mark Kettenis <mark dot kettenis at xs4all dot nl>
- Date: Wed, 10 Aug 2011 15:08:03 +0100
- Subject: Re: [RFA 4/8] New port: TI C6x: Read loadmap from gdbserver
- References: <4E263865.2070100@codesourcery.com> <4E40A565.2020208@codesourcery.com> <201108101503.28057.pedro@codesourcery.com>
On Wednesday 10 August 2011 15:03:27, Pedro Alves wrote:
> On Tuesday 09 August 2011 04:11:33, Yao Qi wrote:
> > 2011-08-09 Andrew Stubbs <ams@codesourcery.com>
> > Yao Qi <yao@codesourcery.com>
> >
> > gdb/
> > * remote.c (PACKET_qXfer_fdpic): New enum value.
> > (remote_protocol_features): Add qXfer:fdpic:read packet.
> > (remote_xfer_partial): Support TARGET_OBJECT_FDPIC.
> > (_initialize_remote): Add set/show remote read-fdpic-loadmap command.
> > * target.h (enum target_object): Add TARGET_OBJECT_FDPIC.
> >
> > gdb/gdbserver:
> > * target.h (struct target_ops): Add read_loadmap.
> > * linux-low.c (struct target_loadseg): New type.
> > (struct target_loadmap): New type.
> > (linux_read_loadmap): New function.
> > (linux_target_ops): Add linux_read_loadmap.
> > * server.c (handle_query): Support qXfer:fdpic:read packet.
> > * win32-low.c (win32_target_ops): Initialize field `read_loadmap' to NULL.
> >
> > gdb/doc/
> > * gdb.texinfo : Document qXfer:fdpic:read packet.
>
> Thanks. Looks good to me.
Oh, just one detail:
> + /* Read loadmaps. Read LEN bytes at OFFSET into a buffer at MYADDR. */
> + int (*read_loadmap) (const char *annex, CORE_ADDR offset,
> + unsigned char *myaddr, unsigned int len);
...
> + NULL, /* read_fdpic_loadmap */
Please make those coincide though.
--
Pedro Alves