On Fri, Oct 28, 2016 at 5:43 AM, Arnd Bergmann <arnd@xxxxxxxx> wrote: > On Monday, October 17, 2016 8:27:32 PM CEST Deepa Dinamani wrote: >> @@ -55,24 +60,24 @@ struct ff_effect_compat { >> >> static inline size_t input_event_size(void) >> { >> - return (in_compat_syscall() && !COMPAT_USE_64BIT_TIME) ? >> - sizeof(struct input_event_compat) : sizeof(struct input_event); >> + return in_compat_syscall() ? sizeof(struct raw_input_event_compat) : >> + sizeof(struct raw_input_event); >> } > > I think the COMPAT_USE_64BIT_TIME check has to stay here, > it's needed for x32 mode on x86-64. There is no time_t anymore in the raw_input_event structure. The struct uses __kernel_ulong_t type. This should take care of x32 support. >From this cover letter: https://www.spinics.net/lists/linux-arch/msg16356.html I see that that the __kernel types were introduced to address the ABI issues for x32. -Deepa -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html