{"action":"created","issue":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/4538","repository_url":"https://api.github.com/repos/ARMmbed/mbed-os","labels_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/4538/labels{/name}","comments_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/4538/comments","events_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/4538/events","html_url":"https://github.com/ARMmbed/mbed-os/pull/4538","id":235539570,"number":4538,"title":"Avoid lock collision b/w SerialBase & UARTSerial","user":{"login":"hasnainvirk","id":13431190,"avatar_url":"https://avatars0.githubusercontent.com/u/13431190?v=3","gravatar_id":"","url":"https://api.github.com/users/hasnainvirk","html_url":"https://github.com/hasnainvirk","followers_url":"https://api.github.com/users/hasnainvirk/followers","following_url":"https://api.github.com/users/hasnainvirk/following{/other_user}","gists_url":"https://api.github.com/users/hasnainvirk/gists{/gist_id}","starred_url":"https://api.github.com/users/hasnainvirk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hasnainvirk/subscriptions","organizations_url":"https://api.github.com/users/hasnainvirk/orgs","repos_url":"https://api.github.com/users/hasnainvirk/repos","events_url":"https://api.github.com/users/hasnainvirk/events{/privacy}","received_events_url":"https://api.github.com/users/hasnainvirk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-06-13T12:38:33Z","updated_at":"2017-06-13T18:28:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/pulls/4538","html_url":"https://github.com/ARMmbed/mbed-os/pull/4538","diff_url":"https://github.com/ARMmbed/mbed-os/pull/4538.diff","patch_url":"https://github.com/ARMmbed/mbed-os/pull/4538.patch"},"body":"## Description\r\nFixes issue #4537. SerialBase and UARTSerial happened to have same names\r\nfor the mutex locking that confused the system of holding a mutex in interrupt context.\r\nUARTSerial has to change so as to provide empty implementations for lock() and unlock()\r\nbecuase it uses SerialBase from interrupt context only or from its own critical section,\r\nso no extra locks required.\r\nPrivate locks for UARTSerial itself are renamed to api_lock() and api_unlock().\r\n\r\n\r\n## Status\r\n**READY**\r\n\r\n\r\n## Migrations\r\nDoesn't change behaviour.\r\n\r\nNO\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/comments/308206587","html_url":"https://github.com/ARMmbed/mbed-os/pull/4538#issuecomment-308206587","issue_url":"https://api.github.com/repos/ARMmbed/mbed-os/issues/4538","id":308206587,"user":{"login":"mbed-bot","id":12690234,"avatar_url":"https://avatars2.githubusercontent.com/u/12690234?v=3","gravatar_id":"","url":"https://api.github.com/users/mbed-bot","html_url":"https://github.com/mbed-bot","followers_url":"https://api.github.com/users/mbed-bot/followers","following_url":"https://api.github.com/users/mbed-bot/following{/other_user}","gists_url":"https://api.github.com/users/mbed-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/mbed-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mbed-bot/subscriptions","organizations_url":"https://api.github.com/users/mbed-bot/orgs","repos_url":"https://api.github.com/users/mbed-bot/repos","events_url":"https://api.github.com/users/mbed-bot/events{/privacy}","received_events_url":"https://api.github.com/users/mbed-bot/received_events","type":"User","site_admin":false},"created_at":"2017-06-13T18:28:16Z","updated_at":"2017-06-13T18:28:16Z","body":"# [Result: SUCCESS](http://mbed-ci-master-2.austin.arm.com:8081/job/pr_pipeline/543/)\n\nYour command has finished executing! Here's what you wrote!\n\n> /morph test\n\n## Output\n\n[mbed Build Number: 543](http://mbedosci.cloudapp.net/results/pr/4538/543)\n\nAll builds and test passed!"}}
{"actor":{"display_login":"mbed-bot"}}