2023-04-28 08:18:38

by bluez.test.bot

[permalink] [raw]
Subject: RE: [v1,1/2] Bluetooth: hci_qca: Add qcomm devcoredump sysfs support

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

---Test result---

Test Summary:
CheckPatch PASS 1.57 seconds
GitLint PASS 0.69 seconds
SubjectPrefix PASS 0.25 seconds
BuildKernel FAIL 26.11 seconds
CheckAllWarning FAIL 28.77 seconds
CheckSparse FAIL 32.42 seconds
CheckSmatch FAIL 85.67 seconds
BuildKernel32 FAIL 25.21 seconds
TestRunnerSetup PASS 445.15 seconds
TestRunner_l2cap-tester PASS 17.21 seconds
TestRunner_iso-tester PASS 21.48 seconds
TestRunner_bnep-tester PASS 5.59 seconds
TestRunner_mgmt-tester PASS 116.11 seconds
TestRunner_rfcomm-tester PASS 9.04 seconds
TestRunner_sco-tester PASS 8.37 seconds
TestRunner_ioctl-tester PASS 9.72 seconds
TestRunner_mesh-tester PASS 7.19 seconds
TestRunner_smp-tester PASS 8.22 seconds
TestRunner_userchan-tester PASS 5.96 seconds
IncrementalBuild FAIL 23.85 seconds

Details
##############################
Test: BuildKernel - FAIL
Desc: Build Kernel for Bluetooth
Output:

