search

zephyriot / zephyr-issues

0 stars 0 forks source link

Connection Timeout during pairing #1109

Closed nashif closed 7 years ago

nashif commented 7 years ago

Reported by Mariusz Skamra:

In the TC_BROB_OBSV_BV_06_C test case, IUT after self-initiated connection with PTS starts pairing with authentication. The pairing process is interrupted after sending SMP Master Identification by HCI Event Disconnection Complete with reason Connection Timeout.

Test is passing on QEMU (with Broadcom controller) but failing on Arduino 101 with Zephyr-based HCI firmware. I enclose failing traces and passing QEMU traces as reference.

TC affected: TC_BROB_OBSV_BV_06_C TC_DISC_RPA_BV_01_C TC_CONN_PRDA_BV_01_C

GATT Test Cases that involve pairing: TC_GAR_SR_BI_05_C, TC_GAR_SR_BI_11_C, TC_GAR_SR_BI_17_C, TC_GAR_SR_BI_22_C,

(Imported from Jira ZEP-1221)

nashif commented 7 years ago

by Mariusz Skamra:

nashif commented 7 years ago

by Mariusz Skamra:

nashif commented 7 years ago

by Mariusz Skamra:

nashif commented 7 years ago

by Mariusz Skamra:

nashif commented 7 years ago

by Mariusz Skamra:

nashif commented 7 years ago

by Mariusz Skamra:

nashif commented 7 years ago

by Carles Cufi:

Please add logs from the Zephyr side of things

nashif commented 7 years ago

by Mariusz Skamra:

Updated list of Test Cases affected: TC_BROB_OBSV_BV_06_C, TC_DISC_RPA_BV_01_C, TC_CONN_UCON_BV_06_C, TC_CONN_PRDA_BV_01_C, TC_GAR_SR_BI_05_C, TC_GAR_SR_BI_11_C, TC_GAR_SR_BI_17_C, TC_GAR_SR_BI_22_C, TC_GAR_SR_BI_27_C, TC_GAR_SR_BI_33_C, TC_GAW_SR_BV_02_C, TC_GAW_SR_BI_01_C, TC_GAW_SR_BI_06_C, TC_GAW_SR_BI_13_C, TC_GAW_SR_BI_19_C, TC_GAW_SR_BI_24_C, TC_GAW_SR_BI_31_C

nashif commented 7 years ago

by Carles Cufi:

After some analysis, this looks like the Host doesn't have enough buffers to process the incoming SMP packets and the ATT request, so the latter is dropped.

nashif commented 7 years ago

by Carles Cufi:

Assigning to Johan Hedberg since this seems to be a host issue with packets being dropped. Szymon Janc could you close it if you can verify that it's fixed?

nashif commented 7 years ago

by Mariusz Skamra:

I've tested today with 1ec994f32070aeaec4f526519c4212617ea39e2b on top. Here are the results: 1/17 GAP TC_BROB_OBSV_BV_06_C PASS 2/17 GAP TC_DISC_RPA_BV_01_C PASS 3/17 GAP TC_CONN_UCON_BV_06_C PASS 4/17 GAP TC_CONN_PRDA_BV_01_C PASS 5/17 GATT TC_GAR_SR_BI_05_C PASS 6/17 GATT TC_GAR_SR_BI_11_C PASS 7/17 GATT TC_GAR_SR_BI_17_C PASS 8/17 GATT TC_GAR_SR_BI_22_C PASS 9/17 GATT TC_GAR_SR_BI_27_C PASS 10/17 GATT TC_GAR_SR_BI_33_C PASS 11/17 GATT TC_GAW_SR_BV_02_C PASS 12/17 GATT TC_GAW_SR_BI_01_C PASS 13/17 GATT TC_GAW_SR_BI_06_C PASS 14/17 GATT TC_GAW_SR_BI_13_C PASS 15/17 GATT TC_GAW_SR_BI_19_C PASS 16/17 GATT TC_GAW_SR_BI_24_C PASS 17/17 GATT TC_GAW_SR_BI_31_C PASS

Summary:

Status Count

PASS 17

Total 17

Johan Hedberg I'll click 'verify' on this if you'll resolve this issue.