diff options
author | Ajay Davanageri <ajay.davanageri@broadcom.com> | 2022-04-21 18:25:09 +0530 |
---|---|---|
committer | Roger Wang <wangroger@google.com> | 2022-04-22 09:31:58 +0800 |
commit | d3af3e99ef5dad16766e30cf42864dd61444e546 (patch) | |
tree | 8987a14fdc784de083b291867636c651c0903710 | |
parent | 46e29fefb507bfdec16517e202c2d3a00378cb02 (diff) | |
download | wlan-d3af3e99ef5dad16766e30cf42864dd61444e546.tar.gz |
Fix for deadlock between hal cmd and event based cmd.
Local pthread_mutex_lock in handleResponse() is not required,
as all the vendor hal apis from legacy Hal are protected with the GlobalLock .
bug:228092853
Test: Verified on oriole platform
Signed-off-by: Ajay Davanageri <ajay.davanageri@broadcom.com>
Change-Id: I77df86dd3f794c4e260af60d979ff81c7074ef9e
-rwxr-xr-x | bcmdhd/wifi_hal/cpp_bindings.cpp | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/bcmdhd/wifi_hal/cpp_bindings.cpp b/bcmdhd/wifi_hal/cpp_bindings.cpp index 5311e6e..7a634bd 100755 --- a/bcmdhd/wifi_hal/cpp_bindings.cpp +++ b/bcmdhd/wifi_hal/cpp_bindings.cpp @@ -647,7 +647,6 @@ int WifiCommand::requestResponse() { } int WifiCommand::requestResponse(WifiRequest& request) { - pthread_mutex_lock(&ResponseMutex); int err = 0; struct nl_cb *cb = nl_cb_alloc(NL_CB_DEFAULT); @@ -674,7 +673,6 @@ int WifiCommand::requestResponse(WifiRequest& request) { } out: nl_cb_put(cb); - pthread_mutex_unlock(&ResponseMutex); return mapErrorCodes(err); } |