Re: [dpdk-dev] app/testpmd: parse flow command line for AH

[Ye Xiaolong <xiaolong.ye@intel.com>]

Hi, Xiao

On 02/13, Xiao Zhang wrote:
>add ITEM_AH
>add ITEM_AH_SPI

Better to add some descriptive sentence about this patch in commit log instead
of some simple bullets.

Thanks,
Xiaolong

>
>Signed-off-by: Xiao Zhang <xiao.zhang@intel.com>
>---
> app/test-pmd/cmdline_flow.c | 29 +++++++++++++++++++++++++++++
> 1 file changed, 29 insertions(+)
>
>diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
>index c2cc4c5..680cb1c 100644
>--- a/app/test-pmd/cmdline_flow.c
>+++ b/app/test-pmd/cmdline_flow.c
>@@ -220,6 +220,8 @@ enum index {
> 	ITEM_L2TPV3OIP_SESSION_ID,
> 	ITEM_ESP,
> 	ITEM_ESP_SPI,
>+	ITEM_AH,
>+	ITEM_AH_SPI,
> 
> 	/* Validate/create actions. */
> 	ACTIONS,
>@@ -768,6 +770,7 @@ static const enum index next_item[] = {
> 	ITEM_TAG,
> 	ITEM_L2TPV3OIP,
> 	ITEM_ESP,
>+	ITEM_AH,
> 	END_SET,
> 	ZERO,
> };
>@@ -1046,6 +1049,12 @@ static const enum index item_esp[] = {
> 	ZERO,
> };
> 
>+static const enum index item_ah[] = {
>+	ITEM_AH_SPI,
>+	ITEM_NEXT,
>+	ZERO,
>+};
>+
> static const enum index next_set_raw[] = {
> 	SET_RAW_INDEX,
> 	ITEM_ETH,
>@@ -2699,6 +2708,19 @@ static const struct token token_list[] = {
> 		.args = ARGS(ARGS_ENTRY_HTON(struct rte_flow_item_esp,
> 				hdr.spi)),
> 	},
>+	[ITEM_AH] = {
>+		.name = "ah",
>+		.help = "match AH header",
>+		.priv = PRIV_ITEM(AH, sizeof(struct rte_flow_item_ah)),
>+		.next = NEXT(item_ah),
>+		.call = parse_vc,
>+	},
>+	[ITEM_AH_SPI] = {
>+		.name = "spi",
>+		.help = "security parameters index",
>+		.next = NEXT(item_ah, NEXT_ENTRY(UNSIGNED), item_param),
>+		.args = ARGS(ARGS_ENTRY_HTON(struct rte_flow_item_ah, spi)),
>+	},
> 	/* Validate/create actions. */
> 	[ACTIONS] = {
> 		.name = "actions",
>@@ -6444,6 +6466,9 @@ flow_item_default_mask(const struct rte_flow_item *item)
> 	case RTE_FLOW_ITEM_TYPE_ESP:
> 		mask = &rte_flow_item_esp_mask;
> 		break;
>+	case RTE_FLOW_ITEM_TYPE_AH:
>+		mask = &rte_flow_item_ah_mask;
>+		break;
> 	default:
> 		break;
> 	}
>@@ -6541,6 +6566,10 @@ cmd_set_raw_parsed(const struct buffer *in)
> 			size = sizeof(struct rte_flow_item_esp);
> 			proto = 0x32;
> 			break;
>+		case RTE_FLOW_ITEM_TYPE_AH:
>+			size = sizeof(struct rte_flow_item_ah);
>+			proto = 0x33;
>+			break;
> 		default:
> 			printf("Error - Not supported item\n");
> 			*total_size = 0;
>-- 
>2.7.4
>