Hi Roland, V6 changes: - Addressed comments from Roland on patch #1 and #3 - Don't allow unsupported comp_mask values - Added a check in ib_uverbs_create_flow to verify the size passed from the user space. - Cosmetic changes: don't use anonymous union as the only field in a struct.
V5 changes: - Addressed comments from Roland on patch #2 -- enhanced the change-log V4 changes: - Addressed comments from Sean on the uverbs code that copies the flow specifications from user space. Now we make sure that passed sizes match the kernel flow specs, and in any case, don't look on addresses which are beyond the overall size passed by the write system call. V3 changes: - Addressed comments from Sean: - modified the change-log of patch #1 to be clearer on the priority and domain semantics and usage - re-arranged the fields of struct ib_flow_attr - removed check from ib_flow_destroy - removed the IB flow spec which wasn't inline with the L2/L3/L4 approach done for Ethernet/IP/TCP|UDP, will use proper IB flow specs when adding the support for IPoIB flow steering V2 changes: - dropped struct ib_kern_flow from patch #3, this structure wasn't used and was left there by mistake (bug, thanks Roland) - removed the void *flow_context field from struct ib_flow, this was pointing to driver private data for that flow, but doesn't belong here, i.e need not be seen by the verbs consumer but rather hidden. - renamed struct mlx4_flow_handle to mlx4_ib_flow, a structure that contains the verbs level struct ib_flow and the mlx4 registeration ID for that flow V1 changes: - dropped the five pre-patches which were accepted into 3.10 - rebased the patches against Roland's for-next / 3.10-rc4 - in patch #3, ib_uverbs_destroy_flow was returning too quickly when the driver returned failure for ib_destroy_flow, need to free some uverbs resources 1st. - in patch #4, check index before accessing the array at mlx4_ib_create/destroy_flow These patches add Flow Steering support to the kernel IB core, to uverbs and to the mlx4 IB (verbs) driver along with one patch to uverbs which adds some code to support extensions. IB/core: Add receive Flow Steering support IB/core: Infra-structure to support verbs extensions through uverbs IB/core: Export ib_create/destroy_flow through uverbs IB/mlx4: Add receive Flow Steering support The main patch which introduces the Flow-Steering API is "IB/core: Add receive Flow Steering support", see its change log. Looking on the "Network Adapter Flow Steering" slides from Tzahi Oved which he presented on the annual OFA 2012 meeting could be helpful https://www.openfabrics.org/resources/document-downloads/presentations/doc_download/518-network-adapter-flow-steering.html Or and Matan. Hadar Hen Zion (3): IB/core: Add receive Flow Steering support IB/core: Export ib_create/destroy_flow through uverbs IB/mlx4: Add receive Flow Steering support Igor Ivanov (1): IB/core: Infrastructure to support verbs extensions through uverbs drivers/infiniband/core/uverbs.h | 3 + drivers/infiniband/core/uverbs_cmd.c | 228 ++++++++++++++++++++++++++++++++ drivers/infiniband/core/uverbs_main.c | 42 +++++- drivers/infiniband/core/verbs.c | 27 ++++ drivers/infiniband/hw/mlx4/main.c | 235 +++++++++++++++++++++++++++++++++ drivers/infiniband/hw/mlx4/mlx4_ib.h | 12 ++ include/linux/mlx4/device.h | 5 - include/rdma/ib_verbs.h | 122 +++++++++++++++++- include/uapi/rdma/ib_user_verbs.h | 99 ++++++++++++++- 9 files changed, 759 insertions(+), 14 deletions(-) -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html