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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
