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 /src/plugins/unittest | |
| 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 'src/plugins/unittest')
| -rw-r--r-- | src/plugins/unittest/segment_manager_test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/plugins/unittest/segment_manager_test.c b/src/plugins/unittest/segment_manager_test.c index c6f52972e51..31b417aef24 100644 --- a/src/plugins/unittest/segment_manager_test.c +++ b/src/plugins/unittest/segment_manager_test.c @@ -448,7 +448,7 @@ segment_manager_test_fifo_balanced_alloc (vlib_main_t * vm, } /* add another 2MB segment */ - fs_index = segment_manager_add_segment (sm, size_2MB); + fs_index = segment_manager_add_segment (sm, size_2MB, 0); SEG_MGR_TEST ((fs_index == 1), "fs_index %d", fs_index); /* allocate fifos : 4KB x2 |