On Thu, Oct 04, 2018 at 01:17:26PM +0300, Yishai Hadas wrote:
> From: Mark Bloch <markb@xxxxxxxxxxxx>
>
> Use flags defined under enum ibv_flow_flags, this will allow creating
> egress matcher.
>
> Signed-off-by: Mark Bloch <markb@xxxxxxxxxxxx>
> Signed-off-by: Yishai Hadas <yishaih@xxxxxxxxxxxx>
> providers/mlx5/verbs.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/providers/mlx5/verbs.c b/providers/mlx5/verbs.c
> index de731ce..41060bc 100644
> +++ b/providers/mlx5/verbs.c
> @@ -3665,7 +3665,7 @@ mlx5dv_create_flow_matcher(struct ibv_context *context,
> {
> DECLARE_COMMAND_BUFFER(cmd, MLX5_IB_OBJECT_FLOW_MATCHER,
> MLX5_IB_METHOD_FLOW_MATCHER_CREATE,
> - 4);
> + 5);
> struct mlx5dv_flow_matcher *flow_matcher;
> struct ib_uverbs_attr *handle;
> int ret;
> @@ -3695,6 +3695,9 @@ mlx5dv_create_flow_matcher(struct ibv_context *context,
> fill_attr_in_enum(cmd, MLX5_IB_ATTR_FLOW_MATCHER_FLOW_TYPE,
> IBV_FLOW_ATTR_NORMAL, &attr->priority,
> sizeof(attr->priority));
> + if (attr->flags)
> + fill_attr_const_in(cmd, MLX5_IB_ATTR_FLOW_MATCHER_FLOW_FLAGS,
> + attr->flags);
Uhh..
This is 'enum ib_flow_flags' in the kernel, and it is not in a uapi
header. This was missed when the kernel patches were accepted.
Please send a kernel patch to fix it..
Jason
