linux-kernel - Re: [PATCH v3 v3 00/20] Input and HID updates for 3.7

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 3 Sep 2012 15:00:47 +0200
From:	Benjamin Tissoires <benjamin.tissoires@...il.com>
To:	Henrik Rydberg <rydberg@...omail.se>
Cc:	Dmitry Torokhov <dmitry.torokhov@...il.com>,
	Jiri Kosina <jkosina@...e.cz>, linux-input@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 v3 00/20] Input and HID updates for 3.7

Hi Henrik,

Indeed, we are getting close. I tested the whole patch set again.
So:
- patches 1 to 19 are not introducing any bugs (at least from what I've seen).
- patch 20 is not good (see the reply in the patch).

So far, devices tested are:
- Elan
- 3M
- Lumio
- Cypress
- Topseed
- Stantum

you can add my "reviewed by" on patches 1, 7, 8, 10, 17, 18 and 19
(those that are more MT related than generic input).
You can also add my 'tested-by' on the whole patch set except for the 20st.

Cheers,
Benjamin


On Sat, Sep 1, 2012 at 9:46 PM, Henrik Rydberg <rydberg@...omail.se> wrote:
> Hi Dmitry, Jiri,
>
> Here is the third version of the patchset for 3.7. I think we are
> getting close. Changes in short:
>
> Patch 2: Folded the size estimate removal into this patch, since the
> other estimates are still needed for MT-A devices.
>
> Patch 5: Reworked to handle SYN_MT events the 'old' way.
> Patch 10: Reworked to allow for a proper device-to-slot mapping.
> Patch 15: Remove the old size estimate when converting to MT-B.
>
> Patch 18: New patch, carefully perform finalization per input device.
> Patch 19: Shortened version of touch state removal.
>
> Patch 20: New patch, fixing a long-standing bug for devices capable of
> more than ten touches.
>
> Several bugs removed, thanks to everyone involved in finding
> those. The set has been more thoroughly tested, but still needs
> further scrutiny from Benjamin.
>
> Thanks,
> Henrik
>
> Henrik Rydberg (20):
>   Input: Break out MT data
>   Input: Improve the events-per-packet estimate
>   Input: Make sure we follow all EV_KEY events
>   Input: Move autorepeat to the event-passing phase
>   Input: Send events one packet at a time
>   Input: evdev - Add the events() callback
>   Input: MT - Add flags to input_mt_init_slots()
>   Input: MT - Handle frame synchronization in core
>   Input: MT - Add in-kernel tracking
>   Input: MT - Get slot by key
>   Input: bcm5974 - only setup button urb for TYPE1 devices
>   Input: bcm5974 - Preparatory renames
>   Input: bcm5974 - Drop pressure and width emulation
>   Input: bcm5974 - Drop the logical dimensions
>   Input: bcm5974 - Convert to MT-B
>   HID: Only dump input if someone is listening
>   HID: Add an input configured notification callback
>   HID: hid-multitouch: Simplify setup and frame synchronization
>   HID: hid-multitouch: Remove the redundant touch state
>   HID: hid-multitouch: Add missing contact count detection
>
>  drivers/hid/hid-core.c                   |   3 +-
>  drivers/hid/hid-input.c                  |  11 +-
>  drivers/hid/hid-magicmouse.c             |   2 +-
>  drivers/hid/hid-multitouch.c             | 157 ++++++---------
>  drivers/input/evdev.c                    |  78 +++++---
>  drivers/input/input-mt.c                 | 294 +++++++++++++++++++++++++---
>  drivers/input/input.c                    | 245 ++++++++++++++++--------
>  drivers/input/misc/uinput.c              |   2 +-
>  drivers/input/mouse/alps.c               |   2 +-
>  drivers/input/mouse/bcm5974.c            | 317 ++++++++++++-------------------
>  drivers/input/mouse/elantech.c           |   4 +-
>  drivers/input/mouse/sentelic.c           |   2 +-
>  drivers/input/mouse/synaptics.c          |   4 +-
>  drivers/input/tablet/wacom_wac.c         |   6 +-
>  drivers/input/touchscreen/atmel_mxt_ts.c |   2 +-
>  drivers/input/touchscreen/cyttsp_core.c  |   2 +-
>  drivers/input/touchscreen/edt-ft5x06.c   |   2 +-
>  drivers/input/touchscreen/egalax_ts.c    |   2 +-
>  drivers/input/touchscreen/ili210x.c      |   2 +-
>  drivers/input/touchscreen/mms114.c       |   2 +-
>  drivers/input/touchscreen/penmount.c     |   2 +-
>  drivers/input/touchscreen/wacom_w8001.c  |   2 +-
>  include/linux/hid.h                      |   3 +
>  include/linux/input.h                    |  33 +++-
>  include/linux/input/mt.h                 |  55 +++++-
>  25 files changed, 778 insertions(+), 456 deletions(-)
>
> --
> 1.7.12
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-input" in
> the body of a message to majordomo@...r.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/