On Tue, May 15, 2018 at 05:09:45PM +0300, Leon Romanovsky wrote:
> From: Raed Salem <raeds@xxxxxxxxxxxx>
>
> A counters object could be attached to flow on creation
> by providing the counter specification action.
>
> General counters description which count packets and bytes are
> introduced, downstream patches from this series will use them
> as part of flow counters binding.
>
> In addition, increase number of flow specifications supported
> layers to 10 upon adding count specification and for the
> previously added drop specification.
>
> Reviewed-by: Yishai Hadas <yishaih@xxxxxxxxxxxx>
> Signed-off-by: Raed Salem <raeds@xxxxxxxxxxxx>
> Signed-off-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>
> include/rdma/ib_verbs.h | 15 ++++++++++++++-
> 1 file changed, 14 insertions(+), 1 deletion(-)
>
> diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h
> index 869e3cdee7dc..cb00795a86de 100644
> +++ b/include/rdma/ib_verbs.h
> @@ -1861,9 +1861,10 @@ enum ib_flow_spec_type {
> IB_FLOW_SPEC_ACTION_TAG = 0x1000,
> IB_FLOW_SPEC_ACTION_DROP = 0x1001,
> IB_FLOW_SPEC_ACTION_HANDLE = 0x1002,
> + IB_FLOW_SPEC_ACTION_COUNT = 0x1003,
> };
This entire enum is uapi and doesn't belong here.. Please send a
follow up patch to move it
> #define IB_FLOW_SPEC_LAYER_MASK 0xF0
> -#define IB_FLOW_SPEC_SUPPORT_LAYERS 8
> +#define IB_FLOW_SPEC_SUPPORT_LAYERS 10
>
> /* Flow steering rule priority is set according to it's domain.
> * Lower domain value means higher priority.
> @@ -2043,6 +2044,17 @@ struct ib_flow_spec_action_handle {
> struct ib_flow_action *act;
> };
>
> +enum ib_counters_description {
> + IB_COUNTER_PACKETS,
> + IB_COUNTER_BYTES,
> +};
These constants & enum are never used
Jason
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
