[PATCH] Add support for AVR32 architecture to strace

Denys Vlasenko dvlasenk at redhat.com
Mon Feb 23 16:30:17 UTC 2009


On Mon, 2009-02-23 at 08:07 +0100, Hans-Christian Egtvedt wrote:
> Hi,
> 
> Please see the linked patch[0] which adds AVR32 architecture support to
> strace. Implemented by Håvard Skinnemoen (haavard.skinnemoen at atmel.com)
> and ported to latest CVS HEAD by me.
> 
> The patch is based against current CVS HEAD. Tested on AP7000 with some
> random commands, and it seems to behave nice. I also tried going
> through the test/ directory, but they seem to be a bit outdated and a
> bit x86 specific AFAICT?
> 
> If you have a preferred test suite, I can go through that as well.
> 
> 0: http://www.atmel.no/~hcegtvedt/strace/strace-cvs-add-avr32-arch.patch

This looks wrong (look carefully at () nesting):

-#if (defined(LINUXSPARC) || defined(LINUX_X86_64) || defined(LINUX_ARM)) && defined(__GLIBC__)
+#if (defined(LINUXSPARC) || defined(LINUX_X86_64) || defined(LINUX_ARM)) || defined (LINUX_AVR32) && defined(__GLIBC__)

--
vda





More information about the Strace-devel mailing list