From patchwork Fri Feb 3 09:39:36 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Archie Pusaka X-Patchwork-Id: 650380 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9F986C6379F for ; Fri, 3 Feb 2023 09:40:18 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232421AbjBCJkQ (ORCPT ); Fri, 3 Feb 2023 04:40:16 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59432 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232634AbjBCJjs (ORCPT ); Fri, 3 Feb 2023 04:39:48 -0500 Received: from mail-yw1-x114a.google.com (mail-yw1-x114a.google.com [IPv6:2607:f8b0:4864:20::114a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 0585925BAB for ; Fri, 3 Feb 2023 01:39:42 -0800 (PST) Received: by mail-yw1-x114a.google.com with SMTP id 00721157ae682-509ab88f98fso46293277b3.10 for ; Fri, 03 Feb 2023 01:39:41 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20210112; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=QN14ziO4JxILP7mp40ANOysEeXcgBUkL5cm+RkfL8ZI=; b=ALQj60Z8rE+o1jswa2sIaTx9ItPQ/H6fcei4FBro1MVoWPnvBFJmIRPFr3A5je5c+/ XXcuJ5tLa0Iu4OvJYbcRxIRzENg+rRn8Y19EEwtQXH3D4sE+UOgMF12sDkCo80rYqtwo YiD4S0CqHRfYLQQGF+iZ+sH1mmhzL/ujWr2AoiBZTRi62GjDS3inDI9mbBcG9YoOYsk+ vbq15f4bmy1zPTWlVHdvFNH7HWSPhf8Zy0ekRbboT+LJ5SvX3cQAOmpdRh6oeRbg9NZh sJhvGWAa9fI0MXDId99AiT9ATmaoarGE7RwdXyOhuelp1HzXWVfS9VFMERg60w0+5ueD we5Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=QN14ziO4JxILP7mp40ANOysEeXcgBUkL5cm+RkfL8ZI=; b=kTY7ULKK7UWMQt9HeW9PYTmwpFWmLy1flYFcc2isg+nzERFmytYibQCih97B+19dov dIuxJE1pt46KzOleOOivJtbOGF1VCn9yuAx8qLZckjdmAXPzVzv0PelOqXU7JULQ83O4 81o8cEFRwI0w5CSqcBw7aY5UqYBcP+4dqZmzsEC3iJfASuF38QN7NYdlfdbVEpAwX2rN obLpc8j2waeMWpdUy1R35Qyt5sqAbNni0SjZsyhBtU4vP6H6gXkBzd42v8GHgFOPx4DX NF6qmBFVXc5CVybG7lC+BxvfcKvxBjSFH7SDJ0dgH867cRs4UY9yro4oIIZPe0JNWZai VDvA== X-Gm-Message-State: AO0yUKUMkLOteqAMNXwbsbUZW9b5PYBtKUqsQKUY4ZZ8qkdWuqts2xko nu+8h9/VcbvLSdv4xJOCFgd0eGMhIaKt7KrgxExqqVhCgqo93INYEkHy3sOjC9omzDkwxqis3rm lDXD4iFC2KxDrzwyEViVMRk7MMQdVLlbvFfJm0LLGKpioXCGqdIDGQFUshLbW9WW2M2ET9HpeQK Re X-Google-Smtp-Source: AK7set/E9XfPkDUc7TDF1CbiQthpXU/qBRltk6TmvylDtTmU566EojX9yMs6MRHtiyH8PAMAq8X0TmpCgD0I X-Received: from apusaka-p920.tpe.corp.google.com ([2401:fa00:1:17:dc12:2976:77a:8cb7]) (user=apusaka job=sendgmr) by 2002:a81:ac4d:0:b0:507:739d:e0ca with SMTP id z13-20020a81ac4d000000b00507739de0camr1073769ywj.495.1675417181195; Fri, 03 Feb 2023 01:39:41 -0800 (PST) Date: Fri, 3 Feb 2023 17:39:36 +0800 Mime-Version: 1.0 X-Mailer: git-send-email 2.39.1.519.gcb327c4b5f-goog Message-ID: <20230203173900.1.I9ca803e2f809e339da43c103860118e7381e4871@changeid> Subject: [PATCH] Bluetooth: Make sure LE create conn cancel is sent when timeout From: Archie Pusaka To: linux-bluetooth , Luiz Augusto von Dentz , Marcel Holtmann Cc: CrosBT Upstreaming , Archie Pusaka , Ying Hsu , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Johan Hedberg , Paolo Abeni , linux-kernel@vger.kernel.org, netdev@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-bluetooth@vger.kernel.org From: Archie Pusaka When sending LE create conn command, we set a timer with a duration of HCI_LE_CONN_TIMEOUT before timing out and calling create_le_conn_complete. Additionally, when receiving the command complete, we also set a timer with the same duration to call le_conn_timeout. Usually the latter will be triggered first, which then sends a LE create conn cancel command. However, due to the nature of racing, it is possible for the former to be called first, thereby calling the chain hci_conn_failed -> hci_conn_del -> cancel_delayed_work, thereby preventing LE create conn cancel to be sent. In this situation, the controller will be stuck in trying the LE connection. This patch flushes le_conn_timeout on create_le_conn_complete to make sure we always send LE create connection cancel, if necessary. Signed-off-by: Archie Pusaka Reviewed-by: Ying Hsu --- net/bluetooth/hci_conn.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/net/bluetooth/hci_conn.c b/net/bluetooth/hci_conn.c index 61a34801e61e..f2c7edaa7145 100644 --- a/net/bluetooth/hci_conn.c +++ b/net/bluetooth/hci_conn.c @@ -1243,6 +1243,8 @@ static void create_le_conn_complete(struct hci_dev *hdev, void *data, int err) if (conn != hci_lookup_le_connect(hdev)) goto done; + /* Flush to make sure we send create conn cancel command if needed */ + flush_delayed_work(&conn->le_conn_timeout); hci_conn_failed(conn, bt_status(err)); done: