On Sun, Feb 20, 2011 at 06:09:48PM -0600, Jonathan Nieder wrote:
Julien Cristau wrote:
On Wed, Feb 16, 2011 at 13:56:52 +0100, Julien Cristau wrote:

gx_semaphore_alloc() allocates a 8-byte-aligned gx_semaphore_t, which means that sema->native is *not* 8-byte-aligned (its offset is 4), whereas alignof(pt_semaphore_t) == 8. The easiest fix is probably to make sure gp_semaphore has maximal alignment.

Quick test with the following patch seems to work (using Stéphane's test case).

Thanks, both.  I've filed the bug upstream.

Jonas et al, would you like a git branch with this patch applied?

No need - I can grab it from the bugreport.

Thanks, everyone!


 - Jonas

--
 * Jonas Smedegaard - idealist & Internet-arkitekt
 * Tlf.: +45 40843136  Website: http://dr.jones.dk/

 [x] quote me freely  [ ] ask before reusing  [ ] keep private

Attachment: signature.asc
Description: Digital signature

Reply via email to