stplib.py: Should wildcard non-relevant bits in the mask field of por…

…t mod message for OpenFlow 1.2+.

Signed-off-by: FUJITA Tomonori <[email protected]>

ryu/lib/stplib.py

@@ -1151,7 +1151,10 @@ def set_port_status(self, port, state):
         config = {ofproto_v1_2: PORT_CONFIG_V1_2,
                   ofproto_v1_3: PORT_CONFIG_V1_3}
 
-        mask = 0b1111111
+        # Only turn on the relevant bits defined on OpenFlow 1.2+, otherwise
+        # some switch that follows the specification strictly will report
+        # OFPPMFC_BAD_CONFIG error.
+        mask = 0b1100101
         msg = parser.OFPPortMod(self.dp, port.port_no, port.hw_addr,
                                 config[ofp][state], mask, port.advertised)
         self.dp.send_msg(msg)