drivers/bluetooth/hci_qca.c: In function ‘qca_controller_memdump’:
drivers/bluetooth/hci_qca.c:1070:10: error: implicit declaration of function ‘hci_devcoredump_init’; did you mean ‘hci_devcd_init’? [-Werror=implicit-function-declaration]
1070 | ret = hci_devcoredump_init(hu->hdev, qca_memdump->ram_dump_size);
| ^~~~~~~~~~~~~~~~~~~~
| hci_devcd_init
drivers/bluetooth/hci_qca.c:1119:4: error: implicit declaration of function ‘hci_devcoredump_append_pattern’; did you mean ‘hci_devcd_append_pattern’? [-Werror=implicit-function-declaration]
1119 | hci_devcoredump_append_pattern(hu->hdev, 0x00,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append_pattern
drivers/bluetooth/hci_qca.c:1136:4: error: implicit declaration of function ‘hci_devcoredump_append’; did you mean ‘hci_devcd_append’? [-Werror=implicit-function-declaration]
1136 | hci_devcoredump_append(hu->hdev, skb);
| ^~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append
drivers/bluetooth/hci_qca.c:1150:4: error: implicit declaration of function ‘hci_devcoredump_complete’; did you mean ‘hci_devcd_complete’? [-Werror=implicit-function-declaration]
1150 | hci_devcoredump_complete(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_complete
drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’:
drivers/bluetooth/hci_qca.c:1561:3: error: implicit declaration of function ‘hci_devcoredump_abort’; did you mean ‘hci_devcd_abort’? [-Werror=implicit-function-declaration]
1561 | hci_devcoredump_abort(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~
| hci_devcd_abort
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump_enabled’:
drivers/bluetooth/hci_qca.c:1732:25: error: ‘struct device’ has no member named ‘coredump_disabled’
1732 | return !hu->serdev->dev.coredump_disabled;
| ^
drivers/bluetooth/hci_qca.c: In function ‘qca_setup’:
drivers/bluetooth/hci_qca.c:1874:17: error: ‘struct hci_devcoredump’ has no member named ‘enabled’
1874 | hu->hdev->dump.enabled = hciqca_coredump_enabled;
| ^
drivers/bluetooth/hci_qca.c:1914:2: error: implicit declaration of function ‘hci_devcoredump_register’; did you mean ‘hci_devcd_register’? [-Werror=implicit-function-declaration]
1914 | hci_devcoredump_register(hdev, hci_coredump_qca, qca_dmp_hdr, NULL);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_register
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump’:
drivers/bluetooth/hci_qca.c:2487:10: error: ‘struct device’ has no member named ‘coredump_disabled’
2487 | if (!dev->coredump_disabled && hdev->dump.coredump)
| ^~
cc1: some warnings being treated as errors
make[3]: *** [scripts/Makefile.build:252: drivers/bluetooth/hci_qca.o] Error 1
make[2]: *** [scripts/Makefile.build:494: drivers/bluetooth] Error 2
make[1]: *** [scripts/Makefile.build:494: drivers] Error 2
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:2025: .] Error 2
##############################
Test: CheckAllWarning - FAIL
Desc: Run linux kernel with all warning enabled
Output:

drivers/bluetooth/hci_qca.c:981:5: warning: no previous prototype for ‘qca_dmp_hdr’ [-Wmissing-prototypes]
981 | int qca_dmp_hdr(struct hci_dev *hdev, char *buf, size_t size)
| ^~~~~~~~~~~
drivers/bluetooth/hci_qca.c: In function ‘qca_controller_memdump’:
drivers/bluetooth/hci_qca.c:1070:10: error: implicit declaration of function ‘hci_devcoredump_init’; did you mean ‘hci_devcd_init’? [-Werror=implicit-function-declaration]
1070 | ret = hci_devcoredump_init(hu->hdev, qca_memdump->ram_dump_size);
| ^~~~~~~~~~~~~~~~~~~~
| hci_devcd_init
drivers/bluetooth/hci_qca.c:1119:4: error: implicit declaration of function ‘hci_devcoredump_append_pattern’; did you mean ‘hci_devcd_append_pattern’? [-Werror=implicit-function-declaration]
1119 | hci_devcoredump_append_pattern(hu->hdev, 0x00,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append_pattern
drivers/bluetooth/hci_qca.c:1136:4: error: implicit declaration of function ‘hci_devcoredump_append’; did you mean ‘hci_devcd_append’? [-Werror=implicit-function-declaration]
1136 | hci_devcoredump_append(hu->hdev, skb);
| ^~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append
drivers/bluetooth/hci_qca.c:1150:4: error: implicit declaration of function ‘hci_devcoredump_complete’; did you mean ‘hci_devcd_complete’? [-Werror=implicit-function-declaration]
1150 | hci_devcoredump_complete(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_complete
drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’:
drivers/bluetooth/hci_qca.c:1561:3: error: implicit declaration of function ‘hci_devcoredump_abort’; did you mean ‘hci_devcd_abort’? [-Werror=implicit-function-declaration]
1561 | hci_devcoredump_abort(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~
| hci_devcd_abort
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump_enabled’:
drivers/bluetooth/hci_qca.c:1732:25: error: ‘struct device’ has no member named ‘coredump_disabled’
1732 | return !hu->serdev->dev.coredump_disabled;
| ^
drivers/bluetooth/hci_qca.c: In function ‘qca_setup’:
drivers/bluetooth/hci_qca.c:1874:17: error: ‘struct hci_devcoredump’ has no member named ‘enabled’
1874 | hu->hdev->dump.enabled = hciqca_coredump_enabled;
| ^
drivers/bluetooth/hci_qca.c:1914:2: error: implicit declaration of function ‘hci_devcoredump_register’; did you mean ‘hci_devcd_register’? [-Werror=implicit-function-declaration]
1914 | hci_devcoredump_register(hdev, hci_coredump_qca, qca_dmp_hdr, NULL);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_register
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump’:
drivers/bluetooth/hci_qca.c:2487:10: error: ‘struct device’ has no member named ‘coredump_disabled’
2487 | if (!dev->coredump_disabled && hdev->dump.coredump)
| ^~
cc1: some warnings being treated as errors
make[3]: *** [scripts/Makefile.build:252: drivers/bluetooth/hci_qca.o] Error 1
make[2]: *** [scripts/Makefile.build:494: drivers/bluetooth] Error 2
make[1]: *** [scripts/Makefile.build:494: drivers] Error 2
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:2025: .] Error 2
##############################
Test: CheckSparse - FAIL
Desc: Run sparse tool with linux kernel
Output:

net/bluetooth/af_bluetooth.c:178:25: warning: context imbalance in 'bt_accept_enqueue' - different lock contexts for basic block
drivers/bluetooth/hci_qca.c: In function ‘qca_controller_memdump’:
drivers/bluetooth/hci_qca.c:1070:10: error: implicit declaration of function ‘hci_devcoredump_init’; did you mean ‘hci_devcd_init’? [-Werror=implicit-function-declaration]
1070 | ret = hci_devcoredump_init(hu->hdev, qca_memdump->ram_dump_size);
| ^~~~~~~~~~~~~~~~~~~~
| hci_devcd_init
drivers/bluetooth/hci_qca.c:1119:4: error: implicit declaration of function ‘hci_devcoredump_append_pattern’; did you mean ‘hci_devcd_append_pattern’? [-Werror=implicit-function-declaration]
1119 | hci_devcoredump_append_pattern(hu->hdev, 0x00,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append_pattern
drivers/bluetooth/hci_qca.c:1136:4: error: implicit declaration of function ‘hci_devcoredump_append’; did you mean ‘hci_devcd_append’? [-Werror=implicit-function-declaration]
1136 | hci_devcoredump_append(hu->hdev, skb);
| ^~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append
drivers/bluetooth/hci_qca.c:1150:4: error: implicit declaration of function ‘hci_devcoredump_complete’; did you mean ‘hci_devcd_complete’? [-Werror=implicit-function-declaration]
1150 | hci_devcoredump_complete(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_complete
drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’:
drivers/bluetooth/hci_qca.c:1561:3: error: implicit declaration of function ‘hci_devcoredump_abort’; did you mean ‘hci_devcd_abort’? [-Werror=implicit-function-declaration]
1561 | hci_devcoredump_abort(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~
| hci_devcd_abort
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump_enabled’:
drivers/bluetooth/hci_qca.c:1732:25: error: ‘struct device’ has no member named ‘coredump_disabled’
1732 | return !hu->serdev->dev.coredump_disabled;
| ^
drivers/bluetooth/hci_qca.c: In function ‘qca_setup’:
drivers/bluetooth/hci_qca.c:1874:17: error: ‘struct hci_devcoredump’ has no member named ‘enabled’
1874 | hu->hdev->dump.enabled = hciqca_coredump_enabled;
| ^
drivers/bluetooth/hci_qca.c:1914:2: error: implicit declaration of function ‘hci_devcoredump_register’; did you mean ‘hci_devcd_register’? [-Werror=implicit-function-declaration]
1914 | hci_devcoredump_register(hdev, hci_coredump_qca, qca_dmp_hdr, NULL);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_register
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump’:
drivers/bluetooth/hci_qca.c:2487:10: error: ‘struct device’ has no member named ‘coredump_disabled’
2487 | if (!dev->coredump_disabled && hdev->dump.coredump)
| ^~
cc1: some warnings being treated as errors
make[3]: *** [scripts/Makefile.build:252: drivers/bluetooth/hci_qca.o] Error 1
make[2]: *** [scripts/Makefile.build:494: drivers/bluetooth] Error 2
make[1]: *** [scripts/Makefile.build:494: drivers] Error 2
make[1]: *** Waiting for unfinished jobs....
net/bluetooth/hci_event.c: note: in included file (through include/net/bluetooth/hci_core.h):
./include/net/bluetooth/hci.h:2646:47: warning: array of flexible structures
./include/net/bluetooth/hci.h:2732:43: warning: array of flexible structures
net/bluetooth/hci_codec.c: note: in included file:
./include/net/bluetooth/hci_core.h:150:35: warning: array of flexible structures
net/bluetooth/sco.c: note: in included file:
./include/net/bluetooth/hci_core.h:150:35: warning: array of flexible structures
make: *** [Makefile:2025: .] Error 2
##############################
Test: CheckSmatch - FAIL
Desc: Run smatch tool with source
Output:

drivers/bluetooth/hci_qca.c: In function ‘qca_controller_memdump’:
drivers/bluetooth/hci_qca.c:1070:10: error: implicit declaration of function ‘hci_devcoredump_init’; did you mean ‘hci_devcd_init’? [-Werror=implicit-function-declaration]
1070 | ret = hci_devcoredump_init(hu->hdev, qca_memdump->ram_dump_size);
| ^~~~~~~~~~~~~~~~~~~~
| hci_devcd_init
drivers/bluetooth/hci_qca.c:1119:4: error: implicit declaration of function ‘hci_devcoredump_append_pattern’; did you mean ‘hci_devcd_append_pattern’? [-Werror=implicit-function-declaration]
1119 | hci_devcoredump_append_pattern(hu->hdev, 0x00,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append_pattern
drivers/bluetooth/hci_qca.c:1136:4: error: implicit declaration of function ‘hci_devcoredump_append’; did you mean ‘hci_devcd_append’? [-Werror=implicit-function-declaration]
1136 | hci_devcoredump_append(hu->hdev, skb);
| ^~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append
drivers/bluetooth/hci_qca.c:1150:4: error: implicit declaration of function ‘hci_devcoredump_complete’; did you mean ‘hci_devcd_complete’? [-Werror=implicit-function-declaration]
1150 | hci_devcoredump_complete(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_complete
drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’:
drivers/bluetooth/hci_qca.c:1561:3: error: implicit declaration of function ‘hci_devcoredump_abort’; did you mean ‘hci_devcd_abort’? [-Werror=implicit-function-declaration]
1561 | hci_devcoredump_abort(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~
| hci_devcd_abort
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump_enabled’:
drivers/bluetooth/hci_qca.c:1732:25: error: ‘struct device’ has no member named ‘coredump_disabled’
1732 | return !hu->serdev->dev.coredump_disabled;
| ^
drivers/bluetooth/hci_qca.c: In function ‘qca_setup’:
drivers/bluetooth/hci_qca.c:1874:17: error: ‘struct hci_devcoredump’ has no member named ‘enabled’
1874 | hu->hdev->dump.enabled = hciqca_coredump_enabled;
| ^
drivers/bluetooth/hci_qca.c:1914:2: error: implicit declaration of function ‘hci_devcoredump_register’; did you mean ‘hci_devcd_register’? [-Werror=implicit-function-declaration]
1914 | hci_devcoredump_register(hdev, hci_coredump_qca, qca_dmp_hdr, NULL);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_register
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump’:
drivers/bluetooth/hci_qca.c:2487:10: error: ‘struct device’ has no member named ‘coredump_disabled’
2487 | if (!dev->coredump_disabled && hdev->dump.coredump)
| ^~
cc1: some warnings being treated as errors
make[3]: *** [scripts/Makefile.build:252: drivers/bluetooth/hci_qca.o] Error 1
make[2]: *** [scripts/Makefile.build:494: drivers/bluetooth] Error 2
make[1]: *** [scripts/Makefile.build:494: drivers] Error 2
make[1]: *** Waiting for unfinished jobs....
net/bluetooth/hci_event.c: note: in included file (through include/net/bluetooth/hci_core.h):
./include/net/bluetooth/hci.h:2646:47: warning: array of flexible structures
./include/net/bluetooth/hci.h:2732:43: warning: array of flexible structures
net/bluetooth/hci_codec.c: note: in included file:
./include/net/bluetooth/hci_core.h:150:35: warning: array of flexible structures
net/bluetooth/sco.c: note: in included file:
./include/net/bluetooth/hci_core.h:150:35: warning: array of flexible structures
make: *** [Makefile:2025: .] Error 2
##############################
Test: BuildKernel32 - FAIL
Desc: Build 32bit Kernel for Bluetooth
Output:

drivers/bluetooth/hci_qca.c: In function ‘qca_controller_memdump’:
drivers/bluetooth/hci_qca.c:1070:10: error: implicit declaration of function ‘hci_devcoredump_init’; did you mean ‘hci_devcd_init’? [-Werror=implicit-function-declaration]
1070 | ret = hci_devcoredump_init(hu->hdev, qca_memdump->ram_dump_size);
| ^~~~~~~~~~~~~~~~~~~~
| hci_devcd_init
drivers/bluetooth/hci_qca.c:1119:4: error: implicit declaration of function ‘hci_devcoredump_append_pattern’; did you mean ‘hci_devcd_append_pattern’? [-Werror=implicit-function-declaration]
1119 | hci_devcoredump_append_pattern(hu->hdev, 0x00,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append_pattern
drivers/bluetooth/hci_qca.c:1136:4: error: implicit declaration of function ‘hci_devcoredump_append’; did you mean ‘hci_devcd_append’? [-Werror=implicit-function-declaration]
1136 | hci_devcoredump_append(hu->hdev, skb);
| ^~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_append
drivers/bluetooth/hci_qca.c:1150:4: error: implicit declaration of function ‘hci_devcoredump_complete’; did you mean ‘hci_devcd_complete’? [-Werror=implicit-function-declaration]
1150 | hci_devcoredump_complete(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_complete
drivers/bluetooth/hci_qca.c: In function ‘qca_hw_error’:
drivers/bluetooth/hci_qca.c:1561:3: error: implicit declaration of function ‘hci_devcoredump_abort’; did you mean ‘hci_devcd_abort’? [-Werror=implicit-function-declaration]
1561 | hci_devcoredump_abort(hu->hdev);
| ^~~~~~~~~~~~~~~~~~~~~
| hci_devcd_abort
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump_enabled’:
drivers/bluetooth/hci_qca.c:1732:25: error: ‘struct device’ has no member named ‘coredump_disabled’
1732 | return !hu->serdev->dev.coredump_disabled;
| ^
drivers/bluetooth/hci_qca.c: In function ‘qca_setup’:
drivers/bluetooth/hci_qca.c:1874:17: error: ‘struct hci_devcoredump’ has no member named ‘enabled’
1874 | hu->hdev->dump.enabled = hciqca_coredump_enabled;
| ^
drivers/bluetooth/hci_qca.c:1914:2: error: implicit declaration of function ‘hci_devcoredump_register’; did you mean ‘hci_devcd_register’? [-Werror=implicit-function-declaration]
1914 | hci_devcoredump_register(hdev, hci_coredump_qca, qca_dmp_hdr, NULL);
| ^~~~~~~~~~~~~~~~~~~~~~~~
| hci_devcd_register
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump’:
drivers/bluetooth/hci_qca.c:2487:10: error: ‘struct device’ has no member named ‘coredump_disabled’
2487 | if (!dev->coredump_disabled && hdev->dump.coredump)
| ^~
cc1: some warnings being treated as errors
make[3]: *** [scripts/Makefile.build:252: drivers/bluetooth/hci_qca.o] Error 1
make[2]: *** [scripts/Makefile.build:494: drivers/bluetooth] Error 2
make[1]: *** [scripts/Makefile.build:494: drivers] Error 2
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:2025: .] Error 2
##############################
Test: IncrementalBuild - FAIL
Desc: Incremental build with the patches in the series
Output:
[v1,1/2] Bluetooth: hci_qca: Add qcomm devcoredump sysfs support

drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump_enabled’:
drivers/bluetooth/hci_qca.c:1710:25: error: ‘struct device’ has no member named ‘coredump_disabled’
1710 | return !hu->serdev->dev.coredump_disabled;
| ^
drivers/bluetooth/hci_qca.c: In function ‘qca_setup’:
drivers/bluetooth/hci_qca.c:1793:17: error: ‘struct hci_devcoredump’ has no member named ‘enabled’
1793 | hu->hdev->dump.enabled = hciqca_coredump_enabled;
| ^
drivers/bluetooth/hci_qca.c: In function ‘hciqca_coredump’:
drivers/bluetooth/hci_qca.c:2403:10: error: ‘struct device’ has no member named ‘coredump_disabled’
2403 | if (!dev->coredump_disabled && hdev->dump.coredump)
| ^~
make[3]: *** [scripts/Makefile.build:252: drivers/bluetooth/hci_qca.o] Error 1
make[2]: *** [scripts/Makefile.build:494: drivers/bluetooth] Error 2
make[1]: *** [scripts/Makefile.build:494: drivers] Error 2
make[1]: *** Waiting for unfinished jobs....
make: *** [Makefile:2025: .] Error 2


---
Regards,
Linux Bluetooth