[RFC] New verbs API for flow action: ENCAP

Message ID	CAFgAxU81GmtWkHpeF4CTPq2KDezVRmc7kkrPdAS5H8Higxg=jQ@mail.gmail.com (mailing list archive)
State	RFC
Headers	show Return-Path: <linux-rdma-owner@kernel.org> MIME-Version: 1.0 From: Alex Rosenbaum <rosenbaumalex@gmail.com> Date: Wed, 4 Apr 2018 12:15:53 +0300 Message-ID: <CAFgAxU81GmtWkHpeF4CTPq2KDezVRmc7kkrPdAS5H8Higxg=jQ@mail.gmail.com> Subject: [PATCH RFC] New verbs API for flow action: ENCAP To: linux-rdma@vger.kernel.org Cc: Yishai Hadas <yishaih@mellanox.com>, Leon Romanovsky <leonro@mellanox.com>, "Alex @ Mellanox" <alexr@mellanox.com>, Mark Bloch <markb@mellanox.com> Content-Type: text/plain; charset="UTF-8" Sender: linux-rdma-owner@vger.kernel.org Precedence: bulk

diff --git a/libibverbs/examples/encap_flow.c b/libibverbs/examples/encap_flow.c new file mode 100644 index 0000000..1e8e547 --- /dev/null +++ b/libibverbs/examples/encap_flow.c @@ -0,0 +1,152 @@ +/* + * Copyright (c) 2018 Mellanox Technologies, Ltd. All rights reserved. + * + * This software is available to you under a choice of one of two + * licenses. You may choose to be licensed under the terms of the GNU + * General Public License (GPL) Version 2, available from the file + * COPYING in the main directory of this source tree, or the + * OpenIB.org BSD license below: + * + * Redistribution and use in source and binary forms, with or + * without modification, are permitted provided that the following + * conditions are met: + * + * - Redistributions of source code must retain the above + * copyright notice, this list of conditions and the following + * disclaimer. + * + * - Redistributions in binary form must reproduce the above + * copyright notice, this list of conditions and the following + * disclaimer in the documentation and/or other materials + * provided with the distribution. + * + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, + * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF + * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND + * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS + * BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN + * ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN + * CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE + * SOFTWARE. + */ + +#include <config.h> + +#include <stdio.h> +#include <stdlib.h> + +#include <netinet/if_ether.h> //Provides declarations for ethernet header +#include <netinet/udp.h> //Provides declarations for udp header +#include <netinet/ip.h> //Provides declarations for ip header + +#include <infiniband/verbs.h> + +struct vxlanhdr { + __be32 flags; + __be32 vni; +}; + +static uint8_t buf[sizeof(struct ethhdr) + + sizeof(struct iphdr) + + sizeof(struct udphdr) + + sizeof(struct vxlanhdr)] = {}; + +struct ethhdr eth = {.h_source = {0x52, 0x54, 0x00, 0xf7, 0x56, 0xe2}, + .h_dest = {0x52, 0x54, 0x00, 0xf7, 0x56, 0xe2}, + .h_proto = 0x0800}; + +struct iphdr ip = {.version = 4, + .ihl = 5, + .tos = 0, + .tot_len = 0, + .id = 0x73a7, + .frag_off = 0x02, + .ttl = 64, + .protocol = 17, + .check = 0, + .saddr = (0x1d << 24) | (0x00 << 16) | (0x0a << 8) | 0x64, + .daddr = (0x1a << 24) | (0x00 << 16) | (0x0a << 8) | 0x78}; + +struct udphdr udp = {.source = 0, + .dest = 4789, + .len = 0, + .check = 0}; + +struct vxlanhdr vxlan = {.flags = 0x0800, + .vni = 0x0028}; + +static int create_encap_hdr(struct ibv_device *ib_dev) +{ + struct ibv_flow_action_encap_attr attr; + struct ibv_flow_action *action; + struct ethhdr *tmp_eth = (struct ethhdr *)buf; + struct iphdr *tmp_ip = (struct iphdr *)(tmp_eth + 1); + struct udphdr *tmp_udp = (struct udphdr *)(tmp_ip + 1); + struct vxlanhdr *tmp_vxlan = (struct vxlanhdr *)(tmp_udp + 1); + struct ibv_context *ctx; + int ret = 0; + + ctx = ibv_open_device(ib_dev); + if (!ctx) { + fprintf(stderr, "Failed to open device\n"); + return ret; + } + + memcpy(tmp_eth, &eth, sizeof(*tmp_eth)); + memcpy(tmp_ip, &ip, sizeof(*tmp_ip)); + memcpy(tmp_udp, &udp, sizeof(*tmp_udp)); + memcpy(tmp_vxlan, &vxlan, sizeof(*tmp_vxlan)); + + attr.hdr_ptr = buf; + attr.hdr_len = sizeof(buf); + attr.hdr_proto = IBV_FLOW_ACTION_ENCAP_RAW; + + action = ibv_create_flow_action_encap(ctx, &attr); + + printf("%s- got action %p\n", __func__, action); + if (action) + ibv_destroy_flow_action(action); + + ibv_close_device(ctx); + + return ret; +} + +int main(int argc, char *argv[]) +{ + struct ibv_device **dev_list; + struct ibv_device *ib_dev; + char *ib_devname = NULL; + int ret = 1; + int i; + + dev_list = ibv_get_device_list(NULL); + + if (!dev_list) { + perror("Failed to get IB devices list"); + return 1; + } + + if (argc != 2) { + perror("Didn't provide IB device name"); + goto free_dev_list; + } + + ib_devname = argv[1]; + for (i = 0; dev_list[i]; ++i) + if (!strcmp(ibv_get_device_name(dev_list[i]), ib_devname)) + break; + + ib_dev = dev_list[i]; + if (!ib_dev) { + fprintf(stderr, "IB device %s not found\n", ib_devname); + goto free_dev_list; + } + + ret = create_encap_hdr(ib_dev); + +free_dev_list: + ibv_free_device_list(dev_list); + + return ret; +} diff --git a/libibverbs/man/ibv_create_flow_action_encap.3.md b/libibverbs/man/ibv_create_flow_action_encap.3.md new file mode 100644 index 0000000..5ff18d9 --- /dev/null +++ b/libibverbs/man/ibv_create_flow_action_encap.3.md @@ -0,0 +1,82 @@ +--- +date: 2018-03-27 +footer: libibverbs +header: "Libibverbs Programmer's Manual" +layout: page +license: 'Licensed under the OpenIB.org BSD license (FreeBSD Variant) - See COPYING.md' +section: 3 +title: ibv_flow_action_encap(3) +tagline: Verbs +--- + +# NAME + +ibv_flow_action_encap(3) -- Flow action encap for verbs(3) + +# SYNOPSIS + +```c +#include <infiniband/verbs.h> + +struct ibv_flow_action * +ibv_create_flow_action_encap(struct ibv_context *ctx, + struct ibv_flow_action_encap_attr *attr); + +int ibv_destroy_flow_action(struct ibv_flow_action *action); +``` + +# DESCRIPTION + +An ENCAP flow steering action allows a flow steering rule to encapsulate +a packet after matching. + +After the action is created, it should be associated with a *struct +ibv_flow_attr* using *struct ibv_flow_spec_action_handle* flow specification. +An action can be associated with multiple flows. + +# ARGUMENTS + +*ctx* +: RDMA device context to create the action on. + +*attr* +: ENCAP type and header to be used. + +*action* +: Existing action to be destroyed. + +## *action* Argument + +```c +struct ibv_flow_action_encap_attr { + enum ibv_flow_action_encap hdr_proto; + uint16_t hdr_len; + void *hdr_ptr; +}; +``` + +*hdr_proto* +: Encapsulation protocol to be used. + +*hdr_len* +: Length in bytes of the provided encap headers. + +*hdr_ptr* +: Buffing containing the encap headers. + +## Encapsulate protocols (*ibv_flow_action_encap*) + +A user can create flow actions that implement different encapsulation +protocols. + +*IBV_FLOW_ACTION_ENCAP_RAW* provides the ability to provide unspecified +encapsulation protocol, this can be expressed by passing it in *hdr_proto*. + +# RETURN VALUE + +Upon success *ibv_create_flow_action_encap* will return a new *struct +ibv_flow_action* object, on error NULL will be returned and errno will be set. + +# SEE ALSO + +*ibv_create_flow(3)*, *ibv_destroy_flow_action(3)* diff --git a/libibverbs/verbs.h b/libibverbs/verbs.h index eb57824..2f5fc8f 100644 --- a/libibverbs/verbs.h +++ b/libibverbs/verbs.h @@ -44,6 +44,7 @@ #include <linux/types.h> #include <stdint.h> #include <infiniband/verbs_api.h> +#include <stdio.h> #ifdef __cplusplus #include <limits> @@ -1583,6 +1584,12 @@ struct ibv_flow_action_esp_attr { uint32_t esn; }; +struct ibv_flow_action_encap_attr { + enum ibv_flow_action_encap hdr_proto; + uint16_t hdr_len; + void *hdr_ptr; +}; + struct ibv_device; struct ibv_context; @@ -1737,6 +1737,8 @@ struct ibv_values_ex { struct verbs_context { /* "grows up" - new fields go here */ + struct ibv_flow_action *(*create_flow_action_encap)(struct ibv_context *context, + struct ibv_flow_action_encap_attr *attr); struct ibv_mr *(*reg_dm_mr)(struct ibv_pd *pd, struct ibv_dm *dm,

[RFC] New verbs API for flow action: ENCAP

Commit Message

Comments

Patch