On 1/14/2020 1:55 PM, Bernard Iremonger wrote:
> add ITEM_ESP
> add ITEM_ESP_SPI
>
> Signed-off-by: Bernard Iremonger <bernard.iremon...@intel.com>
> Acked-by: Ori Kam <or...@mellanox.com>
> ---
> app/test-pmd/cmdline_flow.c | 4 +---
> app/test-pmd/config.c | 2 +-
> 2 files changed, 2 insertions(+), 4 deletions(-)
>
> diff --git a/app/test-pmd/cmdline_flow.c b/app/test-pmd/cmdline_flow.c
> index 9643148..9c6edb8 100644
> --- a/app/test-pmd/cmdline_flow.c
> +++ b/app/test-pmd/cmdline_flow.c
> @@ -6075,9 +6075,6 @@ cmd_flow_tok(cmdline_parse_token_hdr_t **hdr,
> static void
> cmd_flow_parsed(const struct buffer *in)
> {
> - printf("Flow command line parsed successfully for command=%d.\n",
> - in->command);
> -
Hi Bernard,
Is this patchset depends another, or can there be something went wrong here, I
can't find these lines removed also it is not really clear what this patchset
does.
> switch (in->command) {
> case VALIDATE:
> port_flow_validate(in->port, &in->args.vc.attr,
> @@ -6261,6 +6258,7 @@ flow_item_default_mask(const struct rte_flow_item *item)
> break;
> case RTE_FLOW_ITEM_TYPE_PPPOE_PROTO_ID:
> mask = &rte_flow_item_pppoe_proto_id_mask;
> + break;
> case RTE_FLOW_ITEM_TYPE_ESP:
> mask = &rte_flow_item_esp_mask;
> break;
> diff --git a/app/test-pmd/config.c b/app/test-pmd/config.c
> index 2753ec5..d599682 100644
> --- a/app/test-pmd/config.c
> +++ b/app/test-pmd/config.c
> @@ -1303,7 +1303,7 @@ port_flow_complain(struct rte_flow_error *error)
> errstr = "unknown type";
> else
> errstr = errstrlist[error->type];
> - printf("%s(): Caught PMD error type %d (%s): %s%s: %s\n", __func__,
> + printf("Caught error type %d (%s): %s%s: %s\n",
> error->type, errstr,
> error->cause ? (snprintf(buf, sizeof(buf), "cause: %p, ",
> error->cause), buf) : "",
>
