Cover Letter Detail
Show a cover letter.
GET /api/covers/1528338/?format=api
{ "id": 1528338, "url": "http://patchwork.ozlabs.org/api/covers/1528338/?format=api", "web_url": "http://patchwork.ozlabs.org/project/openvswitch/cover/20210915124319.1683219-1-cmi@nvidia.com/", "project": { "id": 47, "url": "http://patchwork.ozlabs.org/api/projects/47/?format=api", "name": "Open vSwitch", "link_name": "openvswitch", "list_id": "ovs-dev.openvswitch.org", "list_email": "ovs-dev@openvswitch.org", "web_url": "http://openvswitch.org/", "scm_url": "git@github.com:openvswitch/ovs.git", "webscm_url": "https://github.com/openvswitch/ovs", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20210915124319.1683219-1-cmi@nvidia.com>", "list_archive_url": null, "date": "2021-09-15T12:43:12", "name": "[ovs-dev,v15,0/7] Add offload support for sFlow", "submitter": { "id": 80086, "url": "http://patchwork.ozlabs.org/api/people/80086/?format=api", "name": "Chris Mi", "email": "cmi@nvidia.com" }, "mbox": "http://patchwork.ozlabs.org/project/openvswitch/cover/20210915124319.1683219-1-cmi@nvidia.com/mbox/", "series": [ { "id": 262447, "url": "http://patchwork.ozlabs.org/api/series/262447/?format=api", "web_url": "http://patchwork.ozlabs.org/project/openvswitch/list/?series=262447", "date": "2021-09-15T12:43:12", "name": "Add offload support for sFlow", "version": 15, "mbox": "http://patchwork.ozlabs.org/series/262447/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/1528338/comments/", "headers": { "Return-Path": "<ovs-dev-bounces@openvswitch.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "dev@openvswitch.org" ], "Delivered-To": [ "patchwork-incoming@ozlabs.org", "ovs-dev@lists.linuxfoundation.org" ], "Authentication-Results": "ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org\n (client-ip=140.211.166.138; helo=smtp1.osuosl.org;\n envelope-from=ovs-dev-bounces@openvswitch.org; receiver=<UNKNOWN>)", "Received": [ "from smtp1.osuosl.org (smtp1.osuosl.org [140.211.166.138])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest\n SHA256)\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 4H8fys6VpDz9sSs\n\tfor <incoming@patchwork.ozlabs.org>; Wed, 15 Sep 2021 22:43:31 +1000 (AEST)", "from localhost (localhost [127.0.0.1])\n\tby smtp1.osuosl.org (Postfix) with ESMTP id 8A3CB82978;\n\tWed, 15 Sep 2021 12:43:29 +0000 (UTC)", "from smtp1.osuosl.org ([127.0.0.1])\n\tby localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id I2wtpIfUytKp; Wed, 15 Sep 2021 12:43:28 +0000 (UTC)", "from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56])\n\tby smtp1.osuosl.org (Postfix) with ESMTPS id D13DE826D6;\n\tWed, 15 Sep 2021 12:43:27 +0000 (UTC)", "from lf-lists.osuosl.org (localhost [127.0.0.1])\n\tby lists.linuxfoundation.org (Postfix) with ESMTP id A9494C0011;\n\tWed, 15 Sep 2021 12:43:27 +0000 (UTC)", "from smtp1.osuosl.org (smtp1.osuosl.org [140.211.166.138])\n by lists.linuxfoundation.org (Postfix) with ESMTP id A607BC000D\n for <dev@openvswitch.org>; Wed, 15 Sep 2021 12:43:25 +0000 (UTC)", "from localhost (localhost [127.0.0.1])\n by smtp1.osuosl.org (Postfix) with ESMTP id 8FF2482702\n for <dev@openvswitch.org>; Wed, 15 Sep 2021 12:43:25 +0000 (UTC)", "from smtp1.osuosl.org ([127.0.0.1])\n by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n with ESMTP id XLI-83lOhP6T for <dev@openvswitch.org>;\n Wed, 15 Sep 2021 12:43:24 +0000 (UTC)", "from mellanox.co.il (mail-il-dmz.mellanox.com [193.47.165.129])\n by smtp1.osuosl.org (Postfix) with ESMTP id F097F826D6\n for <dev@openvswitch.org>; Wed, 15 Sep 2021 12:43:23 +0000 (UTC)", "from Internal Mail-Server by MTLPINE1 (envelope-from cmi@nvidia.com)\n with SMTP; 15 Sep 2021 15:43:21 +0300", "from c-141-46-1-010.mtl.labs.mlnx (c-141-46-1-010.mtl.labs.mlnx\n [10.141.46.10])\n by labmailer.mlnx (8.13.8/8.13.8) with ESMTP id 18FChLbU010242;\n Wed, 15 Sep 2021 15:43:21 +0300" ], "X-Virus-Scanned": [ "amavisd-new at osuosl.org", "amavisd-new at osuosl.org" ], "X-Greylist": "domain auto-whitelisted by SQLgrey-1.8.0", "To": "dev@openvswitch.org", "Date": "Wed, 15 Sep 2021 15:43:12 +0300", "Message-Id": "<20210915124319.1683219-1-cmi@nvidia.com>", "X-Mailer": "git-send-email 2.27.0", "MIME-Version": "1.0", "Cc": "elibr@nvidia.com, simon.horman@netronome.com, roniba@nvidia.com,\n i.maximets@ovn.org", "Subject": "[ovs-dev] [PATCH v15 0/7] Add offload support for sFlow", "X-BeenThere": "ovs-dev@openvswitch.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "<ovs-dev.openvswitch.org>", "List-Unsubscribe": "<https://mail.openvswitch.org/mailman/options/ovs-dev>,\n <mailto:ovs-dev-request@openvswitch.org?subject=unsubscribe>", "List-Archive": "<http://mail.openvswitch.org/pipermail/ovs-dev/>", "List-Post": "<mailto:ovs-dev@openvswitch.org>", "List-Help": "<mailto:ovs-dev-request@openvswitch.org?subject=help>", "List-Subscribe": "<https://mail.openvswitch.org/mailman/listinfo/ovs-dev>,\n <mailto:ovs-dev-request@openvswitch.org?subject=subscribe>", "From": "Chris Mi via dev <ovs-dev@openvswitch.org>", "Reply-To": "Chris Mi <cmi@nvidia.com>", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Errors-To": "ovs-dev-bounces@openvswitch.org", "Sender": "\"dev\" <ovs-dev-bounces@openvswitch.org>" }, "content": "This patch set adds offload support for sFlow.\n\nPsample is a genetlink channel for packet sampling. TC action act_sample\nuses psample to send sampled packets to userspace.\n\nWhen offloading sample action to TC, userspace creates a unique ID to\nmap sFlow action and tunnel info and passes this ID to kernel instead\nof the sFlow info. psample will send this ID and sampled packet to\nuserspace. Using the ID, userspace can recover the sFlow info and send\nsampled packet to the right sFlow monitoring host.\n\nv2-v1:\n- Fix robot errors.\nv3-v2:\n- Remove Gerrit Change-Id.\n- Add patch #9 to fix older kernels build issue.\n- Add travis test result.\nv4-v3:\n- Fix offload issue when sampling rate is 1.\nv5-v4:\n- Move polling thread from ofproto to netdev-offload-tc.\nv6-v5:\n- Rebase.\n- Add GitHub Actions test result.\nv7-v6:\n- Remove Gerrit Change-Id.\n- Fix \"ERROR: Inappropriate spacing around cast\"\nv8-v7\n- Address Eelco Chaudron's comment for patch #11.\nv9-v8\n- Remove sflow_len from struct dpif_sflow_attr.\n- Log a debug message for other userspace actions.\nv10-v9\n- Address Eelco Chaudron's comments on v9.\nv11-v10\n- Fix a bracing error.\nv12-v11\n- Add duplicate sample group id check.\nv13-v12\n- Remove the psample poll thread from netdev-offload-tc and reuse\n ofproto handler thread according to Ilya's new desgin.\n- Add dpif-offload-provider layer according to Eli's suggestion.\nv14-v13\n- Fix a robot error.\nv15-v14\n- Address Eelco Chaudron's comments on v14.\n\nChris Mi (7):\n compat: Add psample and tc sample action defines for older kernels\n ovs-kmod-ctl: Load kernel module psample\n dpif-offload-provider: Introduce dpif-offload-provider layer\n netdev-offload-tc: Introduce group ID management API\n dpif-offload-netlink: Implement dpif-offload-provider API\n ofproto: Introduce API to process sFlow offload packet\n netdev-offload-tc: Add offload support for sFlow\n\n NEWS | 1 +\n include/linux/automake.mk | 4 +-\n include/linux/psample.h | 62 +++++\n include/linux/tc_act/tc_sample.h | 25 ++\n lib/automake.mk | 3 +\n lib/dpif-netdev.c | 1 +\n lib/dpif-netlink.c | 2 +\n lib/dpif-offload-netlink.c | 208 ++++++++++++++\n lib/dpif-offload-provider.h | 75 +++++\n lib/dpif-offload.c | 43 +++\n lib/dpif-provider.h | 8 +-\n lib/dpif.c | 10 +\n lib/netdev-offload-tc.c | 459 +++++++++++++++++++++++++++++--\n lib/netdev-offload.h | 1 +\n lib/tc.c | 61 +++-\n lib/tc.h | 16 +-\n ofproto/ofproto-dpif-upcall.c | 63 +++++\n utilities/ovs-kmod-ctl.in | 14 +\n 18 files changed, 1030 insertions(+), 26 deletions(-)\n create mode 100644 include/linux/psample.h\n create mode 100644 include/linux/tc_act/tc_sample.h\n create mode 100644 lib/dpif-offload-netlink.c\n create mode 100644 lib/dpif-offload-provider.h\n create mode 100644 lib/dpif-offload.c" }