Sunday, May 13, 2018 4:28 PM, Xueming(Steven) Li:
> Thanks for fixing this, one minor issue below.
>
> > -----Original Message-----
> > From: Shahaf Shuler <shah...@mellanox.com>
> > Sent: Sunday, May 13, 2018 4:08 PM
> > To: Nélio Laranjeiro <nelio.laranje...@6wind.com>; Adrien Mazarguil
> > <adrien.mazarg...@6wind.com>; Yongseok Koh <ys...@mellanox.com>
> > Cc: dev@dpdk.org; Xueming(Steven) Li <xuemi...@mellanox.com>
> > Subject: [PATCH] net/mlx5: fix compilation without tunnel RSS support
> >
> > IBV_RX_HASH_INNER should be referenced only when having tunnel
> support in the Verbs headers.
> >
> > Fixes: 80f2d0ed7ff9 ("net/mlx5: add hardware flow debug dump")
> > Cc: xuemi...@mellanox.com
> >
> > Signed-off-by: Shahaf Shuler <shah...@mellanox.com>
> > ---
> > drivers/net/mlx5/mlx5_flow.c | 9 ++++++---
> > 1 file changed, 6 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/net/mlx5/mlx5_flow.c
> > b/drivers/net/mlx5/mlx5_flow.c index ec6d00f21b..04ace3ee79
> > 100644
> > --- a/drivers/net/mlx5/mlx5_flow.c
> > +++ b/drivers/net/mlx5/mlx5_flow.c
> > @@ -2188,7 +2188,12 @@ mlx5_flow_dump(struct rte_eth_dev *dev
> __rte_unused,
> > uint16_t j;
> > char buf[256];
> > uint8_t off;
> > + uint64_t extra_hash_fields = 0;
> >
> > +#ifdef HAVE_IBV_DEVICE_TUNNEL_SUPPORT
> > + if (flow->tunnel && flow->rss_conf.level > 1)
> > + extra_hash_fields = (uint32_t)IBV_RX_HASH_INNER; #endif
>
> Should "#endif" start with a new line?
It is in new line. You probably see it wrongly in outlook.
Have a look on the patch in patchwork:
http://dpdk.org/dev/patchwork/patch/39927/
