Failed read remote used features

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.86k stars 6.62k forks source link

Failed read remote used features #2708

Closed zephyrbot closed 7 years ago

zephyrbot commented 8 years ago

Reported by Mariusz Skamra:

Test pass on QEMU.

IUT is not sending L2CAP Connection Parameter Update Request, because don't receive LE Read Remote Used Features event.

Test Cases affected: TC_LE_CPU_BV_01_C, TC_CONN_CPUP_BV_01_C, TC_CONN_CPUP_BV_02_C, TC_CONN_CPUP_BV_03_C

Traces attached. Those includes failing with Arduino 101 with Zephyr-based HCI fw and passing on QEMU

(Imported from Jira ZEP-1220)

zephyrbot commented 8 years ago

by Mariusz Skamra:

001bdc0606ca_1.zip (size: 1806204)

zephyrbot commented 8 years ago

by Mariusz Skamra:

001bdc0606ca_2.zip (size: 1867785)

zephyrbot commented 8 years ago

by Mariusz Skamra:

09112016.btsnoop (size: 2926)

zephyrbot commented 8 years ago

by Mariusz Skamra:

logs.btsnoop (size: 2706)

zephyrbot commented 8 years ago

by Mariusz Skamra:

TC_LE_CPU_BV_01_C_2016_11_07_10_55_44.zip (size: 41018)

zephyrbot commented 8 years ago

by Carles Cufi:

The command is successful and we get a command status for it. Then the actual event never comes, we need to debug it. There is no assert in the link layer since the disconneciton complete event arrives fine some time later.

Vinayak Kariappa Chettimada : any ideas so far?

zephyrbot commented 8 years ago

by Carles Cufi:

Has this been fixed by the change we pushed?

zephyrbot commented 8 years ago

by Carles Cufi:

Szymon Janc , Mariusz Skamra please close this if resolved by patch pushed

zephyrbot commented 7 years ago

by Mariusz Skamra:

Patches merged into bluetooth and v1.6.0-branch branches. Tests are passing

1/4 L2CAP TC_LE_CPU_BV_01_C PASS 2/4 GAP TC_CONN_CPUP_BV_01_C PASS 3/4 GAP TC_CONN_CPUP_BV_02_C PASS 4/4 GAP TC_CONN_CPUP_BV_03_C PASS

zephyrbot commented 7 years ago

by Sharron LIU:

Please reporter Mariusz Skamra verify this. Thanks.

zephyrbot commented 7 years ago

by Mariusz Skamra:

Sharron LIU how can I verify this, since there is no "Verify" button? TBH I already verified this adding my previous comment and changing status to "Merged".

zephyrbot commented 7 years ago

by Mark Linkmeyer:

Fixing incorrect priority