Hi,

Thank you for your posting your patch!

Refining example looks great for me! Thanks!


On 2017年11月20日 21:12, Meng Qingyu wrote:
From: morland <mqy...@hotmail.com>

Signed-off-by: morland <mqy...@hotmail.com>
---
  ryu/ofproto/ofproto_v1_0.py | 1 +
  1 file changed, 1 insertion(+)

diff --git a/ryu/ofproto/ofproto_v1_0.py b/ryu/ofproto/ofproto_v1_0.py
index 52d1b746..21671bc7 100644
--- a/ryu/ofproto/ofproto_v1_0.py
+++ b/ryu/ofproto/ofproto_v1_0.py
@@ -51,6 +51,7 @@ OFPP_ALL = 0xfffc       # All physical ports except input 
port.
  OFPP_CONTROLLER = 0xfffd        # Send to controller.
  OFPP_LOCAL = 0xfffe     # Local openflow "port".
  OFPP_NONE = 0xffff      # Not associated with a physical port.
+OFPT_ALL = 0xffff       # All physical ports. In fact this is a bug from the 
spec of openflow1.0. Already been replaced to OFPP_ANY in openflow 1.1.0

Is this a typo of OFP"P"_ALL? But already "OFPP_ALL = 0xfffc".
(The "OFPT_" prefix means the OpenFlow Protocol message Type, right?)
I guess it is better to use "OFPP_ANY".


Thanks,
Iwase

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to