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.03k stars 6.17k forks source link

GATT/CL/GAR/BV-10-C: IUT failed to send back to back ATT_READ_MULTIPLE_VARIABLE_LENGTH_REQ #74485

Open chandra1787 opened 2 weeks ago

chandra1787 commented 2 weeks ago

Describe the bug According to GATT test spec zephyr(IUT) has to send two ATT_READ_MULTIPLE_VARIABLE_LENGTH_REQ commands to PTS, but in this case IUT sent only one ATT_READ_MULTIPLE_VARIABLE_LENGTH_REQ command to PTS. PTS waited for more than 3 mins and connection got disconnected. Same issue observed for GATT/CL/GAR/BV-11-C as well

Platform: NRF52833-dk

To Reproduce Steps to reproduce the behavior:

  1. Please execute the below 2 test cases with PTS GATT/CL/GAR/BV-10-C and GATT/CL/GAR/BV-11-C

Expected behavior IUT has to send multiple ATT_READ_MULTIPLE_VARIABLE_LENGTH_REQ commands to PTS

Impact Zephyr host stack conformance tests fails

Logs and console output Please find the attached fail logs with PTS)

Environment (please complete the following information):

github-actions[bot] commented 2 weeks ago

Hi @chandra1787! We appreciate you submitting your first issue for our open-source project. 🌟

Even though I'm a bot, I can assure you that the whole community is genuinely grateful for your time and effort. 🤖💙

sjanc commented 2 weeks ago

This requires implementing (rewriting) GATT client part in tester from legacy GATT service (ID 2), to GATT client service (ID 6).

legacy BTP GATT service is not able to handle multiple GATT operations due to design issues.

chandra1787 commented 1 week ago

Hi sjanc, You mean to say Zephyr stack doesn't support these test cases and yet to implement? Or if any code change required to pass these test cases, could you please share with us which file and where to modify.

sjanc commented 1 week ago

I mean that tester application which is used by PTS automation (tests/bluetooth/tester) doesn't support required BTP interface to handle this.

It is not simple fix to re-implement gatt client to new BTP service, so for time being I'd suggest to run those tests manually (ie via shell) if you need them for qualification

chandra1787 commented 1 week ago

Could you please provide documents path how to run manually. We couldn't find any document or readme for manual execution.

jhedberg commented 1 week ago

@sjanc @chandra1787 seems to me like this shouldn't then be a bug report? Can it be closed, or do you e.g. want me to convert it to at GitHub discussion?

jori-nordic commented 3 days ago

It is an issue, but a feature request rather. Changing the labels.