[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))
         __TransmitterRingReleaseLock(Ring);
 }
 
-- 
2.1.1


_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/cgi-bin/mailman/listinfo/win-pv-devel

 


Rackspace

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