diff options
author | Simon Horman <horms@verge.net.au> | 2012-09-05 11:50:38 +0900 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2012-09-05 10:19:14 -0700 |
commit | c22c56bd746352f5c70a0d99bb3f548d03cfd105 (patch) | |
tree | 229fe18525cc8f3b50b16ad5569a3ae60010868d /lib/ofp-errors.h | |
parent | 2e1bfcb6315704b804add452ca6d1324dac5b45c (diff) |
ofp-errors: Use OFPERR_OFPBRC_BAD_TABLE_ID
* In the case of OpenFlow 1.1+ OFPERR_OFPBRC_BAD_TABLE_ID is defined
in the specification and seems to be the most appropriate error
to use when an unknown table id is encountered.
* In the case of OpenFlow 1.0 no appropriate error message
seems to exist. Perhaps because an invalid port is not possible?
I'm unsure.
In any case, make use of a non-standard error code (1,512).
This was formerly known as OFPERR_NXBRC_BAD_TABLE_ID but
has been rolled into OFPERR_OFPBRC_BAD_TABLE_ID to allow the
latter to be used without concern for the prevailing Open Flow version.
Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: Ben Pfaff <blp@nicira.com>
Diffstat (limited to 'lib/ofp-errors.h')
-rw-r--r-- | lib/ofp-errors.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/lib/ofp-errors.h b/lib/ofp-errors.h index 9a5dbcb0..a883e201 100644 --- a/lib/ofp-errors.h +++ b/lib/ofp-errors.h @@ -112,7 +112,10 @@ enum ofperr { /* OF1.0+(1,8). Specified buffer does not exist. */ OFPERR_OFPBRC_BUFFER_UNKNOWN, - /* OF1.1+(1,9). Specified table-id invalid or does not exist. */ + /* NX1.0(1,512), OF1.1+(1,9). Specified table-id invalid or does not exist. + * [ A non-standard error (1,512), formerly OFPERR_NXBRC_BAD_TABLE_ID, + * is used for OpenFlow 1.0 as there seems to be no appropriste error + * code defined the specification. ] */ OFPERR_OFPBRC_BAD_TABLE_ID, /* OF1.2+(1,10). Denied because controller is slave. */ @@ -134,9 +137,6 @@ enum ofperr { * nxm_hasmask or nxm_length or both, is invalid or not implemented. */ OFPERR_NXBRC_NXM_BAD_TYPE, - /* NX1.0+(1,512). A request specified a nonexistent table ID. */ - OFPERR_NXBRC_BAD_TABLE_ID, - /* NX1.0+(1,515). Must-be-zero field had nonzero value. */ OFPERR_NXBRC_MUST_BE_ZERO, |