zephyriot / zep-jira14

0 stars 0 forks source link

Connection Failed to be Established #1232

Closed nashif closed 7 years ago

nashif commented 7 years ago

Reported by Mariusz Skamra:

n51 dongle with Zephyr-based HCI fw.

I run TC_GAD_CL_BV_02_C test case in which connection with PTS is established several times. After n-th connection attempt it failed to be established. I enclose debug logs and HCI traces.

(Imported from Jira ZEP-1355)

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 Vinayak Kariappa Chettimada:

Mariusz Skamra and/or Szymon Janc :

  1. Can you re-run the test with Zephyr build not using continuous scan window in the create connection command? -- in the btsnoop log I see scan interval is 60ms and window of 60 ms, if 60ms and 50 ms, for example, get the test to pass 100%, then the issue could be related to the controller trying to send connect_req PDU too close to scan window being closed.
  2. Please collect sniffer logs for both reproducing the issue and with modified IUT that does not use continuous scan.
nashif commented 7 years ago

by Mariusz Skamra:

Logs provided in "My Capture Files.zip" Contains sniffer traces, HCI logs and debug logs running TC_GAD_CL_BV_02_C test case. I was able to pass without continuous scan first time, but it also failed in this configuration running TC for the second time (both logs attached).

nashif commented 7 years ago

by Carles Cufi:

Can you try this patch? https://gerrit.zephyrproject.org/r/#/c/7746/

nashif commented 7 years ago

by Mariusz Skamra:

I've tested your patch, see [^GH-1213_1.zip] 4 out of 9 runs failed, so there is progress.

nashif commented 7 years ago

by Mariusz Skamra:

Today's test on v1.6.0 tagged 6/10 PASS

nashif commented 7 years ago

by Mark Linkmeyer:

Correcting the priority field

nashif commented 7 years ago

by Carles Cufi:

Mariusz Skamra this should be fixed now. Can we close the Jira issue?

nashif commented 7 years ago

by Mariusz Skamra:

This issue is fixed now.

nashif commented 7 years ago

by Mariusz Skamra:

Carles Cufi Yes.