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