* Re: [PATCH BlueZ] tools: Fix mesh-tester to expect end of ADV
2022-09-27 22:55 [PATCH BlueZ] tools: Fix mesh-tester to expect end of ADV Brian Gix
@ 2022-09-27 23:37 ` An, Tedd
2022-09-28 0:14 ` [BlueZ] " bluez.test.bot
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: An, Tedd @ 2022-09-27 23:37 UTC (permalink / raw)
To: Gix, Brian, linux-bluetooth@vger.kernel.org; +Cc: hj.tedd.an@gmail.com
Hi Brian,
I tested on my local machine and not able to reproduce the issue after applying this patch.
Tested-by: Tedd Ho-Jeong An <tedd.an@intel.com>
On Tue, 2022-09-27 at 15:55 -0700, Brian Gix wrote:
> Tester was failing by not clearing the HCI queue of expected events
> ---
> tools/mesh-tester.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/tools/mesh-tester.c b/tools/mesh-tester.c
> index 6781bd565..d7b3b6f9b 100644
> --- a/tools/mesh-tester.c
> +++ b/tools/mesh-tester.c
> @@ -1345,6 +1345,10 @@ static const uint8_t send_mesh_cancel_2[] = {
> 0x02
> };
>
> +static const uint8_t mesh_cancel_rsp_param_mesh[] = {
> + 0x00
> +};
> +
> static const struct generic_data mesh_send_mesh_cancel_1 = {
> .send_opcode = MGMT_OP_MESH_SEND_CANCEL,
> .send_param = send_mesh_cancel_1,
> @@ -1353,6 +1357,10 @@ static const struct generic_data mesh_send_mesh_cancel_1 = {
> .expect_alt_ev = MGMT_EV_MESH_PACKET_CMPLT,
> .expect_alt_ev_param = send_mesh_cancel_1,
> .expect_alt_ev_len = sizeof(send_mesh_cancel_1),
> +
> + .expect_hci_command = BT_HCI_CMD_LE_SET_ADV_ENABLE,
> + .expect_hci_param = mesh_cancel_rsp_param_mesh,
> + .expect_hci_len = sizeof(mesh_cancel_rsp_param_mesh),
> };
>
> static const struct generic_data mesh_send_mesh_cancel_2 = {
> @@ -1363,6 +1371,10 @@ static const struct generic_data mesh_send_mesh_cancel_2 = {
> .expect_alt_ev = MGMT_EV_MESH_PACKET_CMPLT,
> .expect_alt_ev_param = send_mesh_cancel_2,
> .expect_alt_ev_len = sizeof(send_mesh_cancel_2),
> +
> + .expect_hci_command = BT_HCI_CMD_LE_SET_ADV_ENABLE,
> + .expect_hci_param = mesh_cancel_rsp_param_mesh,
> + .expect_hci_len = sizeof(mesh_cancel_rsp_param_mesh),
> };
>
> static void setup_multi_mesh_send(const void *test_data)
Regards,
Tedd Ho-Jeong An
^ permalink raw reply [flat|nested] 5+ messages in thread* RE: [BlueZ] tools: Fix mesh-tester to expect end of ADV
2022-09-27 22:55 [PATCH BlueZ] tools: Fix mesh-tester to expect end of ADV Brian Gix
2022-09-27 23:37 ` An, Tedd
@ 2022-09-28 0:14 ` bluez.test.bot
2022-09-28 13:50 ` [PATCH BlueZ] " patchwork-bot+bluetooth
2022-09-28 17:26 ` Gix, Brian
3 siblings, 0 replies; 5+ messages in thread
From: bluez.test.bot @ 2022-09-28 0:14 UTC (permalink / raw)
To: linux-bluetooth, brian.gix
[-- Attachment #1: Type: text/plain, Size: 1047 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=681242
---Test result---
Test Summary:
CheckPatch PASS 1.16 seconds
GitLint PASS 0.75 seconds
Prep - Setup ELL PASS 27.19 seconds
Build - Prep PASS 0.73 seconds
Build - Configure PASS 8.47 seconds
Build - Make PASS 951.52 seconds
Make Check PASS 12.09 seconds
Make Check w/Valgrind PASS 292.27 seconds
Make Distcheck PASS 239.91 seconds
Build w/ext ELL - Configure PASS 8.58 seconds
Build w/ext ELL - Make PASS 84.60 seconds
Incremental Build w/ patches PASS 0.00 seconds
Scan Build PASS 503.88 seconds
---
Regards,
Linux Bluetooth
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH BlueZ] tools: Fix mesh-tester to expect end of ADV
2022-09-27 22:55 [PATCH BlueZ] tools: Fix mesh-tester to expect end of ADV Brian Gix
2022-09-27 23:37 ` An, Tedd
2022-09-28 0:14 ` [BlueZ] " bluez.test.bot
@ 2022-09-28 13:50 ` patchwork-bot+bluetooth
2022-09-28 17:26 ` Gix, Brian
3 siblings, 0 replies; 5+ messages in thread
From: patchwork-bot+bluetooth @ 2022-09-28 13:50 UTC (permalink / raw)
To: Brian Gix; +Cc: linux-bluetooth, hj.tedd.an, tedd.an
Hello:
This patch was applied to bluetooth/bluez.git (master)
by Brian Gix <brian.gix@intel.com>:
On Tue, 27 Sep 2022 15:55:14 -0700 you wrote:
> Tester was failing by not clearing the HCI queue of expected events
> ---
> tools/mesh-tester.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
Here is the summary with links:
- [BlueZ] tools: Fix mesh-tester to expect end of ADV
https://git.kernel.org/pub/scm/bluetooth/bluez.git/?id=3b47cf5db9a6
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH BlueZ] tools: Fix mesh-tester to expect end of ADV
2022-09-27 22:55 [PATCH BlueZ] tools: Fix mesh-tester to expect end of ADV Brian Gix
` (2 preceding siblings ...)
2022-09-28 13:50 ` [PATCH BlueZ] " patchwork-bot+bluetooth
@ 2022-09-28 17:26 ` Gix, Brian
3 siblings, 0 replies; 5+ messages in thread
From: Gix, Brian @ 2022-09-28 17:26 UTC (permalink / raw)
To: linux-bluetooth@vger.kernel.org; +Cc: hj.tedd.an@gmail.com, An, Tedd
Hi Tedd,
This patch was upstreamed to the BlueZ tree. Could you add mesh-tester
to CI?
On Tue, 2022-09-27 at 15:55 -0700, Brian Gix wrote:
> Tester was failing by not clearing the HCI queue of expected events
> ---
> tools/mesh-tester.c | 12 ++++++++++++
> 1 file changed, 12 insertions(+)
>
> diff --git a/tools/mesh-tester.c b/tools/mesh-tester.c
> index 6781bd565..d7b3b6f9b 100644
> --- a/tools/mesh-tester.c
> +++ b/tools/mesh-tester.c
> @@ -1345,6 +1345,10 @@ static const uint8_t send_mesh_cancel_2[] = {
> 0x02
> };
>
> +static const uint8_t mesh_cancel_rsp_param_mesh[] = {
> + 0x00
> +};
> +
> static const struct generic_data mesh_send_mesh_cancel_1 = {
> .send_opcode = MGMT_OP_MESH_SEND_CANCEL,
> .send_param = send_mesh_cancel_1,
> @@ -1353,6 +1357,10 @@ static const struct generic_data
> mesh_send_mesh_cancel_1 = {
> .expect_alt_ev = MGMT_EV_MESH_PACKET_CMPLT,
> .expect_alt_ev_param = send_mesh_cancel_1,
> .expect_alt_ev_len = sizeof(send_mesh_cancel_1),
> +
> + .expect_hci_command = BT_HCI_CMD_LE_SET_ADV_ENABLE,
> + .expect_hci_param = mesh_cancel_rsp_param_mesh,
> + .expect_hci_len = sizeof(mesh_cancel_rsp_param_mesh),
> };
>
> static const struct generic_data mesh_send_mesh_cancel_2 = {
> @@ -1363,6 +1371,10 @@ static const struct generic_data
> mesh_send_mesh_cancel_2 = {
> .expect_alt_ev = MGMT_EV_MESH_PACKET_CMPLT,
> .expect_alt_ev_param = send_mesh_cancel_2,
> .expect_alt_ev_len = sizeof(send_mesh_cancel_2),
> +
> + .expect_hci_command = BT_HCI_CMD_LE_SET_ADV_ENABLE,
> + .expect_hci_param = mesh_cancel_rsp_param_mesh,
> + .expect_hci_len = sizeof(mesh_cancel_rsp_param_mesh),
> };
>
> static void setup_multi_mesh_send(const void *test_data)
^ permalink raw reply [flat|nested] 5+ messages in thread