Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <632d5652.c80a0220.3482c.3bac@mx.google.com>
Date:   Thu, 22 Sep 2022 23:46:42 -0700 (PDT)
Content-Type: multipart/mixed; boundary="===============9169417009614600795=="
MIME-Version: 1.0
From:   bluez.test.bot@gmail.com
To:     linux-bluetooth@vger.kernel.org, mmandlik@google.com
Subject: RE: [v6,1/3] Bluetooth: Add support for hci devcoredump
Reply-To: linux-bluetooth@vger.kernel.org
In-Reply-To: <20220922221037.v6.1.I44a65858f594107d3688087dfbdf669fac450b52@changeid>
References: <20220922221037.v6.1.I44a65858f594107d3688087dfbdf669fac450b52@changeid>
Precedence: bulk

--===============9169417009614600795==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit

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=679707

---Test result---

Test Summary:
CheckPatch                    FAIL      3.05 seconds
GitLint                       PASS      3.12 seconds
SubjectPrefix                 PASS      2.73 seconds
BuildKernel                   PASS      35.48 seconds
BuildKernel32                 PASS      31.81 seconds
Incremental Build with patchesPASS      63.14 seconds
TestRunner: Setup             PASS      527.04 seconds
TestRunner: l2cap-tester      PASS      17.42 seconds
TestRunner: iso-tester        PASS      16.74 seconds
TestRunner: bnep-tester       PASS      6.62 seconds
TestRunner: mgmt-tester       PASS      105.17 seconds
TestRunner: rfcomm-tester     PASS      10.40 seconds
TestRunner: sco-tester        PASS      9.85 seconds
TestRunner: smp-tester        PASS      9.89 seconds
TestRunner: userchan-tester   PASS      6.81 seconds

Details
##############################
Test: CheckPatch - FAIL - 3.05 seconds
Run checkpatch.pl script with rule in .checkpatch.conf
[v6,1/3] Bluetooth: Add support for hci devcoredump\Traceback (most recent call last):
  File "scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
Traceback (most recent call last):
  File "scripts/spdxcheck.py", line 6, in <module>
    from ply import lex, yacc
ModuleNotFoundError: No module named 'ply'
WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating?
#146: 
new file mode 100644

WARNING:SPLIT_STRING: quoted string split across lines
#622: FILE: net/bluetooth/coredump.c:300:
+				    "Devcoredump complete with size %u "
+				    "(expect %zu)",

WARNING:SPLIT_STRING: quoted string split across lines
#641: FILE: net/bluetooth/coredump.c:319:
+				    "Devcoredump aborted with size %u "
+				    "(expect %zu)",

WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
#735: FILE: net/bluetooth/coredump.c:413:
+	if (!skb) {
+		bt_dev_err(hdev, "Failed to allocate devcoredump init");

WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
#778: FILE: net/bluetooth/coredump.c:456:
+	if (!skb) {
+		bt_dev_err(hdev, "Failed to allocate devcoredump pattern");

WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
#804: FILE: net/bluetooth/coredump.c:482:
+	if (!skb) {
+		bt_dev_err(hdev, "Failed to allocate devcoredump complete");

WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message
#826: FILE: net/bluetooth/coredump.c:504:
+	if (!skb) {
+		bt_dev_err(hdev, "Failed to allocate devcoredump abort");

total: 0 errors, 7 warnings, 0 checks, 685 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/12986135.patch has style problems, please review.

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.


---
Regards,
Linux Bluetooth


--===============9169417009614600795==--