Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-4625-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Precedence: bulk
MIME-Version: 1.0
References: <20240514174755.750764-1-luiz.dentz@gmail.com> <6643af59.0c0a0220.8776f.2984@mx.google.com>
In-Reply-To: <6643af59.0c0a0220.8776f.2984@mx.google.com>
From: Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date: Tue, 14 May 2024 15:15:45 -0400
Message-ID: <CABBYNZK50b1hbiRbxx23V7y_cHW1nXGOBOq1QnLoqKp3dG98=Q@mail.gmail.com>
Subject: Re: [RFC,v4] Bluetooth: MGMT: Make MGMT_OP_LOAD_CONN_PARAM update
 existing connection
To: linux-bluetooth@vger.kernel.org
Cc: Pauli Virtanen <pav@iki.fi>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

Hi Pauli,

On Tue, May 14, 2024 at 2:37=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=3D853=
196
>
> ---Test result---
>
> Test Summary:
> CheckPatch                    PASS      16.53 seconds
> GitLint                       FAIL      3.37 seconds
> SubjectPrefix                 PASS      0.14 seconds
> BuildKernel                   PASS      30.15 seconds
> CheckAllWarning               PASS      32.80 seconds
> CheckSparse                   PASS      38.06 seconds
> CheckSmatch                   FAIL      35.29 seconds
> BuildKernel32                 PASS      29.14 seconds
> TestRunnerSetup               PASS      524.29 seconds
> TestRunner_l2cap-tester       FAIL      23.05 seconds
> TestRunner_iso-tester         FAIL      37.23 seconds
> TestRunner_bnep-tester        PASS      4.75 seconds
> TestRunner_mgmt-tester        FAIL      122.87 seconds
> TestRunner_rfcomm-tester      PASS      7.38 seconds
> TestRunner_sco-tester         FAIL      17.42 seconds
> TestRunner_ioctl-tester       PASS      7.67 seconds
> TestRunner_mesh-tester        FAIL      6.02 seconds
> TestRunner_smp-tester         PASS      6.82 seconds
> TestRunner_userchan-tester    PASS      4.96 seconds
> IncrementalBuild              PASS      36.21 seconds
>
> Details
> ##############################
> Test: GitLint - FAIL
> Desc: Run gitlint
> Output:
> [RFC,v4] Bluetooth: MGMT: Make MGMT_OP_LOAD_CONN_PARAM update existing co=
nnection
>
> WARNING: I3 - ignore-body-lines: gitlint will be switching from using Pyt=
hon regex 'match' (match beginning) to 'search' (match anywhere) semantics.=
 Please review your ignore-body-lines.regex option accordingly. To remove t=
his warning, set general.regex-style-search=3DTrue. More details: https://j=
orisroovers.github.io/gitlint/configuration/#regex-style-search
> 1: T1 Title exceeds max length (81>80): "[RFC,v4] Bluetooth: MGMT: Make M=
GMT_OP_LOAD_CONN_PARAM update existing connection"
> ##############################
> Test: CheckSmatch - FAIL
> Desc: Run smatch tool with source
> Output:
>
> Segmentation fault (core dumped)
> make[4]: *** [scripts/Makefile.build:244: net/bluetooth/hci_core.o] Error=
 139
> make[4]: *** Deleting file 'net/bluetooth/hci_core.o'
> make[3]: *** [scripts/Makefile.build:485: net/bluetooth] Error 2
> make[2]: *** [scripts/Makefile.build:485: net] Error 2
> make[2]: *** Waiting for unfinished jobs....
> Segmentation fault (core dumped)
> make[4]: *** [scripts/Makefile.build:244: drivers/bluetooth/bcm203x.o] Er=
ror 139
> make[4]: *** Deleting file 'drivers/bluetooth/bcm203x.o'
> make[4]: *** Waiting for unfinished jobs....
> make[3]: *** [scripts/Makefile.build:485: drivers/bluetooth] Error 2
> make[2]: *** [scripts/Makefile.build:485: drivers] Error 2
> make[1]: *** [/github/workspace/src/src/Makefile:1919: .] Error 2
> make: *** [Makefile:240: __sub-make] Error 2
> ##############################
> Test: TestRunner_l2cap-tester - FAIL
> Desc: Run l2cap-tester with test-runner
> Output:
> Total: 55, Passed: 53 (96.4%), Failed: 2, Not Run: 0
>
> Failed Test Cases
> L2CAP BR/EDR Client - TX Timestamping                Timed out    2.313 s=
econds
> L2CAP LE Client - TX Timestamping                    Timed out    2.208 s=
econds
> ##############################
> Test: TestRunner_iso-tester - FAIL
> Desc: Run iso-tester with test-runner
> Output:
> Total: 122, Passed: 118 (96.7%), Failed: 4, Not Run: 0
>
> Failed Test Cases
> ISO Send - TX Timestamping                           Timed out    2.741 s=
econds
> ISO Send - TX Sched Timestamping                     Timed out    1.990 s=
econds
> ISO Send - TX CMSG Timestamping                      Timed out    2.002 s=
econds
> ISO Send - TX No Poll Timestamping                   Failed       0.162 s=
econds

I guess we should disable (not run) these tests that requires
SO_TIMESTAMP for now.

> ##############################
> Test: TestRunner_mgmt-tester - FAIL
> Desc: Run mgmt-tester with test-runner
> Output:
> Total: 492, Passed: 489 (99.4%), Failed: 1, Not Run: 2
>
> Failed Test Cases
> Read Exp Feature - Success (Index None)              Failed       0.080 s=
econds
> ##############################
> Test: TestRunner_sco-tester - FAIL
> Desc: Run sco-tester with test-runner
> Output:
> Total: 15, Passed: 14 (93.3%), Failed: 1, Not Run: 0
>
> Failed Test Cases
> SCO CVSD Send - TX Timestamping                      Timed out    2.245 s=
econds
> ##############################

Ditto.

> Test: TestRunner_mesh-tester - FAIL
> Desc: Run mesh-tester with test-runner
> Output:
> Total: 10, Passed: 9 (90.0%), Failed: 1, Not Run: 0
>
> Failed Test Cases
> Mesh - Send cancel - 1                               Failed       0.104 s=
econds
>
>
> ---
> Regards,
> Linux Bluetooth
>


--=20
Luiz Augusto von Dentz