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

["Zhang, Xiao" <xiao.zhang@intel.com>]

> -----Original Message-----
> From: Ye, Xiaolong <xiaolong.ye@intel.com>
> Sent: Thursday, February 13, 2020 2:46 PM
> To: Zhang, Xiao <xiao.zhang@intel.com>
> Cc: dev@dpdk.org; orika@mellanox.com; Zhang, Qi Z <qi.z.zhang@intel.com>;
> Su, Simei <simei.su@intel.com>; Iremonger, Bernard
> <bernard.iremonger@intel.com>
> Subject: Re: [dpdk-dev] app/testpmd: parse flow command line for AH
> 
> 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.

Yes, added more description in v2 patch.

Thanks,
Xiao

> 
> 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
> >