From: Pravin B Shelar <pshelar@nicira.com>
Date: Fri, 1 Mar 2013 00:15:00 +0000 (-0800)
Subject: datapath: Increase maximum allocation size of action list.
X-Git-Tag: v1.11.0~336
X-Git-Url: http://git.cascardo.eti.br/?a=commitdiff_plain;h=9ccb22ec5f0a9edcba39a1c60469582496a70f8c;p=cascardo%2Fovs.git

datapath: Increase maximum allocation size of action list.

The switch to flow based tunneling increased the size of each output
action in the flow action list.  In extreme cases, this can result
in the action list exceeding the maximum buffer size.
This doubles the maximum buffer size to compensate for the increase
in action size.
Action list is recieved from netlink callback which is allocating
linear-skb, therefore allocating another multi-page buffer would
not increase probability of the allocation-failure a lot.

Signed-off-by: Pravin B Shelar <pshelar@nicira.com>
Acked-by: Jesse Gross <jesse@nicira.com>

Bug #15203
---

diff --git a/datapath/flow.h b/datapath/flow.h
index 694964051..887b6d4b3 100644
--- a/datapath/flow.h
+++ b/datapath/flow.h
@@ -194,7 +194,7 @@ int ovs_flow_from_nlattrs(struct sw_flow_key *swkey, int *key_lenp,
 int ovs_flow_metadata_from_nlattrs(struct sw_flow *flow, int key_len,
 				   const struct nlattr *attr);
 
-#define MAX_ACTIONS_BUFSIZE	(16 * 1024)
+#define MAX_ACTIONS_BUFSIZE	(32 * 1024)
 #define TBL_MIN_BUCKETS		1024
 
 struct flow_table {