search

zephyrproject-rtos / zephyr

Primary Git Repository for the Zephyr Project. Zephyr is a new generation, scalable, optimized, secure RTOS for multiple hardware architectures.
https://docs.zephyrproject.org
Apache License 2.0
10.55k stars 6.46k forks source link

Connection Failed to be Established #2833

Closed zephyrbot closed 7 years ago

zephyrbot 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)

zephyrbot commented 7 years ago

by Mariusz Skamra:

zephyrbot commented 7 years ago

by Mariusz Skamra:

zephyrbot commented 7 years ago

by Mariusz Skamra:

zephyrbot commented 7 years ago

by Mariusz Skamra:

zephyrbot 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.
zephyrbot 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).

zephyrbot commented 7 years ago

by Carles Cufi:

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

zephyrbot commented 7 years ago

by Mariusz Skamra:

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

zephyrbot commented 7 years ago

by Mariusz Skamra:

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

zephyrbot commented 7 years ago

by Mark Linkmeyer:

Correcting the priority field

zephyrbot commented 7 years ago

by Carles Cufi:

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

zephyrbot commented 7 years ago

by Mariusz Skamra:

This issue is fixed now.

zephyrbot commented 7 years ago

by Mariusz Skamra:

Carles Cufi Yes.