This is the mail archive of the ecos-patches@sourceware.org mailing list for the eCos project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH] host/dump_instr : support for byteswapping, real time, wrapped buffers


On 17/08/07, Andrew Lunn <andrew@lunn.ch> wrote:
> On Fri, Aug 17, 2007 at 10:46:15AM +0200, Pieter-Jan Busschaert wrote:
> > On 17/08/07, Andrew Lunn <andrew@lunn.ch> wrote:
> > >
> > > Could you produce a clean patch which i can commit to CVS?
> >
> > I thought about that, but decided not to, because I hard-coded the
> > big-endian / little-endian swapping. I could add a CDL option to
> > enable / disable this swapping, or I could submit a clean patch
> > without swapping at all. What do you prefer ?
>
> This is a host tool. So you have command line parameters. -b and -l.
> Or maybe it is easier to just have -s for swap. Then you don't need to
> worry about the host endien's.



*) This patch adds support for byteswapping to the dump_instr host tool.
*) Support for correct time is added too, based on this version by Andrew Lunn :
http://ecos.sourceware.org/ml/ecos-discuss/2001-10/msg00283.html
*) This patch also adds support for wrapped around instrumentation
buffers. You somehow have to know the wraparound location, ie by

int wrap = (instrument_buffer_pointer - &instrument_buffer[0]);
diag_printf("wrap around = %d\n", wrap);

Attachment: dump_instr.diff
Description: Binary data


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]