@@ -793,9 +793,9 @@ struct ovs_action_push_eth {
struct ovs_key_ethernet addresses;
};
-#define OVS_ENCAP_NSH_MAX_MD_LEN 248
+#define OVS_PUSH_NSH_MAX_MD_LEN 248
/*
- * struct ovs_action_encap_nsh - %OVS_ACTION_ATTR_ENCAP_NSH
+ * struct ovs_action_push_nsh - %OVS_ACTION_ATTR_PUSH_NSH
* @flags: NSH header flags.
* @mdtype: NSH metadata type.
* @mdlen: Length of NSH metadata in bytes.
@@ -803,7 +803,7 @@ struct ovs_action_push_eth {
* @path_hdr: NSH service path id and service index.
* @metadata: NSH metadata for MD type 1 or 2
*/
-struct ovs_action_encap_nsh {
+struct ovs_action_push_nsh {
uint8_t flags;
uint8_t mdtype;
uint8_t mdlen;
@@ -887,8 +887,8 @@ enum ovs_nat_attr {
* @OVS_ACTION_ATTR_PUSH_ETH: Push a new outermost Ethernet header onto the
* packet.
* @OVS_ACTION_ATTR_POP_ETH: Pop the outermost Ethernet header off the packet.
- * @OVS_ACTION_ATTR_ENCAP_NSH: encap NSH action to push NSH header.
- * @OVS_ACTION_ATTR_DECAP_NSH: decap NSH action to remove NSH header.
+ * @OVS_ACTION_ATTR_PUSH_NSH: push NSH header to the packet.
+ * @OVS_ACTION_ATTR_POP_NSH: pop the outermost NSH header off the packet.
*
* Only a single header can be set with a single %OVS_ACTION_ATTR_SET. Not all
* fields within a header are modifiable, e.g. the IPv4 protocol and fragment
@@ -930,8 +930,8 @@ enum ovs_action_attr {
OVS_ACTION_ATTR_TUNNEL_POP, /* u32 port number. */
OVS_ACTION_ATTR_CLONE, /* Nested OVS_CLONE_ATTR_*. */
OVS_ACTION_ATTR_METER, /* u32 meter number. */
- OVS_ACTION_ATTR_ENCAP_NSH, /* struct ovs_action_encap_nsh. */
- OVS_ACTION_ATTR_DECAP_NSH, /* No argument. */
+ OVS_ACTION_ATTR_PUSH_NSH, /* struct ovs_action_push_nsh. */
+ OVS_ACTION_ATTR_POP_NSH, /* No argument. */
#endif
__OVS_ACTION_ATTR_MAX, /* Nothing past this will be accepted
* from userspace. */
@@ -5407,8 +5407,8 @@ dp_execute_cb(void *aux_, struct dp_packet_batch *packets_,
case OVS_ACTION_ATTR_PUSH_ETH:
case OVS_ACTION_ATTR_POP_ETH:
case OVS_ACTION_ATTR_CLONE:
- case OVS_ACTION_ATTR_ENCAP_NSH:
- case OVS_ACTION_ATTR_DECAP_NSH:
+ case OVS_ACTION_ATTR_PUSH_NSH:
+ case OVS_ACTION_ATTR_POP_NSH:
case __OVS_ACTION_ATTR_MAX:
OVS_NOT_REACHED();
}
@@ -1255,8 +1255,8 @@ dpif_execute_helper_cb(void *aux_, struct dp_packet_batch *packets_,
case OVS_ACTION_ATTR_PUSH_ETH:
case OVS_ACTION_ATTR_POP_ETH:
case OVS_ACTION_ATTR_CLONE:
- case OVS_ACTION_ATTR_ENCAP_NSH:
- case OVS_ACTION_ATTR_DECAP_NSH:
+ case OVS_ACTION_ATTR_PUSH_NSH:
+ case OVS_ACTION_ATTR_POP_NSH:
case OVS_ACTION_ATTR_UNSPEC:
case __OVS_ACTION_ATTR_MAX:
OVS_NOT_REACHED();
@@ -652,8 +652,8 @@ requires_datapath_assistance(const struct nlattr *a)
case OVS_ACTION_ATTR_PUSH_ETH:
case OVS_ACTION_ATTR_POP_ETH:
case OVS_ACTION_ATTR_CLONE:
- case OVS_ACTION_ATTR_ENCAP_NSH:
- case OVS_ACTION_ATTR_DECAP_NSH:
+ case OVS_ACTION_ATTR_PUSH_NSH:
+ case OVS_ACTION_ATTR_POP_NSH:
return false;
case OVS_ACTION_ATTR_UNSPEC:
@@ -818,18 +818,18 @@ odp_execute_actions(void *dp, struct dp_packet_batch *batch, bool steal,
}
break;
- case OVS_ACTION_ATTR_ENCAP_NSH: {
- const struct ovs_action_encap_nsh *enc_nsh = nl_attr_get(a);
+ case OVS_ACTION_ATTR_PUSH_NSH: {
+ const struct ovs_action_push_nsh *oapn = nl_attr_get(a);
DP_PACKET_BATCH_FOR_EACH (packet, batch) {
- encap_nsh(packet, enc_nsh);
+ push_nsh(packet, oapn);
}
break;
}
- case OVS_ACTION_ATTR_DECAP_NSH: {
+ case OVS_ACTION_ATTR_POP_NSH: {
size_t i, num = batch->count;
DP_PACKET_BATCH_REFILL_FOR_EACH (i, num, packet, batch) {
- if (decap_nsh(packet)) {
+ if (pop_nsh(packet)) {
dp_packet_batch_refill(batch, packet, i);
} else {
dp_packet_delete(packet);
@@ -129,8 +129,8 @@ odp_action_len(uint16_t type)
case OVS_ACTION_ATTR_PUSH_ETH: return sizeof(struct ovs_action_push_eth);
case OVS_ACTION_ATTR_POP_ETH: return 0;
case OVS_ACTION_ATTR_CLONE: return ATTR_LEN_VARIABLE;
- case OVS_ACTION_ATTR_ENCAP_NSH: return ATTR_LEN_VARIABLE;
- case OVS_ACTION_ATTR_DECAP_NSH: return 0;
+ case OVS_ACTION_ATTR_PUSH_NSH: return ATTR_LEN_VARIABLE;
+ case OVS_ACTION_ATTR_POP_NSH: return 0;
case OVS_ACTION_ATTR_UNSPEC:
case __OVS_ACTION_ATTR_MAX:
@@ -342,23 +342,23 @@ format_nsh_key_mask(struct ds *ds, const struct ovs_key_nsh *key,
}
static void
-format_odp_encap_nsh_action(struct ds *ds,
- const struct ovs_action_encap_nsh *encap_nsh)
+format_odp_push_nsh_action(struct ds *ds,
+ const struct ovs_action_push_nsh *push_nsh)
{
- uint32_t path_hdr = ntohl(encap_nsh->path_hdr);
+ uint32_t path_hdr = ntohl(push_nsh->path_hdr);
uint32_t spi = (path_hdr & NSH_SPI_MASK) >> NSH_SPI_SHIFT;
uint8_t si = (path_hdr & NSH_SI_MASK) >> NSH_SI_SHIFT;
- ds_put_cstr(ds, "encap_nsh(");
- ds_put_format(ds, "flags=%d", encap_nsh->flags);
- ds_put_format(ds, ",mdtype=%d", encap_nsh->mdtype);
- ds_put_format(ds, ",np=%d", encap_nsh->np);
+ ds_put_cstr(ds, "push_nsh(");
+ ds_put_format(ds, "flags=%d", push_nsh->flags);
+ ds_put_format(ds, ",mdtype=%d", push_nsh->mdtype);
+ ds_put_format(ds, ",np=%d", push_nsh->np);
ds_put_format(ds, ",spi=0x%x", spi);
ds_put_format(ds, ",si=%d", si);
- switch (encap_nsh->mdtype) {
+ switch (push_nsh->mdtype) {
case NSH_M_TYPE1: {
struct nsh_md1_ctx *md1_ctx =
- ALIGNED_CAST(struct nsh_md1_ctx *, encap_nsh->metadata);
+ ALIGNED_CAST(struct nsh_md1_ctx *, push_nsh->metadata);
for (int i = 0; i < 4; i++) {
ds_put_format(ds, ",c%d=0x%x", i + 1,
ntohl(get_16aligned_be32(&md1_ctx->c[i])));
@@ -367,7 +367,7 @@ format_odp_encap_nsh_action(struct ds *ds,
}
case NSH_M_TYPE2:
ds_put_cstr(ds, ",md2=");
- ds_put_hex(ds, encap_nsh->metadata, encap_nsh->mdlen);
+ ds_put_hex(ds, push_nsh->metadata, push_nsh->mdlen);
break;
default:
OVS_NOT_REACHED();
@@ -1057,11 +1057,11 @@ format_odp_action(struct ds *ds, const struct nlattr *a,
case OVS_ACTION_ATTR_CLONE:
format_odp_clone_action(ds, a, portno_names);
break;
- case OVS_ACTION_ATTR_ENCAP_NSH:
- format_odp_encap_nsh_action(ds, nl_attr_get(a));
+ case OVS_ACTION_ATTR_PUSH_NSH:
+ format_odp_push_nsh_action(ds, nl_attr_get(a));
break;
- case OVS_ACTION_ATTR_DECAP_NSH:
- ds_put_cstr(ds, "decap_nsh()");
+ case OVS_ACTION_ATTR_POP_NSH:
+ ds_put_cstr(ds, "pop_nsh()");
break;
case OVS_ACTION_ATTR_UNSPEC:
case __OVS_ACTION_ATTR_MAX:
@@ -1781,27 +1781,27 @@ find_end:
}
static int
-parse_odp_encap_nsh_action(const char *s, struct ofpbuf *actions)
+parse_odp_push_nsh_action(const char *s, struct ofpbuf *actions)
{
int n = 0;
int ret = 0;
- struct ovs_action_encap_nsh *encap_nsh =
- xmalloc(sizeof *encap_nsh + OVS_ENCAP_NSH_MAX_MD_LEN);
+ struct ovs_action_push_nsh *push_nsh =
+ xmalloc(sizeof *push_nsh + OVS_PUSH_NSH_MAX_MD_LEN);
uint32_t spi;
uint8_t si;
uint32_t cd;
- if (!ovs_scan_len(s, &n, "encap_nsh(")) {
+ if (!ovs_scan_len(s, &n, "push_nsh(")) {
ret = -EINVAL;
goto out;
}
/* The default is NSH_M_TYPE1 */
- encap_nsh->flags = 0;
- encap_nsh->mdtype = NSH_M_TYPE1;
- encap_nsh->mdlen = NSH_M_TYPE1_MDLEN;
- encap_nsh->path_hdr = htonl(255);
- memset(encap_nsh->metadata, 0, encap_nsh->mdlen);
+ push_nsh->flags = 0;
+ push_nsh->mdtype = NSH_M_TYPE1;
+ push_nsh->mdlen = NSH_M_TYPE1_MDLEN;
+ push_nsh->path_hdr = htonl(255);
+ memset(push_nsh->metadata, 0, push_nsh->mdlen);
for (;;) {
n += strspn(s + n, delimiters);
@@ -1809,17 +1809,17 @@ parse_odp_encap_nsh_action(const char *s, struct ofpbuf *actions)
break;
}
- if (ovs_scan_len(s, &n, "flags=%"SCNi8, &encap_nsh->flags)) {
+ if (ovs_scan_len(s, &n, "flags=%"SCNi8, &push_nsh->flags)) {
continue;
}
- if (ovs_scan_len(s, &n, "mdtype=%"SCNi8, &encap_nsh->mdtype)) {
- switch (encap_nsh->mdtype) {
+ if (ovs_scan_len(s, &n, "mdtype=%"SCNi8, &push_nsh->mdtype)) {
+ switch (push_nsh->mdtype) {
case NSH_M_TYPE1:
/* This is the default format. */;
break;
case NSH_M_TYPE2:
/* Length will be updated later. */
- encap_nsh->mdlen = 0;
+ push_nsh->mdlen = 0;
break;
default:
ret = -EINVAL;
@@ -1827,24 +1827,24 @@ parse_odp_encap_nsh_action(const char *s, struct ofpbuf *actions)
}
continue;
}
- if (ovs_scan_len(s, &n, "np=%"SCNi8, &encap_nsh->np)) {
+ if (ovs_scan_len(s, &n, "np=%"SCNi8, &push_nsh->np)) {
continue;
}
if (ovs_scan_len(s, &n, "spi=0x%"SCNx32, &spi)) {
- encap_nsh->path_hdr =
+ push_nsh->path_hdr =
htonl(((spi << NSH_SPI_SHIFT) & NSH_SPI_MASK) |
- (ntohl(encap_nsh->path_hdr) & ~NSH_SPI_MASK));
+ (ntohl(push_nsh->path_hdr) & ~NSH_SPI_MASK));
continue;
}
if (ovs_scan_len(s, &n, "si=%"SCNi8, &si)) {
- encap_nsh->path_hdr =
+ push_nsh->path_hdr =
htonl((si << NSH_SI_SHIFT) |
- (ntohl(encap_nsh->path_hdr) & ~NSH_SI_MASK));
+ (ntohl(push_nsh->path_hdr) & ~NSH_SI_MASK));
continue;
}
- if (encap_nsh->mdtype == NSH_M_TYPE1) {
+ if (push_nsh->mdtype == NSH_M_TYPE1) {
struct nsh_md1_ctx *md1 =
- ALIGNED_CAST(struct nsh_md1_ctx *, encap_nsh->metadata);
+ ALIGNED_CAST(struct nsh_md1_ctx *, push_nsh->metadata);
if (ovs_scan_len(s, &n, "c1=0x%"SCNx32, &cd)) {
put_16aligned_be32(&md1->c[0], htonl(cd));
continue;
@@ -1862,15 +1862,15 @@ parse_odp_encap_nsh_action(const char *s, struct ofpbuf *actions)
continue;
}
}
- else if (encap_nsh->mdtype == NSH_M_TYPE2) {
+ else if (push_nsh->mdtype == NSH_M_TYPE2) {
struct ofpbuf b;
char buf[512];
size_t mdlen;
if (ovs_scan_len(s, &n, "md2=0x%511[0-9a-fA-F]", buf)) {
- ofpbuf_use_stub(&b, encap_nsh->metadata,
- OVS_ENCAP_NSH_MAX_MD_LEN);
+ ofpbuf_use_stub(&b, push_nsh->metadata,
+ OVS_PUSH_NSH_MAX_MD_LEN);
ofpbuf_put_hex(&b, buf, &mdlen);
- encap_nsh->mdlen = mdlen;
+ push_nsh->mdlen = mdlen;
ofpbuf_uninit(&b);
}
continue;
@@ -1878,17 +1878,17 @@ parse_odp_encap_nsh_action(const char *s, struct ofpbuf *actions)
}
out:
if (ret >= 0) {
- size_t size = sizeof(struct ovs_action_encap_nsh)
- + ROUND_UP(encap_nsh->mdlen, 4);
- size_t pad_len = size - sizeof(struct ovs_action_encap_nsh)
- - encap_nsh->mdlen;
- if (encap_nsh->mdlen > NSH_M_TYPE1_MDLEN && pad_len > 0) {
- memset(encap_nsh->metadata + encap_nsh->mdlen, 0, pad_len);
- }
- nl_msg_put_unspec(actions, OVS_ACTION_ATTR_ENCAP_NSH, encap_nsh, size);
+ size_t size = sizeof(struct ovs_action_push_nsh)
+ + ROUND_UP(push_nsh->mdlen, 4);
+ size_t pad_len = size - sizeof(struct ovs_action_push_nsh)
+ - push_nsh->mdlen;
+ if (push_nsh->mdlen > NSH_M_TYPE1_MDLEN && pad_len > 0) {
+ memset(push_nsh->metadata + push_nsh->mdlen, 0, pad_len);
+ }
+ nl_msg_put_unspec(actions, OVS_ACTION_ATTR_PUSH_NSH, push_nsh, size);
ret = n;
}
- free(encap_nsh);
+ free(push_nsh);
return ret;
}
@@ -2094,8 +2094,8 @@ parse_odp_action(const char *s, const struct simap *port_names,
}
{
- if (!strncmp(s, "encap_nsh(", 10)) {
- int retval = parse_odp_encap_nsh_action(s, actions);
+ if (!strncmp(s, "push_nsh(", 9)) {
+ int retval = parse_odp_push_nsh_action(s, actions);
if (retval < 0) {
return retval;
}
@@ -2105,8 +2105,8 @@ parse_odp_action(const char *s, const struct simap *port_names,
{
int n;
- if (ovs_scan(s, "decap_nsh()%n", &n)) {
- nl_msg_put_flag(actions, OVS_ACTION_ATTR_DECAP_NSH);
+ if (ovs_scan(s, "pop_nsh()%n", &n)) {
+ nl_msg_put_flag(actions, OVS_ACTION_ATTR_POP_NSH);
return n;
}
}
@@ -6793,32 +6793,32 @@ commit_set_pkt_mark_action(const struct flow *flow, struct flow *base_flow,
}
static void
-odp_put_decap_nsh_action(struct ofpbuf *odp_actions)
+odp_put_pop_nsh_action(struct ofpbuf *odp_actions)
{
- nl_msg_put_flag(odp_actions, OVS_ACTION_ATTR_DECAP_NSH);
+ nl_msg_put_flag(odp_actions, OVS_ACTION_ATTR_POP_NSH);
}
static void
-odp_put_encap_nsh_action(struct ofpbuf *odp_actions,
+odp_put_push_nsh_action(struct ofpbuf *odp_actions,
const struct flow *flow,
struct ofpbuf *encap_data)
{
size_t size;
size_t pad_len;
- struct ovs_action_encap_nsh *encap_nsh =
- xmalloc(sizeof *encap_nsh + OVS_ENCAP_NSH_MAX_MD_LEN);
+ struct ovs_action_push_nsh *push_nsh =
+ xmalloc(sizeof *push_nsh + OVS_PUSH_NSH_MAX_MD_LEN);
- encap_nsh->flags = flow->nsh.flags;
- encap_nsh->mdtype = flow->nsh.mdtype;
- encap_nsh->np = flow->nsh.np;
- encap_nsh->path_hdr = htonl((ntohl(flow->nsh.spi) << NSH_SPI_SHIFT) |
+ push_nsh->flags = flow->nsh.flags;
+ push_nsh->mdtype = flow->nsh.mdtype;
+ push_nsh->np = flow->nsh.np;
+ push_nsh->path_hdr = htonl((ntohl(flow->nsh.spi) << NSH_SPI_SHIFT) |
flow->nsh.si);
- switch (encap_nsh->mdtype) {
+ switch (push_nsh->mdtype) {
case NSH_M_TYPE1: {
struct nsh_md1_ctx *md1 =
- ALIGNED_CAST(struct nsh_md1_ctx *, encap_nsh->metadata);
- encap_nsh->mdlen = NSH_M_TYPE1_MDLEN;
+ ALIGNED_CAST(struct nsh_md1_ctx *, push_nsh->metadata);
+ push_nsh->mdlen = NSH_M_TYPE1_MDLEN;
for (int i = 0; i < 4; i++) {
put_16aligned_be32(&md1->c[i], flow->nsh.c[i]);
}
@@ -6826,26 +6826,26 @@ odp_put_encap_nsh_action(struct ofpbuf *odp_actions,
}
case NSH_M_TYPE2:
if (encap_data) {
- ovs_assert(encap_data->size < OVS_ENCAP_NSH_MAX_MD_LEN);
- encap_nsh->mdlen = encap_data->size;
- memcpy(encap_nsh->metadata, encap_data->data, encap_data->size);
+ ovs_assert(encap_data->size < OVS_PUSH_NSH_MAX_MD_LEN);
+ push_nsh->mdlen = encap_data->size;
+ memcpy(push_nsh->metadata, encap_data->data, encap_data->size);
} else {
- encap_nsh->mdlen = 0;
+ push_nsh->mdlen = 0;
}
break;
default:
- encap_nsh->mdlen = 0;
+ push_nsh->mdlen = 0;
break;
}
- size = sizeof(struct ovs_action_encap_nsh)
- + ROUND_UP(encap_nsh->mdlen, 4);
- pad_len = size - sizeof(struct ovs_action_encap_nsh)
- - encap_nsh->mdlen;
+ size = sizeof(struct ovs_action_push_nsh)
+ + ROUND_UP(push_nsh->mdlen, 4);
+ pad_len = size - sizeof(struct ovs_action_push_nsh)
+ - push_nsh->mdlen;
if (pad_len > 0) {
- memset(encap_nsh->metadata + encap_nsh->mdlen, 0, pad_len);
+ memset(push_nsh->metadata + push_nsh->mdlen, 0, pad_len);
}
- nl_msg_put_unspec(odp_actions, OVS_ACTION_ATTR_ENCAP_NSH, encap_nsh, size);
- free(encap_nsh);
+ nl_msg_put_unspec(odp_actions, OVS_ACTION_ATTR_PUSH_NSH, push_nsh, size);
+ free(push_nsh);
}
static void
@@ -6872,8 +6872,8 @@ commit_packet_type_change(const struct flow *flow,
break;
}
case PT_NSH:
- /* encap_nsh */
- odp_put_encap_nsh_action(odp_actions, flow, encap_data);
+ /* push_nsh */
+ odp_put_push_nsh_action(odp_actions, flow, encap_data);
base_flow->packet_type = flow->packet_type;
/* Update all packet headers in base_flow. */
memcpy(&base_flow->dl_dst, &flow->dl_dst,
@@ -6898,8 +6898,8 @@ commit_packet_type_change(const struct flow *flow,
* No need to update the base flow here. */
switch (ntohl(base_flow->packet_type)) {
case PT_NSH:
- /* decap_nsh. */
- odp_put_decap_nsh_action(odp_actions);
+ /* pop_nsh. */
+ odp_put_pop_nsh_action(odp_actions);
break;
default:
/* Checks are done during translation. */
@@ -403,10 +403,10 @@ pop_mpls(struct dp_packet *packet, ovs_be16 ethtype)
}
void
-encap_nsh(struct dp_packet *packet, const struct ovs_action_encap_nsh *encap)
+push_nsh(struct dp_packet *packet, const struct ovs_action_push_nsh *oapn)
{
struct nsh_hdr *nsh;
- size_t length = NSH_BASE_HDR_LEN + encap->mdlen;
+ size_t length = NSH_BASE_HDR_LEN + oapn->mdlen;
uint8_t next_proto;
switch (ntohl(packet->packet_type)) {
@@ -427,18 +427,18 @@ encap_nsh(struct dp_packet *packet, const struct ovs_action_encap_nsh *encap)
}
nsh = (struct nsh_hdr *) dp_packet_push_uninit(packet, length);
- nsh->ver_flags_len = htons(encap->flags << NSH_FLAGS_SHIFT | length >> 2);
+ nsh->ver_flags_len = htons(oapn->flags << NSH_FLAGS_SHIFT | length >> 2);
nsh->next_proto = next_proto;
- put_16aligned_be32(&nsh->path_hdr, encap->path_hdr);
- nsh->md_type = encap->mdtype;
+ put_16aligned_be32(&nsh->path_hdr, oapn->path_hdr);
+ nsh->md_type = oapn->mdtype;
switch (nsh->md_type) {
case NSH_M_TYPE1:
- nsh->md1 = *ALIGNED_CAST(struct nsh_md1_ctx *, encap->metadata);
+ nsh->md1 = *ALIGNED_CAST(struct nsh_md1_ctx *, oapn->metadata);
break;
case NSH_M_TYPE2: {
- /* The MD2 metadata in encap is already padded to 4 bytes. */
- size_t len = ROUND_UP(encap->mdlen, 4);
- memcpy(&nsh->md2, encap->metadata, len);
+ /* The MD2 metadata in oapn is already padded to 4 bytes. */
+ size_t len = ROUND_UP(oapn->mdlen, 4);
+ memcpy(&nsh->md2, oapn->metadata, len);
break;
}
default:
@@ -451,7 +451,7 @@ encap_nsh(struct dp_packet *packet, const struct ovs_action_encap_nsh *encap)
}
bool
-decap_nsh(struct dp_packet *packet)
+pop_nsh(struct dp_packet *packet)
{
struct nsh_hdr *nsh = (struct nsh_hdr *) dp_packet_l3(packet);
size_t length;
@@ -434,9 +434,9 @@ void push_eth(struct dp_packet *packet, const struct eth_addr *dst,
const struct eth_addr *src);
void pop_eth(struct dp_packet *packet);
-void encap_nsh(struct dp_packet *packet,
- const struct ovs_action_encap_nsh *encap_nsh);
-bool decap_nsh(struct dp_packet *packet);
+void push_nsh(struct dp_packet *packet,
+ const struct ovs_action_push_nsh *oapn);
+bool pop_nsh(struct dp_packet *packet);
#define LLC_DSAP_SNAP 0xaa
#define LLC_SSAP_SNAP 0xaa
@@ -2824,8 +2824,8 @@ dpif_ipfix_read_actions(const struct flow *flow,
case OVS_ACTION_ATTR_POP_MPLS:
case OVS_ACTION_ATTR_PUSH_ETH:
case OVS_ACTION_ATTR_POP_ETH:
- case OVS_ACTION_ATTR_ENCAP_NSH:
- case OVS_ACTION_ATTR_DECAP_NSH:
+ case OVS_ACTION_ATTR_PUSH_NSH:
+ case OVS_ACTION_ATTR_POP_NSH:
case OVS_ACTION_ATTR_UNSPEC:
case __OVS_ACTION_ATTR_MAX:
default:
@@ -1199,8 +1199,8 @@ dpif_sflow_read_actions(const struct flow *flow,
break;
case OVS_ACTION_ATTR_SAMPLE:
case OVS_ACTION_ATTR_CLONE:
- case OVS_ACTION_ATTR_ENCAP_NSH:
- case OVS_ACTION_ATTR_DECAP_NSH:
+ case OVS_ACTION_ATTR_PUSH_NSH:
+ case OVS_ACTION_ATTR_POP_NSH:
case OVS_ACTION_ATTR_UNSPEC:
case __OVS_ACTION_ATTR_MAX:
default:
@@ -4392,8 +4392,8 @@ xlate_fixup_actions(struct ofpbuf *b, const struct nlattr *actions,
case OVS_ACTION_ATTR_CT:
case OVS_ACTION_ATTR_PUSH_ETH:
case OVS_ACTION_ATTR_POP_ETH:
- case OVS_ACTION_ATTR_ENCAP_NSH:
- case OVS_ACTION_ATTR_DECAP_NSH:
+ case OVS_ACTION_ATTR_PUSH_NSH:
+ case OVS_ACTION_ATTR_POP_NSH:
case OVS_ACTION_ATTR_METER:
ofpbuf_put(b, a, nl_attr_len_pad(a, left));
break;
@@ -5804,17 +5804,17 @@ rewrite_flow_encap_ethernet(struct xlate_ctx *ctx,
/* For an MD2 NSH header returns a pointer to an ofpbuf with the encoded
* MD2 TLVs provided as encap properties to the encap operation. This
- * will be stored as encap_data in the ctx and copied into the encap_nsh
+ * will be stored as encap_data in the ctx and copied into the push_nsh
* action at the next commit. */
static struct ofpbuf *
-rewrite_flow_encap_nsh(struct xlate_ctx *ctx,
+rewrite_flow_push_nsh(struct xlate_ctx *ctx,
const struct ofpact_encap *encap,
struct flow *flow,
struct flow_wildcards *wc)
{
ovs_be32 packet_type = flow->packet_type;
const char *ptr = (char *) encap->props;
- struct ofpbuf *buf = ofpbuf_new(OVS_ENCAP_NSH_MAX_MD_LEN);
+ struct ofpbuf *buf = ofpbuf_new(OVS_PUSH_NSH_MAX_MD_LEN);
uint8_t md_type = NSH_M_TYPE1;
uint8_t np = 0;
int i;
@@ -5854,7 +5854,7 @@ rewrite_flow_encap_nsh(struct xlate_ctx *ctx,
}
ptr += ROUND_UP(prop_ptr->len, 8);
}
- if (buf->size == 0 || buf->size > OVS_ENCAP_NSH_MAX_MD_LEN) {
+ if (buf->size == 0 || buf->size > OVS_PUSH_NSH_MAX_MD_LEN) {
ofpbuf_delete(buf);
buf = NULL;
}
@@ -5930,7 +5930,7 @@ xlate_generic_encap_action(struct xlate_ctx *ctx,
rewrite_flow_encap_ethernet(ctx, flow, wc);
break;
case PT_NSH:
- encap_data = rewrite_flow_encap_nsh(ctx, encap, flow, wc);
+ encap_data = rewrite_flow_push_nsh(ctx, encap, flow, wc);
break;
default:
/* New packet type was checked during decoding. */
@@ -5973,7 +5973,7 @@ xlate_generic_decap_action(struct xlate_ctx *ctx,
}
return false;
case PT_NSH:
- /* The decap_nsh action is generated at the commit executed as
+ /* The pop_nsh action is generated at the commit executed as
* part of freezing the ctx for recirculation. Here we just set
* the new packet type based on the NSH next protocol field. */
switch (flow->nsh.np) {
@@ -105,7 +105,7 @@ bridge("br0")
Final flow: in_port=1,vlan_tci=0x0000,dl_src=00:00:00:00:00:00,dl_dst=11:22:33:44:55:66,dl_type=0x894f,nsh_flags=0,nsh_mdtype=1,nsh_np=3,nsh_spi=0x1234,nsh_si=255,nsh_c1=0x11223344,nsh_c2=0x0,nsh_c3=0x0,nsh_c4=0x0,nw_proto=0,nw_tos=0,nw_ecn=0,nw_ttl=0
Megaflow: recirc_id=0,eth,ip,in_port=1,dl_dst=66:77:88:99:aa:bb,nw_frag=no
-Datapath actions: encap_nsh(flags=0,mdtype=1,np=3,spi=0x1234,si=255,c1=0x11223344,c2=0x0,c3=0x0,c4=0x0),push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),pop_eth,decap_nsh(),set(eth(dst=11:22:33:44:55:66)),recirc(0x1)
+Datapath actions: push_nsh(flags=0,mdtype=1,np=3,spi=0x1234,si=255,c1=0x11223344,c2=0x0,c3=0x0,c4=0x0),push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),pop_eth,pop_nsh(),set(eth(dst=11:22:33:44:55:66)),recirc(0x1)
])
AT_CHECK([
@@ -121,7 +121,7 @@ bridge("br0")
Final flow: unchanged
Megaflow: recirc_id=0,eth,in_port=4,dl_type=0x894f,nsh_mdtype=1,nsh_np=3,nsh_spi=0x1234,nsh_c1=0x11223344
-Datapath actions: pop_eth,decap_nsh(),recirc(0x2)
+Datapath actions: pop_eth,pop_nsh(),recirc(0x2)
])
# Now send two real ICMP echo request packets in on port p1
@@ -139,7 +139,7 @@ ovs-appctl time/warp 1000
AT_CHECK([
ovs-appctl dpctl/dump-flows dummy@ovs-dummy | strip_used | grep -v ipv6 | sort
], [0], [flow-dump from non-dpdk interfaces:
-recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth(dst=1e:2c:e9:2a:66:9e),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:98, used:0.0s, actions:encap_nsh(flags=0,mdtype=1,np=3,spi=0x1234,si=255,c1=0x11223344,c2=0x0,c3=0x0,c4=0x0),push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),pop_eth,decap_nsh(),set(eth(dst=11:22:33:44:55:66)),recirc(0x3)
+recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth(dst=1e:2c:e9:2a:66:9e),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:98, used:0.0s, actions:push_nsh(flags=0,mdtype=1,np=3,spi=0x1234,si=255,c1=0x11223344,c2=0x0,c3=0x0,c4=0x0),push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),pop_eth,pop_nsh(),set(eth(dst=11:22:33:44:55:66)),recirc(0x3)
recirc_id(0x3),in_port(1),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:98, used:0.0s, actions:2
])
@@ -170,7 +170,7 @@ ovs-appctl time/warp 1000
AT_CHECK([
ovs-appctl dpctl/dump-flows dummy@ovs-dummy | strip_used | grep -v ipv6 | sort
], [0], [flow-dump from non-dpdk interfaces:
-recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:98, used:0.0s, actions:push_vlan(vid=100,pcp=0),encap_nsh(flags=0,mdtype=1,np=3,spi=0x0,si=255,c1=0x0,c2=0x0,c3=0x0,c4=0x0),decap_nsh(),recirc(0x4)
+recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:98, used:0.0s, actions:push_vlan(vid=100,pcp=0),push_nsh(flags=0,mdtype=1,np=3,spi=0x0,si=255,c1=0x0,c2=0x0,c3=0x0,c4=0x0),pop_nsh(),recirc(0x4)
recirc_id(0x4),in_port(1),packet_type(ns=0,id=0),eth_type(0x8100),vlan(vid=100,pcp=0),encap(eth_type(0x0800),ipv4(frag=no)), packets:1, bytes:102, used:0.0s, actions:2
])
@@ -230,7 +230,7 @@ bridge("br0")
Final flow: in_port=1,vlan_tci=0x0000,dl_src=00:00:00:00:00:00,dl_dst=11:22:33:44:55:66,dl_type=0x894f,nsh_flags=0,nsh_mdtype=2,nsh_np=3,nsh_spi=0x1234,nsh_si=255,nw_proto=0,nw_tos=0,nw_ecn=0,nw_ttl=0
Megaflow: recirc_id=0,eth,ip,in_port=1,dl_dst=66:77:88:99:aa:bb,nw_frag=no
-Datapath actions: encap_nsh(flags=0,mdtype=2,np=3,spi=0x1234,si=255,md2=0x10000a041234567820001408fedcba9876543210),push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),pop_eth,decap_nsh(),set(eth(dst=11:22:33:44:55:66)),recirc(0x1)
+Datapath actions: push_nsh(flags=0,mdtype=2,np=3,spi=0x1234,si=255,md2=0x10000a041234567820001408fedcba9876543210),push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),pop_eth,pop_nsh(),set(eth(dst=11:22:33:44:55:66)),recirc(0x1)
])
AT_CHECK([
@@ -246,7 +246,7 @@ bridge("br0")
Final flow: unchanged
Megaflow: recirc_id=0,eth,in_port=4,dl_type=0x894f,nsh_mdtype=2,nsh_np=3,nsh_spi=0x1234
-Datapath actions: pop_eth,decap_nsh(),recirc(0x2)
+Datapath actions: pop_eth,pop_nsh(),recirc(0x2)
])
# Now send two real ICMP echo request packets in on port p1
@@ -264,7 +264,7 @@ ovs-appctl time/warp 1000
AT_CHECK([
ovs-appctl dpctl/dump-flows dummy@ovs-dummy | strip_used | grep -v ipv6 | sort
], [0], [flow-dump from non-dpdk interfaces:
-recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth(dst=1e:2c:e9:2a:66:9e),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:98, used:0.0s, actions:encap_nsh(flags=0,mdtype=2,np=3,spi=0x1234,si=255,md2=0x10000a041234567820001408fedcba9876543210),push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),pop_eth,decap_nsh(),set(eth(dst=11:22:33:44:55:66)),recirc(0x3)
+recirc_id(0),in_port(1),packet_type(ns=0,id=0),eth(dst=1e:2c:e9:2a:66:9e),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:98, used:0.0s, actions:push_nsh(flags=0,mdtype=2,np=3,spi=0x1234,si=255,md2=0x10000a041234567820001408fedcba9876543210),push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),pop_eth,pop_nsh(),set(eth(dst=11:22:33:44:55:66)),recirc(0x3)
recirc_id(0x3),in_port(1),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(frag=no), packets:1, bytes:98, used:0.0s, actions:2
])
@@ -577,8 +577,8 @@ ovs-appctl time/warp 1000
AT_CHECK([
ovs-appctl dpctl/dump-flows dummy@ovs-dummy | strip_used | grep -v ipv6 | sort
], [0], [flow-dump from non-dpdk interfaces:
-recirc_id(0),in_port(4),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(dst=192.168.10.30,frag=no), packets:1, bytes:98, used:0.0s, actions:pop_eth,encap_nsh(flags=0,mdtype=1,np=1,spi=0x3000,si=255,c1=0x0,c2=0x0,c3=0x0,c4=0x0),clone(tnl_push(tnl_port(4789),header(size=50,type=4,eth(dst=aa:55:00:00:00:03,src=aa:55:00:00:00:01,dl_type=0x0800),ipv4(src=10.0.0.1,dst=10.0.0.3,proto=17,tos=0,ttl=64,frag=0x4000),udp(src=0,dst=4789,csum=0x0),vxlan(flags=0xc000004,vni=0x0)),out_port(1)),set(ipv4(src=30.0.0.1,dst=30.0.0.3)),tnl_pop(4789))
-tunnel(tun_id=0x0,src=30.0.0.1,dst=30.0.0.3,flags(-df-csum+key)),recirc_id(0),in_port(4789),packet_type(ns=1,id=0x894f),nsh(np=1,spi=0x3000,si=255), packets:1, bytes:108, used:0.0s, actions:decap_nsh(),recirc(0x1)
+recirc_id(0),in_port(4),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(dst=192.168.10.30,frag=no), packets:1, bytes:98, used:0.0s, actions:pop_eth,push_nsh(flags=0,mdtype=1,np=1,spi=0x3000,si=255,c1=0x0,c2=0x0,c3=0x0,c4=0x0),clone(tnl_push(tnl_port(4789),header(size=50,type=4,eth(dst=aa:55:00:00:00:03,src=aa:55:00:00:00:01,dl_type=0x0800),ipv4(src=10.0.0.1,dst=10.0.0.3,proto=17,tos=0,ttl=64,frag=0x4000),udp(src=0,dst=4789,csum=0x0),vxlan(flags=0xc000004,vni=0x0)),out_port(1)),set(ipv4(src=30.0.0.1,dst=30.0.0.3)),tnl_pop(4789))
+tunnel(tun_id=0x0,src=30.0.0.1,dst=30.0.0.3,flags(-df-csum+key)),recirc_id(0),in_port(4789),packet_type(ns=1,id=0x894f),nsh(np=1,spi=0x3000,si=255), packets:1, bytes:108, used:0.0s, actions:pop_nsh(),recirc(0x1)
tunnel(tun_id=0x0,src=30.0.0.1,dst=30.0.0.3,flags(-df-csum+key)),recirc_id(0x1),in_port(4789),packet_type(ns=1,id=0x800),ipv4(frag=no), packets:1, bytes:84, used:0.0s, actions:push_eth(src=00:00:00:00:00:00,dst=aa:55:aa:55:00:03),6
])
@@ -631,9 +631,9 @@ ovs-appctl time/warp 1000
AT_CHECK([
ovs-appctl dpctl/dump-flows dummy@ovs-dummy | strip_used | grep -v ipv6 | sort
], [0], [flow-dump from non-dpdk interfaces:
-recirc_id(0),in_port(4),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(dst=192.168.10.20/255.255.255.248,frag=no), packets:1, bytes:98, used:0.0s, actions:pop_eth,encap_nsh(flags=0,mdtype=1,np=1,spi=0x3020,si=255,c1=0x0,c2=0x0,c3=0x0,c4=0x0),clone(tnl_push(tnl_port(4789),header(size=50,type=4,eth(dst=aa:55:00:00:00:02,src=aa:55:00:00:00:01,dl_type=0x0800),ipv4(src=10.0.0.1,dst=10.0.0.2,proto=17,tos=0,ttl=64,frag=0x4000),udp(src=0,dst=4789,csum=0x0),vxlan(flags=0xc000004,vni=0x0)),out_port(1)),set(ipv4(src=20.0.0.1,dst=20.0.0.2)),tnl_pop(4789))
+recirc_id(0),in_port(4),packet_type(ns=0,id=0),eth_type(0x0800),ipv4(dst=192.168.10.20/255.255.255.248,frag=no), packets:1, bytes:98, used:0.0s, actions:pop_eth,push_nsh(flags=0,mdtype=1,np=1,spi=0x3020,si=255,c1=0x0,c2=0x0,c3=0x0,c4=0x0),clone(tnl_push(tnl_port(4789),header(size=50,type=4,eth(dst=aa:55:00:00:00:02,src=aa:55:00:00:00:01,dl_type=0x0800),ipv4(src=10.0.0.1,dst=10.0.0.2,proto=17,tos=0,ttl=64,frag=0x4000),udp(src=0,dst=4789,csum=0x0),vxlan(flags=0xc000004,vni=0x0)),out_port(1)),set(ipv4(src=20.0.0.1,dst=20.0.0.2)),tnl_pop(4789))
tunnel(tun_id=0x0,src=20.0.0.1,dst=20.0.0.2,flags(-df-csum+key)),recirc_id(0),in_port(4789),packet_type(ns=1,id=0x894f),nsh(spi=0x3020,si=255), packets:1, bytes:108, used:0.0s, actions:push_eth(src=00:00:00:00:00:00,dst=11:22:33:44:55:66),set(nsh(spi=0x3020,si=254)),pop_eth,clone(tnl_push(tnl_port(4789),header(size=50,type=4,eth(dst=aa:55:00:00:00:03,src=aa:55:00:00:00:02,dl_type=0x0800),ipv4(src=20.0.0.2,dst=20.0.0.3,proto=17,tos=0,ttl=64,frag=0x4000),udp(src=0,dst=4789,csum=0x0),vxlan(flags=0xc000004,vni=0x0)),out_port(2)),set(ipv4(src=30.0.0.2,dst=30.0.0.3)),tnl_pop(4789))
-tunnel(tun_id=0x0,src=30.0.0.2,dst=30.0.0.3,flags(-df-csum+key)),recirc_id(0),in_port(4789),packet_type(ns=1,id=0x894f),nsh(np=1,spi=0x3020,si=254), packets:1, bytes:108, used:0.0s, actions:decap_nsh(),recirc(0x2)
+tunnel(tun_id=0x0,src=30.0.0.2,dst=30.0.0.3,flags(-df-csum+key)),recirc_id(0),in_port(4789),packet_type(ns=1,id=0x894f),nsh(np=1,spi=0x3020,si=254), packets:1, bytes:108, used:0.0s, actions:pop_nsh(),recirc(0x2)
tunnel(tun_id=0x0,src=30.0.0.2,dst=30.0.0.3,flags(-df-csum+key)),recirc_id(0x2),in_port(4789),packet_type(ns=1,id=0x800),ipv4(frag=no), packets:1, bytes:84, used:0.0s, actions:push_eth(src=00:00:00:00:00:00,dst=aa:55:aa:55:00:03),6
])
Per kernel data path naming convention, we need to rename encap_nsh and decap_nsh as push_nsh and pop_nsh, respectively. This change will make sure our data path actions follow the same convention in both userspace and kernel data path. The variables, structs, function names are also changed to apply new names. Signed-off-by: Yi Yang <yi.y.yang@intel.com> --- datapath/linux/compat/include/linux/openvswitch.h | 14 +- lib/dpif-netdev.c | 4 +- lib/dpif.c | 4 +- lib/odp-execute.c | 14 +- lib/odp-util.c | 162 +++++++++++----------- lib/packets.c | 20 +-- lib/packets.h | 6 +- ofproto/ofproto-dpif-ipfix.c | 4 +- ofproto/ofproto-dpif-sflow.c | 4 +- ofproto/ofproto-dpif-xlate.c | 16 +-- tests/nsh.at | 22 +-- 11 files changed, 135 insertions(+), 135 deletions(-)