[PATCH libevdev] Update struct input_event
Deepa Dinamani
deepa.kernel at gmail.com
Tue Jan 16 00:21:08 UTC 2018
The struct input_event is not y2038 safe.
Update the struct according to the kernel patch:
https://lkml.org/lkml/2018/1/6/324
Signed-off-by: Deepa Dinamani <deepa.kernel at gmail.com>
---
include/linux/input.h | 13 ++++++++++++-
libevdev/libevdev.c | 6 ++++--
test/test-libevdev-init.c | 20 ++++++++++----------
tools/libevdev-events.c | 8 ++++----
tools/mouse-dpi-tool.c | 8 +-------
5 files changed, 31 insertions(+), 24 deletions(-)
diff --git a/include/linux/input.h b/include/linux/input.h
index c5c8bc4..8274c29 100644
--- a/include/linux/input.h
+++ b/include/linux/input.h
@@ -1,3 +1,4 @@
+/* SPDX-License-Identifier: GPL-2.0 WITH Linux-syscall-note */
/*
* Copyright (c) 1999-2002 Vojtech Pavlik
*
@@ -18,10 +19,20 @@
/*
* The event structure itself
+ * Note that __USE_TIME_BITS64 is defined by libc based on
+ * application's request to use 64 bit time_t.
*/
-
struct input_event {
+#if (__BITS_PER_LONG != 32 || !defined(__USE_TIME_BITS64)) && !defined(__KERNEL)
struct timeval time;
+#define input_event_sec time.tv_sec
+#define input_event_usec time.tv_usec
+#else
+ __kernel_ulong_t __sec;
+ __kernel_ulong_t __usec;
+#define input_event_sec __sec
+#define input_event_usec __usec
+#endif
__u16 type;
__u16 code;
__s32 value;
diff --git a/libevdev/libevdev.c b/libevdev/libevdev.c
index 41bfd25..a48974f 100644
--- a/libevdev/libevdev.c
+++ b/libevdev/libevdev.c
@@ -524,7 +524,8 @@ libevdev_get_fd(const struct libevdev* dev)
static inline void
init_event(struct libevdev *dev, struct input_event *ev, int type, int code, int value)
{
- ev->time = dev->last_event_time;
+ ev->input_event_sec = dev->last_event_time.tv_sec;
+ ev->input_event_usec = dev->last_event_time.tv_usec;
ev->type = type;
ev->code = code;
ev->value = value;
@@ -966,7 +967,8 @@ update_state(struct libevdev *dev, const struct input_event *e)
break;
}
- dev->last_event_time = e->time;
+ dev->last_event_time.tv_sec = e->input_event_sec;
+ dev->last_event_time.tv_usec = e->input_event_usec;
return rc;
}
diff --git a/test/test-libevdev-init.c b/test/test-libevdev-init.c
index f673a58..feba27f 100644
--- a/test/test-libevdev-init.c
+++ b/test/test-libevdev-init.c
@@ -575,19 +575,19 @@ START_TEST(test_clock_id_events)
ck_assert_int_eq(ev1.code, ev2.code);
ck_assert_int_eq(ev1.value, ev2.value);
- t1 = ev1.time.tv_sec * 1000000LL + ev1.time.tv_usec;
- t2 = ev2.time.tv_sec * 1000000LL + ev2.time.tv_usec;
+ t1 = ev1.input_event_sec * 1000000LL + ev1.input_event_usec;
+ t2 = ev2.input_event_sec * 1000000LL + ev2.input_event_usec;
ck_assert_int_ne(t1, t2);
- ck_assert_int_ge(ev1.time.tv_sec, t1_real.tv_sec);
- ck_assert_int_ge(ev1.time.tv_usec, t1_real.tv_nsec/1000);
- ck_assert_int_le(ev1.time.tv_sec, t2_real.tv_sec);
- ck_assert_int_le(ev1.time.tv_usec, t2_real.tv_nsec/1000);
+ ck_assert_int_ge(ev1.input_event_sec, t1_real.tv_sec);
+ ck_assert_int_ge(ev1.input_event_usec, t1_real.tv_nsec/1000);
+ ck_assert_int_le(ev1.input_event_sec, t2_real.tv_sec);
+ ck_assert_int_le(ev1.input_event_usec, t2_real.tv_nsec/1000);
- ck_assert_int_ge(ev2.time.tv_sec, t1_mono.tv_sec);
- ck_assert_int_ge(ev2.time.tv_usec, t1_mono.tv_nsec/1000);
- ck_assert_int_le(ev2.time.tv_sec, t2_mono.tv_sec);
- ck_assert_int_le(ev2.time.tv_usec, t2_mono.tv_nsec/1000);
+ ck_assert_int_ge(ev2.input_event_sec, t1_mono.tv_sec);
+ ck_assert_int_ge(ev2.input_event_usec, t1_mono.tv_nsec/1000);
+ ck_assert_int_le(ev2.input_event_sec, t2_mono.tv_sec);
+ ck_assert_int_le(ev2.input_event_usec, t2_mono.tv_nsec/1000);
uinput_device_free(uidev);
libevdev_free(dev);
diff --git a/tools/libevdev-events.c b/tools/libevdev-events.c
index 3ce23f1..dfd841f 100644
--- a/tools/libevdev-events.c
+++ b/tools/libevdev-events.c
@@ -112,13 +112,13 @@ print_event(struct input_event *ev)
{
if (ev->type == EV_SYN)
printf("Event: time %ld.%06ld, ++++++++++++++++++++ %s +++++++++++++++\n",
- ev->time.tv_sec,
- ev->time.tv_usec,
+ ev->input_event_sec,
+ ev->input_event_usec,
libevdev_event_type_get_name(ev->type));
else
printf("Event: time %ld.%06ld, type %d (%s), code %d (%s), value %d\n",
- ev->time.tv_sec,
- ev->time.tv_usec,
+ ev->input_event_sec,
+ ev->input_event_usec,
ev->type,
libevdev_event_type_get_name(ev->type),
ev->code,
diff --git a/tools/mouse-dpi-tool.c b/tools/mouse-dpi-tool.c
index b472ad0..da44124 100644
--- a/tools/mouse-dpi-tool.c
+++ b/tools/mouse-dpi-tool.c
@@ -61,12 +61,6 @@ usage(void) {
return 1;
}
-static inline uint64_t
-tv2us(const struct timeval *tv)
-{
- return tv->tv_sec * 1000000 + tv->tv_usec;
-}
-
static inline double
get_frequency(uint64_t last, uint64_t current)
{
@@ -119,7 +113,7 @@ handle_event(struct measurements *m, const struct input_event *ev)
const int idle_reset = 3000000; /* us */
uint64_t last_us = m->us;
- m->us = tv2us(&ev->time);
+ m->us = ev->input_event_sec * 1000000 + ev->input_event_usec;
/* reset after pause */
if (last_us + idle_reset < m->us) {
--
2.14.1
More information about the Input-tools
mailing list