From: Ben Pfaff Date: Thu, 16 Apr 2015 16:01:54 +0000 (-0700) Subject: bridge: Remove unused macro OFP_PORT_ACTION_WINDOW. X-Git-Tag: v2.4.0~291 X-Git-Url: http://git.cascardo.eti.br/?p=cascardo%2Fovs.git;a=commitdiff_plain;h=f4fa820c2111c654474c8d14f2e8ce938300309a bridge: Remove unused macro OFP_PORT_ACTION_WINDOW. I guess that this was missed when the corresponding feature was removed. Reported-by: David Evans Signed-off-by: Ben Pfaff Acked-by: Russell Bryant --- diff --git a/AUTHORS b/AUTHORS index d17ae7acb..a2b43886e 100644 --- a/AUTHORS +++ b/AUTHORS @@ -228,6 +228,7 @@ Chunhe Li lichunhe@huawei.com Ciara Loftus ciara.loftus@intel.com Daniel Badea daniel.badea@windriver.com Dave Walker DaveWalker@ubuntu.com +David Evans davidjoshuaevans@gmail.com David Palma palma@onesource.pt Derek Cormier derek.cormier@lab.ntt.co.jp Dhaval Badiani dbadiani@vmware.com diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c index fa30513bd..5dfe91c63 100644 --- a/vswitchd/bridge.c +++ b/vswitchd/bridge.c @@ -217,16 +217,6 @@ static long long int stats_timer = LLONG_MIN; #define AA_REFRESH_INTERVAL (1000) /* In milliseconds. */ static long long int aa_refresh_timer = LLONG_MIN; -/* In some datapaths, creating and destroying OpenFlow ports can be extremely - * expensive. This can cause bridge_reconfigure() to take a long time during - * which no other work can be done. To deal with this problem, we limit port - * adds and deletions to a window of OFP_PORT_ACTION_WINDOW milliseconds per - * call to bridge_reconfigure(). If there is more work to do after the limit - * is reached, 'need_reconfigure', is flagged and it's done on the next loop. - * This allows the rest of the code to catch up on important things like - * forwarding packets. */ -#define OFP_PORT_ACTION_WINDOW 10 - static void add_del_bridges(const struct ovsrec_open_vswitch *); static void bridge_run__(void); static void bridge_create(const struct ovsrec_bridge *);