On Fri, Oct 28, 2016 at 5:43 AM, Arnd Bergmann [off-list ref] wrote:

On Monday, October 17, 2016 8:27:32 PM CEST Deepa Dinamani wrote:

quoted

@@ -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