From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Tissoires Subject: Re: [PATCH 0/4] HID: debugfs rework Date: Thu, 18 Apr 2013 09:52:40 +0200 Message-ID: <516FA648.20207@redhat.com> References: <1366220296-14346-1-git-send-email-benjamin.tissoires@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Return-path: Received: from mx1.redhat.com ([209.132.183.28]:57702 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965398Ab3DRHx0 (ORCPT ); Thu, 18 Apr 2013 03:53:26 -0400 In-Reply-To: Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Jiri Kosina Cc: Dmitry Torokhov , Benjamin Tissoires , linux-input@vger.kernel.org, linux-kernel@vger.kernel.org On 04/17/2013 07:44 PM, Jiri Kosina wrote: > On Wed, 17 Apr 2013, Benjamin Tissoires wrote: > >> Hi Jiri, >> >> This is a small rework of the HID debugfs. >> I encountered a problem with multitouch devices: they have too much usages to >> fit into the fixed size output buffer of 512. >> So I digg a little, and end up with those 4 patches. > > Hi Benjamin, > > thanks, I will look into it and see whether I would be able to apply it > still for 3.10 merge window. Thanks. I think patches 1 and 2 of this series are pretty straightforward. Patches 3 and 4 will maybe require a little bit more attention. I don't mind if it's postponed to 3.11 (given the long time this has been broken for devices with big reports). I just need to access HID debugfs for hid-replay when hidraw does not send anything. But as I'm also willing to use hid-replay with current kernels, I still need a way to use the best option (hidraw or hid debugfs) for current kernels. > > I also have a locking fix for HID-debugfs which I am going to apply > shortly, but I am travelling this week, so I am in a bit degraded mode. > > For reference, locking fix below. > > > > From: Jiri Kosina > Subject: [PATCH] HID: protect hid_debug_list > > Accesses to hid_device->hid_debug_list are not serialized properly, which > could result in SMP concurrency issues when HID debugfs events are accessesed s/accessesed/accessed ? > by multiple userspace processess. s/processess/processes ? > > Serialize all the list operations by a mutex. > > Reported-by: Al Viro > Signed-off-by: Jiri Kosina Reviewed-by: Benjamin Tissoires I also have a patch regarding forcing hidraw output even if raw_event returns > 0, but I'll send it over a new thread. This thread will start to be quite complicate to follow otherwise... :) Cheers, Benjamin