Hi Ferruh,
> -----Original Message-----
> From: Yigit, Ferruh <ferruh.yi...@intel.com>
> Sent: Tuesday, January 14, 2020 7:04 PM
> To: Iremonger, Bernard <bernard.iremon...@intel.com>; dev@dpdk.org;
> Xing, Beilei <beilei.x...@intel.com>; Zhang, Qi Z <qi.z.zh...@intel.com>;
> Doherty, Declan <declan.dohe...@intel.com>
> Cc: Ananyev, Konstantin <konstantin.anan...@intel.com>; Byrne, Stephen1
> <stephen1.by...@intel.com>; Zhang, Helin <helin.zh...@intel.com>
> Subject: Re: [dpdk-dev] [PATCH v5 01/14] app/testpmd: parse flow
> command line for ESP
>
> 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.
Something has gone wrong here.
I will investigate.
>
> > 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) : "",
> >
Regards,
Bernard
