On Tue, Feb 06, 2018 at 02:20:28PM +0000, Xueming(Steven) Li wrote: > Hi Nelio, > > > -----Original Message----- > > From: Nélio Laranjeiro [mailto:nelio.laranje...@6wind.com] > > Sent: Tuesday, February 6, 2018 9:22 PM > > To: Xueming(Steven) Li <xuemi...@mellanox.com> > > Cc: Nelio Laranjeiro <notificati...@github.com>; dev@dpdk.org; Shahaf > > Shuler <shah...@mellanox.com> > > Subject: Re: [dpdk-dev] [PATCH] net/mlx5: dump flow create error message > > > > On Tue, Feb 06, 2018 at 08:52:53PM +0800, Xueming Li wrote: > > > Add error message dump when flow create error happened. > > > > > > Signed-off-by: Xueming Li <xuemi...@mellanox.com> > > > --- > > > drivers/net/mlx5/mlx5_flow.c | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/drivers/net/mlx5/mlx5_flow.c > > > b/drivers/net/mlx5/mlx5_flow.c index e38452ef3..ea90556d0 100644 > > > --- a/drivers/net/mlx5/mlx5_flow.c > > > +++ b/drivers/net/mlx5/mlx5_flow.c > > > @@ -1890,6 +1890,7 @@ priv_flow_create(struct priv *priv, > > > DEBUG("Flow created %p", (void *)flow); > > > return flow; > > > exit: > > > + ERROR("Flow create error: %s", error->message); > > > for (i = 0; i != hash_rxq_init_n; ++i) { > > > if (parser.queue[i].ibv_attr) > > > rte_free(parser.queue[i].ibv_attr); > > > -- > > > 2.13.3 > > > > Hi Xueming, > > > > Can you move this debug message to mlx5_flow_create(). > > This function also called by control flow and fdir functions, > here should be the best place.
Make sense, Acked-by: Nelio Laranjeiro <nelio.laranje...@6wind.com> Thanks, -- Nélio Laranjeiro 6WIND