Fix Quectel M26 context hang #11717

iliapenev · 2019-10-20T21:54:18Z

Fix Quectel M26 hang on connect().

_at.lock() is acquired but not released in QUECTEL_M26::get_sim_state(SimState &state)

Tested with custom Quectel MC60, which uses Quectel M26 driver

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

ciarmcom · 2019-10-20T23:00:26Z

@iliapenev, thank you for your changes.
@ARMmbed/mbed-os-wan @ARMmbed/mbed-os-maintainers please review.

0xc0170 · 2019-10-21T07:44:26Z

CI started

mbed-ci · 2019-10-21T08:40:38Z

Test run: FAILED

Summary: 1 of 11 test jobs failed
Build number : 1
Build artifacts

Failed test jobs:

0xc0170 · 2019-10-21T09:06:06Z

Exporters restarted (internal error)

Fix Quectel M26 context hang

ad1387b

iliapenev mentioned this pull request Oct 20, 2019

Example not running on custom Quectel port ARMmbed/mbed-os-example-cellular#164

Closed

ciarmcom requested review from a team October 20, 2019 23:00

ciarmcom added the needs: review label Oct 20, 2019

kivaisan approved these changes Oct 21, 2019

View reviewed changes

0xc0170 approved these changes Oct 21, 2019

View reviewed changes

0xc0170 added needs: CI release-version: 5.14.2 and removed needs: review labels Oct 21, 2019

0xc0170 removed the needs: CI label Oct 21, 2019

0xc0170 merged commit 9534327 into ARMmbed:master Oct 21, 2019