diff options
| author | Nathan Sidwell <nathan@codesourcery.com> | 2009-04-22 10:53:53 +0000 |
|---|---|---|
| committer | Nathan Sidwell <nathan@gcc.gnu.org> | 2009-04-22 10:53:53 +0000 |
| commit | 24a40b356f07a2a4d50d4c36e5bbb86ef06d3925 (patch) | |
| tree | e11dcde40e7283e50e32e96ee6676f534174b607 /gcc | |
| parent | aa9037807b6e7a681a4229ab2c34539323d3f8b3 (diff) | |
| download | gcc-24a40b356f07a2a4d50d4c36e5bbb86ef06d3925.zip gcc-24a40b356f07a2a4d50d4c36e5bbb86ef06d3925.tar.gz gcc-24a40b356f07a2a4d50d4c36e5bbb86ef06d3925.tar.bz2 | |
gthr-vxworks.h (struct __gthread_once_t): Add alignment and padding for PPC.
* gthr-vxworks.h (struct __gthread_once_t): Add alignment and
padding for PPC.
(__GTHREAD_ONCE_INIT): Adjust ppc initializer.
* config/vxlib.c (__gthread_once): Add race guard for PPC.
From-SVN: r146566
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/ChangeLog | 7 | ||||
| -rw-r--r-- | gcc/config/vxlib.c | 10 | ||||
| -rw-r--r-- | gcc/gthr-vxworks.h | 23 |
3 files changed, 35 insertions, 5 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index d727019..056f732 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,10 @@ +2009-04-22 Nathan Sidwell <nathan@codesourcery.com> + + * gthr-vxworks.h (struct __gthread_once_t): Add alignment and + padding for PPC. + (__GTHREAD_ONCE_INIT): Adjust ppc initializer. + * config/vxlib.c (__gthread_once): Add race guard for PPC. + 2009-04-22 Paolo Bonzini <bonzini@gnu.org> * config/sh/sh.c (shift_insns_rtx, shiftcosts, gen_shifty_op, diff --git a/gcc/config/vxlib.c b/gcc/config/vxlib.c index 686d66d..0ff996c 100644 --- a/gcc/config/vxlib.c +++ b/gcc/config/vxlib.c @@ -65,7 +65,15 @@ __gthread_once (__gthread_once_t *guard, void (*func)(void)) __gthread_lock_library (); #else while (!vxTas ((void *)&guard->busy)) - taskDelay (1); + { +#ifdef __PPC__ + /* This can happen on powerpc, which is using all 32 bits + of the gthread_once_t structure. */ + if (guard->done) + return; +#endif + taskDelay (1); + } #endif /* Only one thread at a time gets here. Check ->done again, then diff --git a/gcc/gthr-vxworks.h b/gcc/gthr-vxworks.h index c2fb8e6..dee1573 100644 --- a/gcc/gthr-vxworks.h +++ b/gcc/gthr-vxworks.h @@ -115,17 +115,32 @@ __gthread_recursive_mutex_unlock (__gthread_recursive_mutex_t *mutex) typedef struct { -#ifndef __RTP__ +#if !defined(__RTP__) +#if defined(__PPC__) + __attribute ((aligned (__alignof (unsigned)))) +#endif volatile unsigned char busy; #endif volatile unsigned char done; +#if !defined(__RTP__) && defined(__PPC__) + /* PPC's test-and-set implementation requires a 4 byte aligned + object, of which it only sets the first byte. We use padding + here, in order to maintain some amount of backwards + compatibility. Without this padding, gthread_once objects worked + by accident because they happen to be static objects and the ppc + port automatically increased their alignment to 4 bytes. */ + unsigned char pad1; + unsigned char pad2; +#endif } __gthread_once_t; -#ifndef __RTP__ -# define __GTHREAD_ONCE_INIT { 0, 0 } -#else +#if defined (__RTP__) # define __GTHREAD_ONCE_INIT { 0 } +#elif defined (__PPC__) +# define __GTHREAD_ONCE_INIT { 0, 0, 0, 0 } +#else +# define __GTHREAD_ONCE_INIT { 0, 0 } #endif extern int __gthread_once (__gthread_once_t *__once, void (*__func)(void)); |