[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[win-pv-devel] [PATCH 1/2] Revert commit 03dd4642 "Avoid transmitting on the wrong CPU"

It severely impacts performance. The latency of waking up a different CPU
more than defeats the cost of one CPU blocking waiting on another.

Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
 src/xenvif/transmitter.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/xenvif/transmitter.c b/src/xenvif/transmitter.c
index b55daa3..084ca76 100644
--- a/src/xenvif/transmitter.c
+++ b/src/xenvif/transmitter.c
@@ -4008,9 +4008,7 @@ __TransmitterRingQueuePacket(
     // grab it then that's ok because whichever thread is holding it will have 
to call
     // __TransmitterRingReleaseLock() and will therefore drain the atomic 
packet list.
-    if (Ring->Index != KeGetCurrentProcessorNumberEx(NULL))
-        KeInsertQueueDpc(&Ring->Dpc, NULL, NULL);
-    else if (__TransmitterRingTryAcquireLock(Ring))
+    if (__TransmitterRingTryAcquireLock(Ring))

win-pv-devel mailing list



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.