Show a cover letter.

GET /api/covers/44921/?format=api
HTTP 200 OK
Allow: GET, HEAD, OPTIONS
Content-Type: application/json
Vary: Accept

{
    "id": 44921,
    "url": "http://patches.dpdk.org/api/covers/44921/?format=api",
    "web_url": "http://patches.dpdk.org/project/dpdk/cover/cover.1537363686.git.anatoly.burakov@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": "<cover.1537363686.git.anatoly.burakov@intel.com>",
    "list_archive_url": "https://inbox.dpdk.org/dev/cover.1537363686.git.anatoly.burakov@intel.com",
    "date": "2018-09-19T13:56:24",
    "name": "[v2,00/20] Support externally allocated memory in DPDK",
    "submitter": {
        "id": 4,
        "url": "http://patches.dpdk.org/api/people/4/?format=api",
        "name": "Burakov, Anatoly",
        "email": "anatoly.burakov@intel.com"
    },
    "mbox": "http://patches.dpdk.org/project/dpdk/cover/cover.1537363686.git.anatoly.burakov@intel.com/mbox/",
    "series": [
        {
            "id": 1391,
            "url": "http://patches.dpdk.org/api/series/1391/?format=api",
            "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=1391",
            "date": "2018-09-19T13:56:25",
            "name": "Support externally allocated memory in DPDK",
            "version": 2,
            "mbox": "http://patches.dpdk.org/series/1391/mbox/"
        }
    ],
    "comments": "http://patches.dpdk.org/api/covers/44921/comments/",
    "headers": {
        "Return-Path": "<dev-bounces@dpdk.org>",
        "X-Original-To": "patchwork@dpdk.org",
        "Delivered-To": "patchwork@dpdk.org",
        "Received": [
            "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 4DF2F4D3A;\n\tWed, 19 Sep 2018 15:56:58 +0200 (CEST)",
            "from mga01.intel.com (mga01.intel.com [192.55.52.88])\n\tby dpdk.org (Postfix) with ESMTP id AFC90326C\n\tfor <dev@dpdk.org>; Wed, 19 Sep 2018 15:56:49 +0200 (CEST)",
            "from orsmga004.jf.intel.com ([10.7.209.38])\n\tby fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t19 Sep 2018 06:56:48 -0700",
            "from irvmail001.ir.intel.com ([163.33.26.43])\n\tby orsmga004.jf.intel.com with ESMTP; 19 Sep 2018 06:56:45 -0700",
            "from sivswdev01.ir.intel.com (sivswdev01.ir.intel.com\n\t[10.237.217.45])\n\tby irvmail001.ir.intel.com (8.14.3/8.13.6/MailSET/Hub) with ESMTP id\n\tw8JDuiYs004973; Wed, 19 Sep 2018 14:56:44 +0100",
            "from sivswdev01.ir.intel.com (localhost [127.0.0.1])\n\tby sivswdev01.ir.intel.com with ESMTP id w8JDuibr023403;\n\tWed, 19 Sep 2018 14:56:44 +0100",
            "(from aburakov@localhost)\n\tby sivswdev01.ir.intel.com with LOCAL id w8JDuiwZ023399;\n\tWed, 19 Sep 2018 14:56:44 +0100"
        ],
        "X-Amp-Result": "SKIPPED(no attachment in message)",
        "X-Amp-File-Uploaded": "False",
        "X-ExtLoop1": "1",
        "X-IronPort-AV": "E=Sophos;i=\"5.53,394,1531810800\"; d=\"scan'208\";a=\"234223383\"",
        "From": "Anatoly Burakov <anatoly.burakov@intel.com>",
        "To": "dev@dpdk.org",
        "Cc": "laszlo.madarassy@ericsson.com, laszlo.vadkerti@ericsson.com,\n\tandras.kovacs@ericsson.com, winnie.tian@ericsson.com,\n\tdaniel.andrasi@ericsson.com, janos.kobor@ericsson.com,\n\tgeza.koblo@ericsson.com, srinath.mannam@broadcom.com,\n\tscott.branden@broadcom.com, ajit.khaparde@broadcom.com,\n\tkeith.wiles@intel.com, bruce.richardson@intel.com, thomas@monjalon.net,\n\tshreyansh.jain@nxp.com, shahafs@mellanox.com",
        "Date": "Wed, 19 Sep 2018 14:56:24 +0100",
        "Message-Id": "<cover.1537363686.git.anatoly.burakov@intel.com>",
        "X-Mailer": "git-send-email 1.7.0.7",
        "In-Reply-To": "<cover.1536064999.git.anatoly.burakov@intel.com>",
        "References": "<cover.1536064999.git.anatoly.burakov@intel.com>",
        "Subject": "[dpdk-dev] [PATCH v2 00/20] Support externally allocated memory in\n\tDPDK",
        "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\t<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\t<mailto:dev-request@dpdk.org?subject=subscribe>",
        "Errors-To": "dev-bounces@dpdk.org",
        "Sender": "\"dev\" <dev-bounces@dpdk.org>"
    },
    "content": "This is a proposal to enable using externally allocated memory\nin DPDK.\n\nIn a nutshell, here is what is being done here:\n\n- Index internal malloc heaps by NUMA node index, rather than NUMA\n  node itself (external heaps will have ID's in order of creation)\n- Add identifier string to malloc heap, to uniquely identify it\n  - Each new heap will receive a unique socket ID that will be used by\n    allocator to decide from which heap (internal or external) to\n    allocate requested amount of memory\n- Allow creating named heaps and add/remove memory to/from those heaps\n- Allocate memseg lists at runtime, to keep track of IOVA addresses\n  of externally allocated memory\n  - If IOVA addresses aren't provided, use RTE_BAD_IOVA\n- Allow malloc and memzones to allocate from external heaps\n- Allow other data structures to allocate from externall heaps\n\nThe responsibility to ensure memory is accessible before using it is\non the shoulders of the user - there is no checking done with regards\nto validity of the memory (nor could there be...).\n\nThe general approach is to create heap and add memory into it. For any\nother process wishing to use the same memory, said memory must first\nbe attached (otherwise some things will not work).\n\nA design decision was made to make multiprocess synchronization a\nmanual process. Due to underlying issues with attaching to fbarrays in\nsecondary processes, this design was deemed to be better because we\ndon't want to fail to create external heap in the primary because\nsomething in the secondary has failed when in fact we may not eve have\nwanted this memory to be accessible in the secondary in the first\nplace.\n\nUsing external memory in multiprocess is *hard*, because not only\nmemory space needs to be preallocated, but it also needs to be attached\nin each process to allow other processes to access the page table. The\nattach API call may or may not succeed, depending on memory layout, for\nreasons similar to other multiprocess failures. This is treated as a\n\"known issue\" for this release.\n\nCreating and destroying heaps is currently restricted to primary\nprocesses, because we need to keep track of all socket ID's we've ever\nused to prevent their reuse, and obviously different processes would\nhave kept different socket ID counters, and it isn't important enough\nto put into shared memory. This means that secondary processes will\nnot be able to create new heaps. If this use case is important\nenough, we can put the max socket ID into shared memory, or allow\nsocket ID reuse (which i do not think is a good idea because it has\nthe potential to make things harder to debug).\n\nv2 -> v1 changes:\n- Fixed NULL dereference on heap socket ID lookup\n- Fixed memseg offset calculation on adding memory to heap\n- Improved unit test to test for above bugfixes\n- Restricted heap creation to primary processes only\n- Added sample application\n- Added documentation\n\nRFC -> v1 changes:\n- Removed the \"named heaps\" API, allocate using fake socket ID instead\n- Added multiprocess support\n- Everything is now thread-safe\n- Numerous bugfixes and API improvements\n\nAnatoly Burakov (20):\n  mem: add length to memseg list\n  mem: allow memseg lists to be marked as external\n  malloc: index heaps using heap ID rather than NUMA node\n  mem: do not check for invalid socket ID\n  flow_classify: do not check for invalid socket ID\n  pipeline: do not check for invalid socket ID\n  sched: do not check for invalid socket ID\n  malloc: add name to malloc heaps\n  malloc: add function to query socket ID of named heap\n  malloc: allow creating malloc heaps\n  malloc: allow destroying heaps\n  malloc: allow adding memory to named heaps\n  malloc: allow removing memory from named heaps\n  malloc: allow attaching to external memory chunks\n  malloc: allow detaching from external memory\n  test: add unit tests for external memory support\n  examples: add external memory example app\n  doc: add external memory feature to the release notes\n  doc: add external memory feature to programmer's guide\n  doc: add external memory sample application guide\n\n config/common_base                            |   1 +\n config/rte_config.h                           |   1 +\n .../prog_guide/env_abstraction_layer.rst      |  38 ++\n doc/guides/rel_notes/deprecation.rst          |  15 -\n doc/guides/rel_notes/release_18_11.rst        |  24 +-\n doc/guides/sample_app_ug/external_mem.rst     | 115 +++++\n doc/guides/sample_app_ug/index.rst            |   1 +\n drivers/bus/fslmc/fslmc_vfio.c                |   7 +-\n drivers/bus/pci/linux/pci.c                   |   2 +-\n drivers/net/mlx4/mlx4_mr.c                    |   3 +\n drivers/net/mlx5/mlx5.c                       |   5 +-\n drivers/net/mlx5/mlx5_mr.c                    |   3 +\n drivers/net/virtio/virtio_user/vhost_kernel.c |   5 +-\n examples/external_mem/Makefile                |  62 +++\n examples/external_mem/extmem.c                | 461 ++++++++++++++++++\n examples/external_mem/meson.build             |  12 +\n lib/librte_eal/bsdapp/eal/Makefile            |   2 +-\n lib/librte_eal/bsdapp/eal/eal.c               |   3 +\n lib/librte_eal/bsdapp/eal/eal_memory.c        |   9 +-\n lib/librte_eal/common/eal_common_memory.c     |   9 +-\n lib/librte_eal/common/eal_common_memzone.c    |   8 +-\n .../common/include/rte_eal_memconfig.h        |   6 +-\n lib/librte_eal/common/include/rte_malloc.h    | 183 +++++++\n .../common/include/rte_malloc_heap.h          |   3 +\n lib/librte_eal/common/include/rte_memory.h    |   9 +\n lib/librte_eal/common/malloc_heap.c           | 300 ++++++++++--\n lib/librte_eal/common/malloc_heap.h           |  17 +\n lib/librte_eal/common/rte_malloc.c            | 393 ++++++++++++++-\n lib/librte_eal/linuxapp/eal/Makefile          |   2 +-\n lib/librte_eal/linuxapp/eal/eal.c             |  10 +-\n lib/librte_eal/linuxapp/eal/eal_memalloc.c    |  12 +-\n lib/librte_eal/linuxapp/eal/eal_memory.c      |   4 +-\n lib/librte_eal/linuxapp/eal/eal_vfio.c        |  17 +-\n lib/librte_eal/meson.build                    |   2 +-\n lib/librte_eal/rte_eal_version.map            |   7 +\n lib/librte_flow_classify/rte_flow_classify.c  |   3 +-\n lib/librte_mempool/rte_mempool.c              |  31 +-\n lib/librte_pipeline/rte_pipeline.c            |   3 +-\n lib/librte_sched/rte_sched.c                  |   2 +-\n test/test/Makefile                            |   1 +\n test/test/autotest_data.py                    |  14 +-\n test/test/meson.build                         |   1 +\n test/test/test_external_mem.c                 | 389 +++++++++++++++\n test/test/test_malloc.c                       |   3 +\n test/test/test_memzone.c                      |   3 +\n 45 files changed, 2096 insertions(+), 105 deletions(-)\n create mode 100644 doc/guides/sample_app_ug/external_mem.rst\n create mode 100644 examples/external_mem/Makefile\n create mode 100644 examples/external_mem/extmem.c\n create mode 100644 examples/external_mem/meson.build\n create mode 100644 test/test/test_external_mem.c"
}