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

[win-pv-devel] [PATCH 3/4] Remove update of defunct statistic



Nothing looks a the 'InNDISMax' value, yet the receiver code still jumps
through potentially performance damaging hoops to update it. This patch
finally blows it away.

Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>
---
 src/xennet/receiver.c | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/src/xennet/receiver.c b/src/xennet/receiver.c
index ba05799..693d1ac 100644
--- a/src/xennet/receiver.c
+++ b/src/xennet/receiver.c
@@ -46,7 +46,6 @@ struct _XENNET_RECEIVER {
     PNET_BUFFER_LIST            PutList;
     PNET_BUFFER_LIST            GetList[HVM_MAX_VCPUS];
     LONG                        InNDIS;
-    LONG                        InNDISMax;
     XENVIF_VIF_OFFLOAD_OPTIONS  OffloadOptions;
 };
 
@@ -287,19 +286,6 @@ __ReceiverPushPacket(
     if (InNDIS > IN_NDIS_MAX)
         Flags |= NDIS_RECEIVE_FLAGS_RESOURCES;
 
-    for (;;) {
-        LONG    InNDISMax;
-
-        InNDISMax = Receiver->InNDISMax;
-        KeMemoryBarrier();
-
-        if (InNDIS <= InNDISMax)
-            break;
-
-        if (InterlockedCompareExchange(&Receiver->InNDISMax, InNDIS, 
InNDISMax) == InNDISMax)
-            break;
-    }
-
     NdisMIndicateReceiveNetBufferLists(AdapterGetHandle(Receiver->Adapter),
                                        NetBufferList,
                                        NDIS_DEFAULT_PORT_NUMBER,
-- 
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®.