From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129]) by dpdk.org (Postfix) with ESMTP id 46498A3 for ; Mon, 25 Mar 2019 18:03:46 +0100 (CET) Received: from Internal Mail-Server by MTLPINE1 (envelope-from viacheslavo@mellanox.com) with ESMTPS (AES256-SHA encrypted); 25 Mar 2019 19:03:41 +0200 Received: from pegasus12.mtr.labs.mlnx. (pegasus12.mtr.labs.mlnx [10.210.17.40]) by labmailer.mlnx (8.13.8/8.13.8) with ESMTP id x2PH3fbW020888; Mon, 25 Mar 2019 19:03:41 +0200 From: Viacheslav Ovsiienko To: dev@dpdk.org Cc: shahafs@mellanox.com Date: Mon, 25 Mar 2019 17:03:21 +0000 Message-Id: <1553533414-9911-1-git-send-email-viacheslavo@mellanox.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1553155888-27498-1-git-send-email-viacheslavo@mellanox.com> References: <1553155888-27498-1-git-send-email-viacheslavo@mellanox.com> Subject: [dpdk-dev] [PATCH v2 00/14] net/mlx5: add support for multiport IB devices X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 25 Mar 2019 17:03:46 -0000 The Mellanox NICs support SR-IOV and have E-Switch feature. When SR-IOV is set up in switchdev mode and E-Switch is enabled we have so called VF representors in the system. All representors belonging to the same E-Switch are created on the basis of the single PCI function and with current implementation each representor has its own dedicated Infiniband device and operates within its own Infiniband context. It is proposed to provide representors as ports of the single Infiniband device and operate on the shared Infiniband context. mlx5_pci_probe() routine is planned to be extended to probe the ports of found Infiniband devices. All active ports, belonging to the same Infiniband device will use the signle shared Infiniband context of that device: - QP are created within shared context, with assigning to specified port - Verb flows are also created with specifying port index - Protection Domain is shared by all representors - Global Memory Regions Cache is shared by all representors, local ones are not (not in this patchset) - Shared Asynchronous Event Handler distributes events among the device ports (not in this patchset) - UAR mapping to user space is also shared (not in patchset) Each representor has its own attach network interface device, interface name and index can be retrieved via IB device and port index, so the link control is performed via Netlink in the same way as in existing implementation. This patchset also superseedes http://patches.dpdk.org/patch/50486/ Signed-off-by: Viacheslav Ovsiienko --- v2: - clarifying comments added - asserts added - secondary process does not probe IB device at all and does not create shared context - minor bugs (like ++ in assert) fixed - representor_id contains corresponding VF vport index, no corrections +1 is needed anymore. vport index is calculated in the only place in code and if indexing schema is changed it would be easy to modify v1: http://patches.dpdk.org/cover/51451/ Viacheslav Ovsiienko (13): net/mlx5: add representor recognition on kernels 5.x net/mlx5: modify get ifindex routine for multiport IB net/mlx5: add getting IB ports number for multiport IB net/mlx5: add multiport IB device support to probing net/mlx5: add IB shared context alloc/free functions net/mlx5: switch to the names in the shared IB context net/mlx5: switch to the shared Protection Domain net/mlx5: switch to the shared context IB attributes net/mlx5: switch to the shared IB device context net/mlx5: provide IB port for the object being created net/mlx5: update install/uninstall int handler routines net/mlx5: update event handler for multiport IB devices net/mlx5: add source vport match to the ingress rules drivers/net/mlx5/Makefile | 10 + drivers/net/mlx5/meson.build | 4 + drivers/net/mlx5/mlx5.c | 644 ++++++++++++++++++++++++++----------- drivers/net/mlx5/mlx5.h | 44 ++- drivers/net/mlx5/mlx5_ethdev.c | 242 +++++++++----- drivers/net/mlx5/mlx5_flow.c | 2 + drivers/net/mlx5/mlx5_flow_dv.c | 45 ++- drivers/net/mlx5/mlx5_flow_verbs.c | 7 +- drivers/net/mlx5/mlx5_mr.c | 4 +- drivers/net/mlx5/mlx5_nl.c | 102 +++++- drivers/net/mlx5/mlx5_rxq.c | 41 +-- drivers/net/mlx5/mlx5_socket.c | 4 +- drivers/net/mlx5/mlx5_stats.c | 22 +- drivers/net/mlx5/mlx5_trigger.c | 2 +- drivers/net/mlx5/mlx5_txq.c | 24 +- 15 files changed, 859 insertions(+), 338 deletions(-) -- 1.8.3.1 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by dpdk.space (Postfix) with ESMTP id 6F871A05D3 for ; Mon, 25 Mar 2019 18:04:08 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 7684A3772; Mon, 25 Mar 2019 18:03:54 +0100 (CET) Received: from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129]) by dpdk.org (Postfix) with ESMTP id 46498A3 for ; Mon, 25 Mar 2019 18:03:46 +0100 (CET) Received: from Internal Mail-Server by MTLPINE1 (envelope-from viacheslavo@mellanox.com) with ESMTPS (AES256-SHA encrypted); 25 Mar 2019 19:03:41 +0200 Received: from pegasus12.mtr.labs.mlnx. (pegasus12.mtr.labs.mlnx [10.210.17.40]) by labmailer.mlnx (8.13.8/8.13.8) with ESMTP id x2PH3fbW020888; Mon, 25 Mar 2019 19:03:41 +0200 From: Viacheslav Ovsiienko To: dev@dpdk.org Cc: shahafs@mellanox.com Date: Mon, 25 Mar 2019 17:03:21 +0000 Message-Id: <1553533414-9911-1-git-send-email-viacheslavo@mellanox.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1553155888-27498-1-git-send-email-viacheslavo@mellanox.com> References: <1553155888-27498-1-git-send-email-viacheslavo@mellanox.com> Subject: [dpdk-dev] [PATCH v2 00/14] net/mlx5: add support for multiport IB devices X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Content-Type: text/plain; charset="UTF-8" Message-ID: <20190325170321.wM8dXdAxYwI3S_If2jxgx18PeLKbZWHlKqGMlwx0VqU@z> The Mellanox NICs support SR-IOV and have E-Switch feature. When SR-IOV is set up in switchdev mode and E-Switch is enabled we have so called VF representors in the system. All representors belonging to the same E-Switch are created on the basis of the single PCI function and with current implementation each representor has its own dedicated Infiniband device and operates within its own Infiniband context. It is proposed to provide representors as ports of the single Infiniband device and operate on the shared Infiniband context. mlx5_pci_probe() routine is planned to be extended to probe the ports of found Infiniband devices. All active ports, belonging to the same Infiniband device will use the signle shared Infiniband context of that device: - QP are created within shared context, with assigning to specified port - Verb flows are also created with specifying port index - Protection Domain is shared by all representors - Global Memory Regions Cache is shared by all representors, local ones are not (not in this patchset) - Shared Asynchronous Event Handler distributes events among the device ports (not in this patchset) - UAR mapping to user space is also shared (not in patchset) Each representor has its own attach network interface device, interface name and index can be retrieved via IB device and port index, so the link control is performed via Netlink in the same way as in existing implementation. This patchset also superseedes http://patches.dpdk.org/patch/50486/ Signed-off-by: Viacheslav Ovsiienko --- v2: - clarifying comments added - asserts added - secondary process does not probe IB device at all and does not create shared context - minor bugs (like ++ in assert) fixed - representor_id contains corresponding VF vport index, no corrections +1 is needed anymore. vport index is calculated in the only place in code and if indexing schema is changed it would be easy to modify v1: http://patches.dpdk.org/cover/51451/ Viacheslav Ovsiienko (13): net/mlx5: add representor recognition on kernels 5.x net/mlx5: modify get ifindex routine for multiport IB net/mlx5: add getting IB ports number for multiport IB net/mlx5: add multiport IB device support to probing net/mlx5: add IB shared context alloc/free functions net/mlx5: switch to the names in the shared IB context net/mlx5: switch to the shared Protection Domain net/mlx5: switch to the shared context IB attributes net/mlx5: switch to the shared IB device context net/mlx5: provide IB port for the object being created net/mlx5: update install/uninstall int handler routines net/mlx5: update event handler for multiport IB devices net/mlx5: add source vport match to the ingress rules drivers/net/mlx5/Makefile | 10 + drivers/net/mlx5/meson.build | 4 + drivers/net/mlx5/mlx5.c | 644 ++++++++++++++++++++++++++----------- drivers/net/mlx5/mlx5.h | 44 ++- drivers/net/mlx5/mlx5_ethdev.c | 242 +++++++++----- drivers/net/mlx5/mlx5_flow.c | 2 + drivers/net/mlx5/mlx5_flow_dv.c | 45 ++- drivers/net/mlx5/mlx5_flow_verbs.c | 7 +- drivers/net/mlx5/mlx5_mr.c | 4 +- drivers/net/mlx5/mlx5_nl.c | 102 +++++- drivers/net/mlx5/mlx5_rxq.c | 41 +-- drivers/net/mlx5/mlx5_socket.c | 4 +- drivers/net/mlx5/mlx5_stats.c | 22 +- drivers/net/mlx5/mlx5_trigger.c | 2 +- drivers/net/mlx5/mlx5_txq.c | 24 +- 15 files changed, 859 insertions(+), 338 deletions(-) -- 1.8.3.1