[GitHub] [mynewt-nimble] prasad-alatkar commented on pull request #540: nimble/host: Add support to accept or reject pairing request through GAP event

[GitBox <gi...@apache.org>]

prasad-alatkar commented on pull request #540:
URL: https://github.com/apache/mynewt-nimble/pull/540#issuecomment-887586637


   > @prasad-alatkar looks like unit tests are failing. newt test @apache-mynewt-nimble/nimble/host/test could you check that? Also could you please add a patch to btshell which would allow to test this patch? Thanks
   
   @rymanluk , as per my understanding the failure is seen in unit test : `ble_sm_test_case_peer_lgcy_fail_confirm`. However, I am a little confused with the error prints here: `nimble_host_test.elf: nimble/host/src/ble_hs.c:194: ble_hs_lock: Assertion `!ble_hs_locked_by_cur_task()' failed`. It seems that `ble_hs_lock` is getting called twice somewhere, but the PR changes do not add any extra host lock. Could you please help here to pass the CI tests ? I am not very familiar with BT tests here, could you please let me know if it is possible to collect debug logs through CI ? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@mynewt.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org