Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
MIME-Version: 1.0
References: <20230913193839.3029428-1-luiz.dentz@gmail.com> <65021d28.020a0220.6f2d9.5519@mx.google.com>
In-Reply-To: <65021d28.020a0220.6f2d9.5519@mx.google.com>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Wed, 13 Sep 2023 13:44:04 -0700
Message-ID: <CABBYNZLSsyh96AphQBOckn+Y3SUX2MuXL8UiqMwMh7pZqX8mzg@mail.gmail.com>
Subject: Re: Bluetooth: L2CAP: Fix leaking l2cap_conn objects
To:     linux-bluetooth@vger.kernel.org
Cc:     "Olivier L'Heureux" <olivier.lheureux@mind.be>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Olivier,

On Wed, Sep 13, 2023 at 1:35=E2=80=AFPM <bluez.test.bot@gmail.com> wrote:
>
> This is automated email and please do not reply to this email!
>
> Dear submitter,
>
> Thank you for submitting the patches to the linux bluetooth mailing list.
> This is a CI test results with your patch series:
> PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=3D783=
902
>
> ---Test result---
>
> Test Summary:
> CheckPatch                    FAIL      1.28 seconds
> GitLint                       PASS      0.33 seconds
> SubjectPrefix                 PASS      0.10 seconds
> BuildKernel                   PASS      32.82 seconds
> CheckAllWarning               PASS      35.82 seconds
> CheckSparse                   PASS      40.91 seconds
> CheckSmatch                   PASS      115.39 seconds
> BuildKernel32                 PASS      31.58 seconds
> TestRunnerSetup               PASS      481.60 seconds
> TestRunner_l2cap-tester       PASS      27.15 seconds
> TestRunner_iso-tester         PASS      47.77 seconds
> TestRunner_bnep-tester        PASS      10.25 seconds
> TestRunner_mgmt-tester        PASS      217.08 seconds
> TestRunner_rfcomm-tester      PASS      15.95 seconds
> TestRunner_sco-tester         PASS      19.14 seconds
> TestRunner_ioctl-tester       PASS      17.60 seconds
> TestRunner_mesh-tester        PASS      13.02 seconds
> TestRunner_smp-tester         PASS      14.12 seconds
> TestRunner_userchan-tester    PASS      10.90 seconds
> IncrementalBuild              PASS      29.64 seconds
>
> Details
> ##############################
> Test: CheckPatch - FAIL
> Desc: Run checkpatch.pl script
> Output:
> Bluetooth: L2CAP: Fix leaking l2cap_conn objects
> WARNING: Reported-by: should be immediately followed by Closes: with a UR=
L to the report
> #88:
> Reported-by: Olivier L'Heureux <olivier.lheureux@fortrobotics.com>
> Signed-off-by: Luiz Augusto von Dentz <luiz.von.dentz@intel.com>
>
> total: 0 errors, 1 warnings, 0 checks, 84 lines checked
>
> NOTE: For some of the reported defects, checkpatch may be able to
>       mechanically convert to the typical style using --fix or --fix-inpl=
ace.
>
> /github/workspace/src/src/13383710.patch has style problems, please revie=
w.
>
> NOTE: Ignored message types: UNKNOWN_COMMIT_ID
>
> NOTE: If any of the errors are false positives, please report
>       them to the maintainer, see CHECKPATCH in MAINTAINERS.
>

Would you be able to check if this would fix the problem for you? I
will be working on a way to reproduce it with the timeout as well,
just want a quick validation before doing it.

--=20
Luiz Augusto von Dentz