Code Review
/
vpp.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
session: prioritize postponed sessions
[vpp.git]
/
src
/
vnet
/
session
/
segment_manager.c
diff --git
a/src/vnet/session/segment_manager.c
b/src/vnet/session/segment_manager.c
index
83d1963
..
6386645
100644
(file)
--- a/
src/vnet/session/segment_manager.c
+++ b/
src/vnet/session/segment_manager.c
@@
-82,7
+82,7
@@
segment_manager_del_segment (segment_manager_t * sm,
ssvm_delete (&fs->ssvm);
if (CLIB_DEBUG)
ssvm_delete (&fs->ssvm);
if (CLIB_DEBUG)
- memset (fs, 0xfb, sizeof (*fs));
+
clib_
memset (fs, 0xfb, sizeof (*fs));
pool_put (sm->segments, fs);
}
pool_put (sm->segments, fs);
}
@@
-182,7
+182,7
@@
segment_manager_add_segment (segment_manager_t * sm, u32 segment_size)
{
pool_get (sm->segments, seg);
}
{
pool_get (sm->segments, seg);
}
- memset (seg, 0, sizeof (*seg));
+
clib_
memset (seg, 0, sizeof (*seg));
/*
* Initialize ssvm segment and svm fifo private header
/*
* Initialize ssvm segment and svm fifo private header
@@
-193,7
+193,7
@@
segment_manager_add_segment (segment_manager_t * sm, u32 segment_size)
if (props->segment_type != SSVM_SEGMENT_PRIVATE)
{
seg_name = format (0, "%d-%d%c", getpid (), segment_name_counter++, 0);
if (props->segment_type != SSVM_SEGMENT_PRIVATE)
{
seg_name = format (0, "%d-%d%c", getpid (), segment_name_counter++, 0);
- alloc_size = segment_size + rnd_margin;
+ alloc_size =
(uword)
segment_size + rnd_margin;
baseva = clib_valloc_alloc (&smm->va_allocator, alloc_size, 0);
if (!baseva)
{
baseva = clib_valloc_alloc (&smm->va_allocator, alloc_size, 0);
if (!baseva)
{
@@
-238,7
+238,7
@@
segment_manager_new ()
segment_manager_main_t *smm = &segment_manager_main;
segment_manager_t *sm;
pool_get (smm->segment_managers, sm);
segment_manager_main_t *smm = &segment_manager_main;
segment_manager_t *sm;
pool_get (smm->segment_managers, sm);
- memset (sm, 0, sizeof (*sm));
+
clib_
memset (sm, 0, sizeof (*sm));
clib_rwlock_init (&sm->segments_rwlock);
return sm;
}
clib_rwlock_init (&sm->segments_rwlock);
return sm;
}
@@
-253,8
+253,7
@@
segment_manager_init (segment_manager_t * sm, u32 first_seg_size,
{
u32 rx_fifo_size, tx_fifo_size, pair_size;
u32 rx_rounded_data_size, tx_rounded_data_size;
{
u32 rx_fifo_size, tx_fifo_size, pair_size;
u32 rx_rounded_data_size, tx_rounded_data_size;
- u64 approx_total_size, max_seg_size =
- ((u64) 1 << 32) - clib_mem_get_page_size ();
+ u64 approx_total_size, max_seg_size = ((u64) 1 << 32) - (128 << 10);
segment_manager_properties_t *props;
svm_fifo_segment_private_t *segment;
u32 approx_segment_count;
segment_manager_properties_t *props;
svm_fifo_segment_private_t *segment;
u32 approx_segment_count;
@@
-367,11
+366,11
@@
segment_manager_del_sessions (segment_manager_t * sm)
*/
while (fifo)
{
*/
while (fifo)
{
- if (fifo->
master_thread_index == 255
)
+ if (fifo->
ct_session_index != SVM_FIFO_INVALID_SESSION_INDEX
)
{
svm_fifo_t *next = fifo->next;
application_local_session_disconnect_w_index (sm->app_wrk_index,
{
svm_fifo_t *next = fifo->next;
application_local_session_disconnect_w_index (sm->app_wrk_index,
- fifo->
master
_session_index);
+ fifo->
ct
_session_index);
fifo = next;
continue;
}
fifo = next;
continue;
}
@@
-420,7
+419,7
@@
segment_manager_del (segment_manager_t * sm)
clib_rwlock_free (&sm->segments_rwlock);
if (CLIB_DEBUG)
clib_rwlock_free (&sm->segments_rwlock);
if (CLIB_DEBUG)
- memset (sm, 0xfe, sizeof (*sm));
+
clib_
memset (sm, 0xfe, sizeof (*sm));
pool_put (smm->segment_managers, sm);
}
pool_put (smm->segment_managers, sm);
}
@@
-592,7
+591,9
@@
segment_manager_dealloc_fifos (u32 segment_index, svm_fifo_t * rx_fifo,
/* Remove segment manager if no sessions and detached from app */
if (segment_manager_app_detached (sm)
&& !segment_manager_has_fifos (sm))
/* Remove segment manager if no sessions and detached from app */
if (segment_manager_app_detached (sm)
&& !segment_manager_has_fifos (sm))
- segment_manager_del (sm);
+ {
+ segment_manager_del (sm);
+ }
}
else
segment_manager_segment_reader_unlock (sm);
}
else
segment_manager_segment_reader_unlock (sm);