From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, pav@iki.fi
Subject: RE: [BlueZ,1/5] iso-tester: always use DEFER_SETUP for multiple CIS in same CIG
Date: Sun, 21 May 2023 11:11:10 -0700 (PDT) [thread overview]
Message-ID: <646a5ebe.9d0a0220.8363d.5c84@mx.google.com> (raw)
In-Reply-To: <024df2d86c14fc811701ba27bfa576476bc9c0d6.1684682575.git.pav@iki.fi>
[-- Attachment #1: Type: text/plain, Size: 3235 bytes --]
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=749544
---Test result---
Test Summary:
CheckPatch FAIL 3.27 seconds
GitLint FAIL 2.14 seconds
BuildEll PASS 35.87 seconds
BluezMake PASS 1238.75 seconds
MakeCheck PASS 13.36 seconds
MakeDistcheck PASS 202.49 seconds
CheckValgrind PASS 329.93 seconds
CheckSmatch WARNING 459.17 seconds
bluezmakeextell PASS 136.68 seconds
IncrementalBuild PASS 5329.21 seconds
ScanBuild WARNING 1454.33 seconds
Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script
Output:
[BlueZ,3/5] btdev: support multiple CIG
WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed))
#74: FILE: emulator/btdev.c:108:
+} __attribute__ ((packed));
/github/workspace/src/src/13249428.patch total: 0 errors, 1 warnings, 279 lines checked
NOTE: For some of the reported defects, checkpatch may be able to
mechanically convert to the typical style using --fix or --fix-inplace.
/github/workspace/src/src/13249428.patch has style problems, please review.
NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO
NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.
##############################
Test: GitLint - FAIL
Desc: Run gitlint
Output:
[BlueZ,5/5] iso-tester: add tests for multiple simultaneous CIG
WARNING: I3 - ignore-body-lines: gitlint will be switching from using Python regex 'match' (match beginning) to 'search' (match anywhere) semantics. Please review your ignore-body-lines.regex option accordingly. To remove this warning, set general.regex-style-search=True. More details: https://jorisroovers.github.io/gitlint/configuration/#regex-style-search
17: B2 Line has trailing whitespace: " "
##############################
Test: CheckSmatch - WARNING
Desc: Run smatch tool with source
Output:
emulator/btdev.c:416:29: warning: Variable length array is used.emulator/btdev.c:416:29: warning: Variable length array is used.
##############################
Test: ScanBuild - WARNING
Desc: Run Scan Build
Output:
emulator/btdev.c:1079:10: warning: Although the value stored to 'conn' is used in the enclosing expression, the value is never actually read from 'conn'
while ((conn = queue_find(dev->conns, match_handle,
^ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
emulator/btdev.c:1391:13: warning: Access to field 'dev' results in a dereference of a null pointer (loaded from variable 'conn')
send_event(conn->dev, BT_HCI_EVT_AUTH_COMPLETE, &ev, sizeof(ev));
^~~~~~~~~
2 warnings generated.
---
Regards,
Linux Bluetooth
next prev parent reply other threads:[~2023-05-21 18:28 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-05-21 15:27 [PATCH BlueZ 1/5] iso-tester: always use DEFER_SETUP for multiple CIS in same CIG Pauli Virtanen
2023-05-21 15:27 ` [PATCH BlueZ 2/5] shared/tester: retain test failure status Pauli Virtanen
2023-05-21 15:27 ` [PATCH BlueZ 3/5] btdev: support multiple CIG Pauli Virtanen
2023-05-21 15:27 ` [PATCH BlueZ 4/5] btdev: report right reason for local Disconnect complete Pauli Virtanen
2023-05-21 15:27 ` [PATCH BlueZ 5/5] iso-tester: add tests for multiple simultaneous CIG Pauli Virtanen
2023-05-22 18:28 ` Luiz Augusto von Dentz
2023-05-22 20:38 ` Pauli Virtanen
2023-05-22 20:48 ` Luiz Augusto von Dentz
2023-05-22 23:08 ` Luiz Augusto von Dentz
2023-05-21 16:15 ` hci_sync req_complete_skb does not get called? Pauli Virtanen
2023-05-21 16:37 ` bluez.test.bot
2023-05-21 18:11 ` bluez.test.bot [this message]
2023-05-22 20:20 ` [PATCH BlueZ 1/5] iso-tester: always use DEFER_SETUP for multiple CIS in same CIG patchwork-bot+bluetooth
2023-05-22 20:27 ` Luiz Augusto von Dentz
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=646a5ebe.9d0a0220.8363d.5c84@mx.google.com \
--to=bluez.test.bot@gmail.com \
--cc=linux-bluetooth@vger.kernel.org \
--cc=pav@iki.fi \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).