path: root/include
diff options
authorRich Felker <>2013-03-06 23:57:39 -0500
committerRich Felker <>2013-03-06 23:57:39 -0500
commit08514d0614c97f8c7055955073517844e1b1cf0b (patch)
treeb94de9ea117b914099d0780ec73dce7359527136 /include
parentf44076181d25dd2401d182c111330b4290a9535c (diff)
fix epoll structure alignment on non-x86_64 archs
this fix is far from ideal and breaks the rule of not using arch-specific #ifdefs, but for now we just need a solution to the existing breakage. the underlying problem is that the kernel folks made a very stupid decision to make misalignment of this struct part of the kernel API/ABI for x86_64, in order to avoid writing a few extra lines of code to handle both 32- and 64-bit userspace on 64-bit kernels. I had just added the packed attribute unconditionally thinking it was harmless on 32-bit archs, but non-x86 32-bit archs have 8-byte alignment on 64-bit types.
Diffstat (limited to 'include')
1 files changed, 5 insertions, 1 deletions
diff --git a/include/sys/epoll.h b/include/sys/epoll.h
index 5e131ab1..1f0312e5 100644
--- a/include/sys/epoll.h
+++ b/include/sys/epoll.h
@@ -46,7 +46,11 @@ typedef union epoll_data {
struct epoll_event {
uint32_t events;
epoll_data_t data;
-} __attribute__ ((__packed__));
+#ifdef __x86_64__
+__attribute__ ((__packed__))
int epoll_create(int);