From: Jesse Gross <jesse@nicira.com>
Date: Tue, 1 Nov 2011 22:35:31 +0000 (-0700)
Subject: datapath: Drop useless WARN_ON_ONCE during flow conversion.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=a7d7f493ba47b1986857b57103f75dccf65d548a;p=openvswitch

datapath: Drop useless WARN_ON_ONCE during flow conversion.

This checks whether key_len is not zero but we set the key length
at the beginning of the function, so I don't see this as a useful
check.

Signed-off-by: Jesse Gross <jesse@nicira.com>
Acked-by: Ben Pfaff <blp@nicira.com>
---

diff --git a/datapath/flow.c b/datapath/flow.c
index 95297a52..9e0b842e 100644
--- a/datapath/flow.c
+++ b/datapath/flow.c
@@ -1143,7 +1143,6 @@ invalid:
 	error = -EINVAL;
 
 ok:
-	WARN_ON_ONCE(!key_len && !error);
 	*key_lenp = key_len;
 	return error;
 }