Hi Ferruh,
> -----Original Message----- > From: Yigit, Ferruh <[email protected]> > Sent: Tuesday, January 14, 2020 7:04 PM > To: Iremonger, Bernard <[email protected]>; [email protected]; > Xing, Beilei <[email protected]>; Zhang, Qi Z <[email protected]>; > Doherty, Declan <[email protected]> > Cc: Ananyev, Konstantin <[email protected]>; Byrne, Stephen1 > <[email protected]>; Zhang, Helin <[email protected]> > 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 <[email protected]> > > Acked-by: Ori Kam <[email protected]> > > --- > > 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
