| Message ID | 1451943026-1240-1-git-send-email-a.mathur@samsung.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
Hi Aniroop,
[auto build test ERROR on input/next]
[also build test ERROR on v4.4-rc8 next-20160104]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
url: https://github.com/0day-ci/linux/commits/Aniroop-Mathur/Input-evdev-drop-partial-events-after-emptying-the-buffer/20160105-053003
base: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next
config: avr32-atstk1006_defconfig (attached as .config)
reproduce:
wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=avr32
All errors (new ones prefixed by >>):
drivers/input/evdev.c: In function '__evdev_queue_syn_dropped':
drivers/input/evdev.c:195: error: request for member 'time' in something not a structure or union
>> drivers/input/evdev.c:196: error: incompatible types in assignment
--
/kbuild/src/um/drivers/input/evdev.c: In function '__evdev_queue_syn_dropped':
/kbuild/src/um/drivers/input/evdev.c:195: error: request for member 'time' in something not a structure or union
>> /kbuild/src/um/drivers/input/evdev.c:196: error: incompatible types in assignment
vim +196 drivers/input/evdev.c
189 * ignore partial events and if last packet is fully stored, queue
190 * SYN_REPORT so that clients would not ignore next full packet.
191 */
192 if (prev_ev->type != EV_SYN && prev_ev->code != SYN_REPORT) {
193 client->drop_pevent = true;
194 } else if (prev_ev->type == EV_SYN && prev_ev->code == SYN_REPORT) {
> 195 prev_ev.time = ev.time;
> 196 client->buffer[client->head++] = prev_ev;
197 client->head &= mask;
198 client->packet_head = client->head;
199
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
Just now sent v4. Please ignore v3. On Tue, Jan 5, 2016 at 3:19 AM, kbuild test robot <lkp@intel.com> wrote: > Hi Aniroop, > > [auto build test ERROR on input/next] > [also build test ERROR on v4.4-rc8 next-20160104] > [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] > > url: https://github.com/0day-ci/linux/commits/Aniroop-Mathur/Input-evdev-drop-partial-events-after-emptying-the-buffer/20160105-053003 > base: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next > config: avr32-atstk1006_defconfig (attached as .config) > reproduce: > wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross > chmod +x ~/bin/make.cross > # save the attached .config to linux build tree > make.cross ARCH=avr32 > > All errors (new ones prefixed by >>): > > drivers/input/evdev.c: In function '__evdev_queue_syn_dropped': > drivers/input/evdev.c:195: error: request for member 'time' in something not a structure or union >>> drivers/input/evdev.c:196: error: incompatible types in assignment > -- > /kbuild/src/um/drivers/input/evdev.c: In function '__evdev_queue_syn_dropped': > /kbuild/src/um/drivers/input/evdev.c:195: error: request for member 'time' in something not a structure or union >>> /kbuild/src/um/drivers/input/evdev.c:196: error: incompatible types in assignment > > vim +196 drivers/input/evdev.c > > 189 * ignore partial events and if last packet is fully stored, queue > 190 * SYN_REPORT so that clients would not ignore next full packet. > 191 */ > 192 if (prev_ev->type != EV_SYN && prev_ev->code != SYN_REPORT) { > 193 client->drop_pevent = true; > 194 } else if (prev_ev->type == EV_SYN && prev_ev->code == SYN_REPORT) { > > 195 prev_ev.time = ev.time; > > 196 client->buffer[client->head++] = prev_ev; > 197 client->head &= mask; > 198 client->packet_head = client->head; > 199 > > --- > 0-DAY kernel test infrastructure Open Source Technology Center > https://lists.01.org/pipermail/kbuild-all Intel Corporation -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Aniroop,
[auto build test ERROR on input/next]
[also build test ERROR on v4.4-rc8 next-20160104]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
url: https://github.com/0day-ci/linux/commits/Aniroop-Mathur/Input-evdev-drop-partial-events-after-emptying-the-buffer/20160105-053003
base: https://git.kernel.org/pub/scm/linux/kernel/git/dtor/input.git next
config: i386-randconfig-x005-01040616 (attached as .config)
reproduce:
# save the attached .config to linux build tree
make ARCH=i386
All errors (new ones prefixed by >>):
drivers/input/evdev.c: In function '__evdev_queue_syn_dropped':
>> drivers/input/evdev.c:195:10: error: request for member 'time' in something not a structure or union
prev_ev.time = ev.time;
^
>> drivers/input/evdev.c:196:34: error: incompatible types when assigning to type 'struct input_event' from type 'struct input_event *'
client->buffer[client->head++] = prev_ev;
^
vim +/time +195 drivers/input/evdev.c
189 * ignore partial events and if last packet is fully stored, queue
190 * SYN_REPORT so that clients would not ignore next full packet.
191 */
192 if (prev_ev->type != EV_SYN && prev_ev->code != SYN_REPORT) {
193 client->drop_pevent = true;
194 } else if (prev_ev->type == EV_SYN && prev_ev->code == SYN_REPORT) {
> 195 prev_ev.time = ev.time;
> 196 client->buffer[client->head++] = prev_ev;
197 client->head &= mask;
198 client->packet_head = client->head;
199
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
diff --git a/drivers/input/evdev.c b/drivers/input/evdev.c index e9ae3d5..2d670e4 100644 --- a/drivers/input/evdev.c +++ b/drivers/input/evdev.c @@ -58,6 +58,7 @@ struct evdev_client { struct list_head node; unsigned int clk_type; bool revoked; + bool drop_pevent; /* specifies if partial events need to be dropped */ unsigned long *evmasks[EV_CNT]; unsigned int bufsize; struct input_event buffer[]; @@ -156,7 +157,12 @@ static void __evdev_flush_queue(struct evdev_client *client, unsigned int type) static void __evdev_queue_syn_dropped(struct evdev_client *client) { struct input_event ev; + struct input_event *prev_ev; ktime_t time; + unsigned int mask = client->bufsize - 1; + + /* Store previous event */ + prev_ev = &client->buffer[(client->head - 1) & mask]; time = client->clk_type == EV_CLK_REAL ? ktime_get_real() : @@ -170,13 +176,33 @@ static void __evdev_queue_syn_dropped(struct evdev_client *client) ev.value = 0; client->buffer[client->head++] = ev; - client->head &= client->bufsize - 1; + client->head &= mask; if (unlikely(client->head == client->tail)) { /* drop queue but keep our SYN_DROPPED event */ - client->tail = (client->head - 1) & (client->bufsize - 1); + client->tail = (client->head - 1) & mask; client->packet_head = client->tail; } + + /* + * If last packet is NOT fully stored, set drop_pevent to true to + * ignore partial events and if last packet is fully stored, queue + * SYN_REPORT so that clients would not ignore next full packet. + */ + if (prev_ev->type != EV_SYN && prev_ev->code != SYN_REPORT) { + client->drop_pevent = true; + } else if (prev_ev->type == EV_SYN && prev_ev->code == SYN_REPORT) { + prev_ev.time = ev.time; + client->buffer[client->head++] = prev_ev; + client->head &= mask; + client->packet_head = client->head; + + /* drop queue but keep our SYN_DROPPED & SYN_REPORT event */ + if (unlikely(client->head == client->tail)) { + client->tail = (client->head - 2) & mask; + client->packet_head = client->tail; + } + } } static void evdev_queue_syn_dropped(struct evdev_client *client) @@ -235,18 +261,8 @@ static void __pass_event(struct evdev_client *client, client->head &= client->bufsize - 1; if (unlikely(client->head == client->tail)) { - /* - * This effectively "drops" all unconsumed events, leaving - * EV_SYN/SYN_DROPPED plus the newest event in the queue. - */ - client->tail = (client->head - 2) & (client->bufsize - 1); - - client->buffer[client->tail].time = event->time; - client->buffer[client->tail].type = EV_SYN; - client->buffer[client->tail].code = SYN_DROPPED; - client->buffer[client->tail].value = 0; - client->packet_head = client->tail; + __evdev_queue_syn_dropped(client); } if (event->type == EV_SYN && event->code == SYN_REPORT) { @@ -284,6 +300,17 @@ static void evdev_pass_values(struct evdev_client *client, wakeup = true; } + /* + * drop partial events of last packet but queue SYN_REPORT + * so that clients would not ignore extra full packet. + */ + if (client->drop_pevent) { + if (v->type == EV_SYN && v->code == SYN_REPORT) + client->drop_pevent = false; + else + continue; + } + event.type = v->type; event.code = v->code; event.value = v->value;
This patch introduces concept to drop partial events in evdev handler itself after emptying the buffer which are dropped by all evdev clients in userspace after SYN_DROPPED occurs and this in turn reduces evdev client reading requests plus saves memory space filled by partial events in evdev handler buffer. Also, this patch prevents dropping of full packet by evdev client after SYN_DROPPED occurs in case last stored event was SYN_REPORT. (like after clock change request) Signed-off-by: Aniroop Mathur <a.mathur@samsung.com> --- drivers/input/evdev.c | 53 ++++++++++++++++++++++++++++++++++++++------------- 1 file changed, 40 insertions(+), 13 deletions(-)