From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 88C05C433E6 for ; Wed, 6 Jan 2021 16:43:28 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 554EC23132 for ; Wed, 6 Jan 2021 16:43:28 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727475AbhAFQnO (ORCPT ); Wed, 6 Jan 2021 11:43:14 -0500 Received: from relay9-d.mail.gandi.net ([217.70.183.199]:58563 "EHLO relay9-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726661AbhAFQnO (ORCPT ); Wed, 6 Jan 2021 11:43:14 -0500 X-Originating-IP: 82.255.60.242 Received: from [192.168.1.150] (lns-bzn-39-82-255-60-242.adsl.proxad.net [82.255.60.242]) (Authenticated sender: hadess@hadess.net) by relay9-d.mail.gandi.net (Postfix) with ESMTPSA id 22AD9FF806; Wed, 6 Jan 2021 16:42:31 +0000 (UTC) Message-ID: <20346dac6bd4c7fa6642b5516d89fdb275ed4990.camel@hadess.net> Subject: Re: [PATCH] Bluetooth: L2CAP: Try harder to accept device not knowing options From: Bastien Nocera To: Luiz Augusto von Dentz Cc: "linux-bluetooth@vger.kernel.org" , Florian Dollinger Date: Wed, 06 Jan 2021 17:42:31 +0100 In-Reply-To: References: <20201208172912.4352-1-hadess@hadess.net> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.38.2 (3.38.2-1.fc33) MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org On Tue, 2020-12-08 at 10:09 -0800, Luiz Augusto von Dentz wrote: > Hi Bastien, > > On Tue, Dec 8, 2020 at 9:36 AM Bastien Nocera > wrote: > > > > The current implementation of L2CAP options negotiation will > > continue > > the negotiation when a device responds with L2CAP_CONF_UNACCEPT > > ("unaccepted > > options"), but not when the device replies with L2CAP_CONF_UNKNOWN > > ("unknown > > options"). > > > > Trying to continue the negotiation without ERTM support will allow > > Bluetooth-capable XBox One controllers (notably models 1708 and > > 1797) > > to connect. > > While the bellow traces looks fine we need to confirm that it doesn't > break the qualification tests e.g: > > L2CAP/COS/CFD/BV-14-C [Unknown Mandatory Options Request] > > • Test Purpose Verify that the IUT can give the appropriate error > code > when the Lower Tester proposes any number of unknown options where at > least one is mandatory. > > Afaik it should be fine to continue with another round of > configuration given that it only expects the error 0x0003, but we > better confirm PTS doesn't expect a L2CAP Disconnect after it. I tested this today using Fedora's kernel-5.11.0-0.rc2.114.fc34: https://koji.fedoraproject.org/koji/buildinfo?buildID=1664670 And a local build using the same source kernel with this patch on top. Both managed to pass the test without any problems. I'll send the results of the test privately to you and Marcel. Cheers