Cover Detail
Show a cover letter.
GET /api/covers/69208/?format=api
{ "id": 69208, "url": "http://patches.dpdk.org/api/covers/69208/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/20200424092445.44693-1-yong.liu@intel.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20200424092445.44693-1-yong.liu@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20200424092445.44693-1-yong.liu@intel.com", "date": "2020-04-24T09:24:36", "name": "[v9,0/9] add packed ring vectorized path", "submitter": { "id": 17, "url": "http://patches.dpdk.org/api/people/17/?format=api", "name": "Marvin Liu", "email": "yong.liu@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/cover/20200424092445.44693-1-yong.liu@intel.com/mbox/", "series": [ { "id": 9605, "url": "http://patches.dpdk.org/api/series/9605/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=9605", "date": "2020-04-24T09:24:36", "name": "add packed ring vectorized path", "version": 9, "mbox": "http://patches.dpdk.org/series/9605/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/69208/comments/", "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from dpdk.org (dpdk.org [92.243.14.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 08F04A00C4;\n\tFri, 24 Apr 2020 03:49:31 +0200 (CEST)", "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 1A1671C208;\n\tFri, 24 Apr 2020 03:49:30 +0200 (CEST)", "from mga06.intel.com (mga06.intel.com [134.134.136.31])\n by dpdk.org (Postfix) with ESMTP id 3744A1C204\n for <dev@dpdk.org>; Fri, 24 Apr 2020 03:49:28 +0200 (CEST)", "from orsmga007.jf.intel.com ([10.7.209.58])\n by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 23 Apr 2020 18:49:27 -0700", "from npg-dpdk-virtual-marvin-dev.sh.intel.com ([10.67.119.56])\n by orsmga007.jf.intel.com with ESMTP; 23 Apr 2020 18:49:25 -0700" ], "IronPort-SDR": [ "\n I0mm+LDWqmF5RP1dCSGKe2WrSQH/CCheqc6hRoZOSMzr2WoCOYEyzY49io+AsFY4K/bWHxBJwi\n 2Nz7e1TucbDg==", "\n E9FumEX0MBI+O+eocI6lc/7YqKMlVv1wZ8s1rJsZc7u0FY25/sVLPe0TEd/v9lfmzGoSycoXiO\n 5y3RtssxjRkA==" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.73,310,1583222400\"; d=\"scan'208\";a=\"245083868\"", "From": "Marvin Liu <yong.liu@intel.com>", "To": "maxime.coquelin@redhat.com, xiaolong.ye@intel.com, zhihong.wang@intel.com", "Cc": "dev@dpdk.org, harry.van.haaren@intel.com, Marvin Liu <yong.liu@intel.com>", "Date": "Fri, 24 Apr 2020 17:24:36 +0800", "Message-Id": "<20200424092445.44693-1-yong.liu@intel.com>", "X-Mailer": "git-send-email 2.17.1", "In-Reply-To": "<20200313174230.74661-1-yong.liu@intel.com>", "References": "<20200313174230.74661-1-yong.liu@intel.com>", "Subject": "[dpdk-dev] [PATCH v9 0/9] add packed ring vectorized path", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "This patch set introduced vectorized path for packed ring.\n\nThe size of packed ring descriptor is 16Bytes. Four batched descriptors\nare just placed into one cacheline. AVX512 instructions can well handle\nthis kind of data. Packed ring TX path can fully transformed into\nvectorized path. Packed ring Rx path can be vectorized when requirements\nmet(LRO and mergeable disabled).\n\nNew option RTE_LIBRTE_VIRTIO_INC_VECTOR will be introduced in this\npatch set. This option will unify split and packed ring vectorized\npath default setting. Meanwhile user can specify whether enable\nvectorized path at runtime by 'vectorized' parameter of virtio user\nvdev.\n\nv9:\n* replace RTE_LIBRTE_VIRTIO_INC_VECTOR with vectorized devarg\n* reorder patch sequence\n\nv8:\n* fix meson build error on ubuntu16.04 and suse15\n\nv7:\n* default vectorization is disabled\n* compilation time check dependency on rte_mbuf structure\n* offsets are calcuated when compiling\n* remove useless barrier as descs are batched store&load\n* vindex of scatter is directly set\n* some comments updates\n* enable vectorized path in meson build\n\nv6:\n* fix issue when size not power of 2\n\nv5:\n* remove cpuflags definition as required extensions always come with\n AVX512F on x86_64\n* inorder actions should depend on feature bit\n* check ring type in rx queue setup\n* rewrite some commit logs\n* fix some checkpatch warnings\n\nv4:\n* rename 'packed_vec' to 'vectorized', also used in split ring\n* add RTE_LIBRTE_VIRTIO_INC_VECTOR config for virtio ethdev\n* check required AVX512 extensions cpuflags\n* combine split and packed ring datapath selection logic\n* remove limitation that size must power of two\n* clear 12Bytes virtio_net_hdr\n\nv3:\n* remove virtio_net_hdr array for better performance\n* disable 'packed_vec' by default\n\nv2:\n* more function blocks replaced by vector instructions\n* clean virtio_net_hdr by vector instruction\n* allow header room size change\n* add 'packed_vec' option in virtio_user vdev \n* fix build not check whether AVX512 enabled\n* doc update\n\nTested-by: Wang, Yinan <yinan.wang@intel.com>\n\nMarvin Liu (9):\n net/virtio: add Rx free threshold setting\n net/virtio: inorder should depend on feature bit\n net/virtio: add vectorized devarg\n net/virtio-user: add vectorized devarg\n net/virtio: add vectorized packed ring Rx path\n net/virtio: reuse packed ring xmit functions\n net/virtio: add vectorized packed ring Tx path\n net/virtio: add election for vectorized path\n doc: add packed vectorized path\n\n doc/guides/nics/virtio.rst | 52 +-\n drivers/net/virtio/Makefile | 35 ++\n drivers/net/virtio/meson.build | 14 +\n drivers/net/virtio/virtio_ethdev.c | 136 +++-\n drivers/net/virtio/virtio_ethdev.h | 6 +\n drivers/net/virtio/virtio_pci.h | 3 +-\n drivers/net/virtio/virtio_rxtx.c | 212 ++-----\n drivers/net/virtio/virtio_rxtx_packed_avx.c | 665 ++++++++++++++++++++\n drivers/net/virtio/virtio_user_ethdev.c | 32 +-\n drivers/net/virtio/virtqueue.c | 7 +-\n drivers/net/virtio/virtqueue.h | 168 ++++-\n 11 files changed, 1112 insertions(+), 218 deletions(-)\n create mode 100644 drivers/net/virtio/virtio_rxtx_packed_avx.c" }