nicira-ext: Correct and extend examples for NXM_OF_VLAN_TCI field.
authorBen Pfaff <blp@nicira.com>
Tue, 23 Nov 2010 18:15:43 +0000 (10:15 -0800)
committerBen Pfaff <blp@nicira.com>
Mon, 13 Dec 2010 21:21:05 +0000 (13:21 -0800)
The final example for this field was wrong.  This corrects it and adds
two more examples.

Reported-by: Natasha Gude <natasha@nicira.com>
include/openflow/nicira-ext.h

index 135f793..cdccb30 100644 (file)
@@ -761,8 +761,14 @@ OFP_ASSERT(sizeof(struct nx_action_note) == 16);
  *   - Testing with a specific PCP and CFI=1, with nxm_mask=0xf000, matches
  *     packets that have an 802.1Q header with that PCP (and any VID).
  *
- *   - Testing with nxm_value=0, nxm_mask=0xe000 matches packets with no 802.1Q
+ *   - Testing with nxm_value=0, nxm_mask=0x0fff matches packets with no 802.1Q
  *     header or with an 802.1Q header with a VID of 0.
+ *
+ *   - Testing with nxm_value=0, nxm_mask=0xe000 matches packets with no 802.1Q
+ *     header or with an 802.1Q header with a PCP of 0.
+ *
+ *   - Testing with nxm_value=0, nxm_mask=0xefff matches packets with no 802.1Q
+ *     header or with an 802.1Q header with both VID and PCP of 0.
  */
 #define NXM_OF_VLAN_TCI   NXM_HEADER  (0x0000,  4, 2)
 #define NXM_OF_VLAN_TCI_W NXM_HEADER_W(0x0000,  4, 2)