ExternalVendorCode/genode
1
0
Fork 0
mirror of https://github.com/genodelabs/genode.git synced 2024-12-18 21:27:56 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

vbox: improve network tx throughput

Browse Source 
When multiple threads (EMT-0..X + nic_ep) enter the very same
critical section, the use of RTCritSectTryEnter may reflect the contention
case to the Network Model (E1000). Since no one notifies the model, when the
critical section is free again, solely the next packet/event triggered by the
guest will resume the former operation. This may lead to long delays until
packets are sent actually.

Instead of using the RTCritSectTryEnter use RTCritSecEnter to avoid the
situation. All of our network code is non blocking, so the network backend
will only be contented a short time.

Follow up commit to

Issue #5045
This commit is contained in:
Alexander Boettcher 2023-12-01 15:20:13 +01:00 committed by Norman Feske
parent 342e48115e
commit ce66e12699
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
repos/ports/src/virtualbox5/network.cpp
View File

@ -336,7 +336,7 @@ class Nic_client
static DECLCALLBACK(int) drvNicNetworkUp_BeginXmit(PPDMINETWORKUP pInterface, bool fOnWorkerThread)
{
PDRVNIC pThis = PDMINETWORKUP_2_DRVNIC(pInterface);
int rc = RTCritSectTryEnter(&pThis->XmitLock);
int rc = RTCritSectEnter(&pThis->XmitLock);
if (RT_FAILURE(rc))
rc = VERR_TRY_AGAIN;
return rc;

2
repos/ports/src/virtualbox6/network.cc
View File

@ -351,7 +351,7 @@ class Nic_client
static DECLCALLBACK(int) drvNicNetworkUp_BeginXmit(PPDMINETWORKUP pInterface, bool fOnWorkerThread)
{
PDRVNIC pThis = PDMINETWORKUP_2_DRVNIC(pInterface);
int rc = RTCritSectTryEnter(&pThis->XmitLock);
int rc = RTCritSectEnter(&pThis->XmitLock);
if (RT_FAILURE(rc))
rc = VERR_TRY_AGAIN;
return rc;