ofproto: Mark NXT_ACT_SET_CONFIG and NXT_ACT_GET_CONFIG obsolete.

These Nicira-specific requests have not been implemented for some time.
In case we need them later we can always reimplement them.

diff --git a/include/openflow/nicira-ext.h b/include/openflow/nicira-ext.h
index a62f59e14c99eb764018d2446a8c7b4941201682..41cd7ec91f9ec7b47ca6b0a6d6472ae230b68177 100644 (file)
--- a/include/openflow/nicira-ext.h
+++ b/include/openflow/nicira-ext.h
@@ -37,12 +37,11 @@ enum nicira_type {
      * pairs in the form "key=value\n". */
     NXT_STATUS_REPLY,
 
-    /* Configure an action.  Most actions do not require configuration
-     * beyond that supplied in the actual action call. */
-    NXT_ACT_SET_CONFIG,
+    /* No longer used. */
+    NXT_ACT_SET_CONFIG__OBSOLETE,
 
-    /* Get configuration of action. */
-    NXT_ACT_GET_CONFIG,
+    /* No longer used. */
+    NXT_ACT_GET_CONFIG__OBSOLETE,
 
     /* No longer used. */
     NXT_COMMAND_REQUEST__OBSOLETE,

diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c
index 5af7dcf4b46c0c3c3991a949cb74915fdb153775..10b4796f19a2a2e3d3667f313eef564dca7d22b6 100644 (file)
--- a/ofproto/ofproto.c
+++ b/ofproto/ofproto.c
@@ -2929,12 +2929,6 @@ handle_vendor(struct ofproto *p, struct ofconn *ofconn, void *msg)
     case NXT_STATUS_REQUEST:
         return switch_status_handle_request(p->switch_status, ofconn->rconn,
                                             msg);
-
-    case NXT_ACT_SET_CONFIG:
-        return ofp_mkerr(OFPET_BAD_REQUEST, OFPBRC_BAD_SUBTYPE); /* XXX */
-
-    case NXT_ACT_GET_CONFIG:
-        return ofp_mkerr(OFPET_BAD_REQUEST, OFPBRC_BAD_SUBTYPE); /* XXX */
     }
 
     return ofp_mkerr(OFPET_BAD_REQUEST, OFPBRC_BAD_SUBTYPE);