You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by GitBox <gi...@apache.org> on 2021/06/08 09:22:27 UTC

[GitHub] [mynewt-nimble] KKopyscinski opened a new pull request #986: Blestress fixes

KKopyscinski opened a new pull request #986:
URL: https://github.com/apache/mynewt-nimble/pull/986


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [mynewt-nimble] sjanc commented on a change in pull request #986: Blestress fixes

Posted by GitBox <gi...@apache.org>.
sjanc commented on a change in pull request #986:
URL: https://github.com/apache/mynewt-nimble/pull/986#discussion_r667918856



##########
File path: apps/blestress/src/tx_stress.c
##########
@@ -848,6 +848,16 @@ tx_stress_9_gap_event(struct ble_gap_event *event, void *arg)
 {
     ble_addr_t addr;
     int test;
+    struct ble_gap_conn_params conn_params = {
+        .scan_itvl = 0x0010,
+        .scan_window = 0x0010,
+        .itvl_min = BLE_GAP_INITIAL_CONN_ITVL_MIN,
+        .itvl_max = BLE_GAP_INITIAL_CONN_ITVL_MAX,
+        .latency = 0,
+        .supervision_timeout = 0x0C80,
+        .min_ce_len = 0x0010,
+        .max_ce_len = 0x0300,
+    };

Review comment:
       are those connections maintained in test for at least this timeout after last one is created?




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [mynewt-nimble] KKopyscinski commented on pull request #986: Blestress fixes

Posted by GitBox <gi...@apache.org>.
KKopyscinski commented on pull request #986:
URL: https://github.com/apache/mynewt-nimble/pull/986#issuecomment-861234131


   All crashes are resolved - the only problem left is silent restart on RX side during test 9 if test 7 was executed before - for now, skip test 7. Reason of this restart is yet unknown - there is no crash visible in debugger, no crash information is printed to console, HCI trace only shows lack of activity - basically, restart causes RTT to deattach.  All tests are working if run independently.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [mynewt-nimble] KKopyscinski commented on pull request #986: Blestress fixes

Posted by GitBox <gi...@apache.org>.
KKopyscinski commented on pull request #986:
URL: https://github.com/apache/mynewt-nimble/pull/986#issuecomment-859558048






-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org