On 9/22/20 5:23 PM, John Paul Adrian Glaubitz wrote:> On 9/22/20 3:27 PM, John Paul Adrian Glaubitz wrote: >>>> That a typical bad alignment check. It should use __alignof__, not >>>> sizeof. >>> >>> Aha, I was assuming something like that already but I wasn't sure since >>> libatomic-ops wasn't >>> updated for a while in Debian [1]. >> >> What would be the correct check then in [1]? > > I was just re-reading the code and realized I'm dumb. Never mind. > > s/sizeof/__alignof__/ should help.
Hmm, now it fails with __alignof__(): (sid-m68k-sbuild)root@epyc:/# debfoster -f debfoster: /usr/include/atomic_ops/sysdeps/loadstore/atomic_load.h:31: AO_load: Assertion `((size_t)addr & (__alignof__(*addr) - 1)) == 0' failed. qemu: uncaught target signal 6 (Aborted) - core dumped Aborted (sid-m68k-sbuild)root@epyc:/# Adrian -- .''`. John Paul Adrian Glaubitz : :' : Debian Developer - glaub...@debian.org `. `' Freie Universitaet Berlin - glaub...@physik.fu-berlin.de `- GPG: 62FF 8A75 84E0 2956 9546 0006 7426 3B37 F5B5 F913
