Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932101AbbDQVDN (ORCPT ); Fri, 17 Apr 2015 17:03:13 -0400 Received: from mail-ig0-f173.google.com ([209.85.213.173]:37669 "EHLO mail-ig0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752975AbbDQVDK (ORCPT ); Fri, 17 Apr 2015 17:03:10 -0400 MIME-Version: 1.0 In-Reply-To: <024FA95B-C70F-42ED-BDBF-69D09AA58203@holtmann.org> References: <300602D8-9DB7-4F8D-AC97-B60924554093@holtmann.org> <38429B86-0E46-4C6A-9954-3260D864B10C@holtmann.org> <024FA95B-C70F-42ED-BDBF-69D09AA58203@holtmann.org> Date: Fri, 17 Apr 2015 17:03:09 -0400 X-Google-Sender-Auth: wtP5J7fT6g7mcI13V9Bhmh_4KEA Message-ID: Subject: Re: [V4.1] Regression: Bluetooth mouse not working. From: Linus Torvalds To: Marcel Holtmann Cc: =?UTF-8?Q?J=C3=B6rg_Otte?= , "Gustavo F. Padovan" , Johan Hedberg , "bluez mailin list (linux-bluetooth@vger.kernel.org)" , Linux Kernel Mailing List Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1317 Lines: 28 On Fri, Apr 17, 2015 at 4:35 PM, Marcel Holtmann wrote: > > accepting all flags regardless was an oversight on my part in the first place. What this patch tried to do is to limit it to what userspace is currently actually using. My mistake was to look only at BlueZ 5.x userspace and not at BlueZ 4.x userspace. So what about anybody else? Android doesn't use BlueZ, afaik. Any other direct accesses? If we already know that BlueZ 4.x did something else, what makes you so sure that this now covers all cases? The thing is, the bluetooth code has clearly never cared about these bits before. Is there any real reason to think that people haven't passed in garbage? Do we even know that those flags were *initialized* at all by user space in all use cases? So I'm ok with trying to fix things up, but I have to say that if the fixed-up case also causes problems (because there was some other case that you didn't think of), I'm going to be pissed off, and I'm going to expect you to *jump* on it, and revert the whole thing. Ok? Linus -- 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/