________________________________________
From: [email protected] <[email protected]> on 
behalf of Alin Serdean <[email protected]>
Sent: Thursday, July 13, 2017 9:40 PM
To: [email protected]
Subject: [ovs-dev] [PATCH 36/40] datapath-windows: Add annotation for 
OvsCtRelatedEntryCleaner

Make the function `OvsCtRelatedEntryCleaner` aware it is a kstart_routine.

Also, the function is not compliant with the coding standard.

Signed-off-by: Alin Gabriel Serdean <[email protected]>
---
 datapath-windows/ovsext/Conntrack-related.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/datapath-windows/ovsext/Conntrack-related.c 
b/datapath-windows/ovsext/Conntrack-related.c
index 2d95bc2..16ed6f7 100644
--- a/datapath-windows/ovsext/Conntrack-related.c
+++ b/datapath-windows/ovsext/Conntrack-related.c
@@ -22,6 +22,7 @@ static UINT64 ctTotalRelatedEntries;
 static OVS_CT_THREAD_CTX ctRelThreadCtx;
 static PNDIS_RW_LOCK_EX ovsCtRelatedLockObj;
 extern POVS_SWITCH_CONTEXT gOvsSwitchContext;
+KSTART_ROUTINE OvsCtRelatedEntryCleaner;

 static __inline UINT32
 OvsExtractCtRelatedKeyHash(OVS_CT_KEY *key)
@@ -170,12 +171,12 @@ OvsCtRelatedFlush()

 /*
  *----------------------------------------------------------------------------
- * ovsCtRelatedEntryCleaner
+ * OvsCtRelatedEntryCleaner
  *     Runs periodically and cleans up the related connections tracker
  *----------------------------------------------------------------------------
  */
 VOID
-ovsCtRelatedEntryCleaner(PVOID data)
+OvsCtRelatedEntryCleaner(PVOID data)
 {
     POVS_CT_THREAD_CTX context = (POVS_CT_THREAD_CTX)data;
     PLIST_ENTRY link, next;
@@ -249,7 +250,7 @@ OvsInitCtRelated(POVS_SWITCH_CONTEXT context)
     /* Init CT Cleaner Thread */
     KeInitializeEvent(&ctRelThreadCtx.event, NotificationEvent, FALSE);
     status = PsCreateSystemThread(&threadHandle, SYNCHRONIZE, NULL, NULL,
-                                  NULL, ovsCtRelatedEntryCleaner,
+                                  NULL, OvsCtRelatedEntryCleaner,
                                   &ctRelThreadCtx);

     if (status != STATUS_SUCCESS) {
--
2.10.2.windows.1
_______________________________________________

Acked-by: Shashank Ram <[email protected]>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to