diff options
author | liuyacan <liuyacan@corp.netease.com> | 2021-04-25 20:11:30 +0800 |
---|---|---|
committer | Florin Coras <florin.coras@gmail.com> | 2021-04-26 04:25:40 +0000 |
commit | 9f299030fd1214eb1fc076cf0c7f44559a7c8f6b (patch) | |
tree | 160e1c524b4e14cfd0c63ae86e9b80a336fc7c50 /test/ext | |
parent | b14c49d2275f6348640572b7b481dad43f3a00d0 (diff) |
session: fix race condition in fifo allocation
Under some timing conditions,VCL may receive CONNECTED/ACCEPTED
event before ADD_SEGMENT event.
Timing example:
2 threads call segment_manager_alloc_session_fifos() parallelly
Thread 1 Thread 2
sm read lock |
| |
try to alloc fifo =>failed |
| |
sm read unlock |
| |
sm write lock |
| |
add segment |
| |
sm write unlock |
| sm read lock
| |
| try to alloc fifo=>successful
sm read lock |
| sm read unlock
| |
| emit CONNECTED/ACCEPTED
emit ADD_SEGMENT event
|
sm read unlock
This commit move ADD_SEGMENT notification under the protection
of the write lock in some scenarios.
Type: fix
Signed-off-by: liuyacan <liuyacan@corp.netease.com>
Change-Id: I25d5475c5e6d37cfccefa9506f6030c26ce8ee9b
Diffstat (limited to 'test/ext')
0 files changed, 0 insertions, 0 deletions