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

Pieter-Jan Busschaert pieterjan.busschaert@gmail.com
Fri Aug 17 12:16:00 GMT 2007


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);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dump_instr.diff
Type: application/octet-stream
Size: 5965 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/ecos-discuss/attachments/20070817/ccac5be1/attachment.obj>
-------------- next part --------------
-- 
Before posting, please read the FAQ: http://ecos.sourceware.org/fom/ecos
and search the list archive: http://ecos.sourceware.org/ml/ecos-discuss


More information about the Ecos-discuss mailing list