2022-09-22 22:38:53

by Gix, Brian

[permalink] [raw]
Subject: [PATCH BlueZ v4 1/2] mesh: Improve PB-ADV timing for reliability

Because provisioning is not speed dependent, Timing on outbound PB-ADV
packets have been modified to be less likely missed by remote controlers
with looser timing capabilities.
---
mesh/pb-adv.c | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/mesh/pb-adv.c b/mesh/pb-adv.c
index 83f922aa8..180b16258 100644
--- a/mesh/pb-adv.c
+++ b/mesh/pb-adv.c
@@ -23,6 +23,8 @@
#include "mesh/provision.h"
#include "mesh/pb-adv.h"

+#include "mesh/util.h"
+

struct pb_adv_session {
mesh_prov_open_func_t open_cb;
@@ -158,7 +160,7 @@ static void send_adv_segs(struct pb_adv_session *session, const uint8_t *data,
l_debug("max_seg: %2.2x", max_seg);
l_debug("size: %2.2x, CRC: %2.2x", size, buf[9]);

- pb_adv_send(session, MESH_IO_TX_COUNT_UNLIMITED, 200,
+ pb_adv_send(session, MESH_IO_TX_COUNT_UNLIMITED, 500,
buf, init_size + 10);

consumed = init_size;
@@ -174,7 +176,7 @@ static void send_adv_segs(struct pb_adv_session *session, const uint8_t *data,
buf[6] = (i << 2) | 0x02;
memcpy(buf + 7, data + consumed, seg_size);

- pb_adv_send(session, MESH_IO_TX_COUNT_UNLIMITED, 200,
+ pb_adv_send(session, MESH_IO_TX_COUNT_UNLIMITED, 500,
buf, seg_size + 7);

consumed += seg_size;
@@ -270,7 +272,8 @@ static void send_ack(struct pb_adv_session *session, uint8_t trans_num)
ack.trans_num = trans_num;
ack.opcode = PB_ADV_ACK;

- pb_adv_send(session, 1, 100, &ack, sizeof(ack));
+ pb_adv_send(session, MESH_IO_TX_COUNT_UNLIMITED, 500,
+ &ack, sizeof(ack));
}

static void send_close_ind(struct pb_adv_session *session, uint8_t reason)
--
2.37.3


2022-09-22 23:34:43

by bluez.test.bot

[permalink] [raw]
Subject: RE: Mesh demon switched to using kernel Mesh MGMT

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

---Test result---

Test Summary:
CheckPatch PASS 4.12 seconds
GitLint PASS 2.16 seconds
Prep - Setup ELL PASS 26.03 seconds
Build - Prep PASS 0.78 seconds
Build - Configure PASS 8.19 seconds
Build - Make PASS 719.47 seconds
Make Check PASS 11.86 seconds
Make Check w/Valgrind PASS 287.40 seconds
Make Distcheck PASS 237.01 seconds
Build w/ext ELL - Configure PASS 8.28 seconds
Build w/ext ELL - Make PASS 83.69 seconds
Incremental Build w/ patches PASS 197.42 seconds
Scan Build WARNING 523.64 seconds

Details
##############################
Test: Scan Build - WARNING
Desc: Run Scan Build with patches
Output:
*****************************************************************************
The bugs reported by the scan-build may or may not be caused by your patches.
Please check the list and fix the bugs if they are caused by your patch.
*****************************************************************************
mesh/mesh-io.c:118:18: warning: Access to field 'rx_regs' results in a dereference of a null pointer (loaded from variable 'io')
l_queue_destroy(io->rx_regs, l_free);
^~~~~~~~~~~
1 warning generated.




---
Regards,
Linux Bluetooth