Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1030514AbbDWSrS (ORCPT ); Thu, 23 Apr 2015 14:47:18 -0400 Received: from mail-ig0-f170.google.com ([209.85.213.170]:35818 "EHLO mail-ig0-f170.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755038AbbDWSrO (ORCPT ); Thu, 23 Apr 2015 14:47:14 -0400 Date: Thu, 23 Apr 2015 11:47:09 -0700 From: Dmitry Torokhov To: Benjamin Tissoires Cc: Peter Hutterer , Henrik Rydberg , Jiri Kosina , "linux-input@vger.kernel.org" , lkml Subject: Re: [PATCH v2] Input - mt: Fix input_mt_get_slot_by_key Message-ID: <20150423184709.GG34808@dtor-ws> References: <1427741655-4142-1-git-send-email-benjamin.tissoires@redhat.com> <20150406163349.GD36770@dtor-ws> <5539279F.4050400@bitmath.org> <20150423182046.GF34808@dtor-ws> <20150423183827.GA10937@mail.corp.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150423183827.GA10937@mail.corp.redhat.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2259 Lines: 52 On Thu, Apr 23, 2015 at 02:38:27PM -0400, Benjamin Tissoires wrote: > On Apr 23 2015 or thereabouts, Dmitry Torokhov wrote: > > On Thu, Apr 23, 2015 at 07:10:55PM +0200, Henrik Rydberg wrote: > > > > "Creation, replacement and destruction of contacts is achieved by > > > > modifying the ABS_MT_TRACKING_ID of the associated slot. A > > > > non-negative tracking id is interpreted as a contact, and the value -1 > > > > denotes an unused slot. A tracking id not previously present is > > > > considered new, and a tracking id no longer present is considered > > > > removed." > > > > > > > > If some userspace is confused with missing -1 tracking ID for that > > > > slot, that userspace should be fixed. > > > > > > I agree. Some userland applications work with add/remove out of convenience, and > > > cannot handle the more compressed notation the kernel slot handling allows. > > > Fixing those applications will be a good thing. > > > > > > Unfortunately the patch already appeared in Linus' tree... > > > > It's in 4.1-rc0 so we can just revert it. > > > > Before we call for a revert, I'd like to hear Peter thoughts about it, > as he is the main user of the slotted protocol. > > Also, Dmitry, can you check if the ChromeOS driver and (or) the android > one would need to be adapted? My guess is that none of these drivers are > able to handle a silent closing of a slot I will, at least for Chrome. But if it is broken I'll simply open a big to fix it ;) > and the easiest solution might > be to simply change the documentation if in fact nobody uses the > compressed notation (which removes just one ABS_SLOT event within the > frame, so not sure we can call it compressed BTW). No, it is more that that I think. Wouldn't we need to allocate memory for 2*n slots to actually reliably track all contacts if they all happen to get released and put back onto the surface in different place very very quickly if we insist on sending tracking id -1 for previously used slots? Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/