Samuka007/asterinas
1
0
Fork 0
mirror of https://github.com/asterinas/asterinas.git synced 2025-06-22 08:53:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

Set hash buckets as 2 to work around performance bottleneck

Browse Source
This commit is contained in:
jiangjianfeng
2024-12-31 02:55:30 +00:00
committed by Tate, Hongliang Tian
parent f509c546f9
commit 631ab21838
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
kernel/libs/aster-bigtcp/src/socket_table.rs
View File

@ -147,9 +147,11 @@ pub(crate) struct SocketTable<E: Ext> {
// On Linux, the number of buckets is determined at runtime based on the available memory. // On Linux, the number of buckets is determined at runtime based on the available memory.
// For simplicity, we use fixed values here. // For simplicity, we use fixed values here.
// The bucket count should be a power of 2 to ensure efficient modulo calculations. // The bucket count should be a power of 2 to ensure efficient modulo calculations.
const LISTENER_BUCKET_COUNT: u32 = 64; // FIXME: We have reduced the bucket count to 2 because iterating over all buckets has become a bottleneck.
// Once we can avoid such iterations, we should set more meaningful bucket counts.
const LISTENER_BUCKET_COUNT: u32 = 2;
const LISTENER_BUCKET_MASK: u32 = LISTENER_BUCKET_COUNT - 1; const LISTENER_BUCKET_MASK: u32 = LISTENER_BUCKET_COUNT - 1;
const CONNECTION_BUCKET_COUNT: u32 = 8192; const CONNECTION_BUCKET_COUNT: u32 = 2;
const CONNECTION_BUCKET_MASK: u32 = CONNECTION_BUCKET_COUNT - 1; const CONNECTION_BUCKET_MASK: u32 = CONNECTION_BUCKET_COUNT - 1;
const_assert!(LISTENER_BUCKET_COUNT.is_power_of_two()); const_assert!(LISTENER_BUCKET_COUNT.is_power_of_two());