Show a cover letter.

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

{
    "id": 43761,
    "url": "http://patches.dpdk.org/api/covers/43761/?format=api",
    "web_url": "http://patches.dpdk.org/project/dpdk/cover/1534502916-31636-1-git-send-email-jia.guo@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": "<1534502916-31636-1-git-send-email-jia.guo@intel.com>",
    "list_archive_url": "https://inbox.dpdk.org/dev/1534502916-31636-1-git-send-email-jia.guo@intel.com",
    "date": "2018-08-17T10:48:27",
    "name": "[v10,0/8] hotplug failure handle mechanism",
    "submitter": {
        "id": 507,
        "url": "http://patches.dpdk.org/api/people/507/?format=api",
        "name": "Guo, Jia",
        "email": "jia.guo@intel.com"
    },
    "mbox": "http://patches.dpdk.org/project/dpdk/cover/1534502916-31636-1-git-send-email-jia.guo@intel.com/mbox/",
    "series": [
        {
            "id": 1007,
            "url": "http://patches.dpdk.org/api/series/1007/?format=api",
            "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=1007",
            "date": "2018-08-17T10:48:27",
            "name": "hotplug failure handle mechanism",
            "version": 10,
            "mbox": "http://patches.dpdk.org/series/1007/mbox/"
        }
    ],
    "comments": "http://patches.dpdk.org/api/covers/43761/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 DFD99FEB;\n\tFri, 17 Aug 2018 12:51:22 +0200 (CEST)",
            "from mga14.intel.com (mga14.intel.com [192.55.52.115])\n\tby dpdk.org (Postfix) with ESMTP id 100EF98\n\tfor <dev@dpdk.org>; Fri, 17 Aug 2018 12:51:20 +0200 (CEST)",
            "from fmsmga007.fm.intel.com ([10.253.24.52])\n\tby fmsmga103.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t17 Aug 2018 03:51:20 -0700",
            "from jeffguo-z170x-ud5.sh.intel.com (HELO localhost.localdomain)\n\t([10.67.104.10])\n\tby fmsmga007.fm.intel.com with ESMTP; 17 Aug 2018 03:51:16 -0700"
        ],
        "X-Amp-Result": "SKIPPED(no attachment in message)",
        "X-Amp-File-Uploaded": "False",
        "X-ExtLoop1": "1",
        "X-IronPort-AV": "E=Sophos;i=\"5.53,250,1531810800\"; d=\"scan'208\";a=\"63131811\"",
        "From": "Jeff Guo <jia.guo@intel.com>",
        "To": "stephen@networkplumber.org, bruce.richardson@intel.com,\n\tferruh.yigit@intel.com, konstantin.ananyev@intel.com,\n\tgaetan.rivet@6wind.com, jingjing.wu@intel.com, thomas@monjalon.net,\n\tmotih@mellanox.com, matan@mellanox.com, harry.van.haaren@intel.com,\n\tqi.z.zhang@intel.com, shaopeng.he@intel.com,\n\tbernard.iremonger@intel.com, \n\tarybchenko@solarflare.com, wenzhuo.lu@intel.com",
        "Cc": "jblunck@infradead.org, shreyansh.jain@nxp.com, dev@dpdk.org,\n\tjia.guo@intel.com, helin.zhang@intel.com",
        "Date": "Fri, 17 Aug 2018 18:48:27 +0800",
        "Message-Id": "<1534502916-31636-1-git-send-email-jia.guo@intel.com>",
        "X-Mailer": "git-send-email 2.7.4",
        "In-Reply-To": "<1498711073-42917-1-git-send-email-jia.guo@intel.com>",
        "References": "<1498711073-42917-1-git-send-email-jia.guo@intel.com>",
        "MIME-Version": "1.0",
        "Content-Type": "text/plain; charset=UTF-8",
        "Content-Transfer-Encoding": "8bit",
        "Subject": "[dpdk-dev] [PATCH v10 0/8] hotplug failure handle mechanism",
        "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": "Hotplug is an important feature for use-cases like the datacenter device's\nfail-safe and for SRIOV Live Migration in SDN/NFV. It could bring higher\nflexibility and continuality to networking services in multiple use-cases\nin the industry. So let's see how DPDK can help users implement hotplug\nsolutions.\n\nWe already have a general device-event monitor mechanism, failsafe driver,\nand hot plug/unplug API in DPDK. We have already got the solution of\n“ethdev event + kernel PMD hotplug handler + failsafe”, but we still not\ngot “eal event + hotplug handler for pci PMD + failsafe” implement, and we\nneed to considerate 2 different solutions between uio  and vfio.\n \nIn the case of hotplug for igb_uio, when a hardware device be removed\nphysically or disabled in software, the application needs to be notified\nand detach the device out of the bus, and then make the device invalidate.\nThe problem is that, the removal of the device is not instantaneous in\nsoftware. If the application data path tries to read/write to the device\nwhen removal is still in process, it will cause an MMIO error and\napplication will crash.\n\nIn this patch set, we propose a PCIe bus failure handler mechanism for\nhotplug in igb_uio. It aims to guarantee that, when a hot unplug occurs,\nthe system will not crash and the application will not break out. \n\nThe mechanism should work as below:\n\nFirst, the application enables the device event monitor, registers the\nhotplug event’s callback and enable hotplug handling before running the\ndata path. Once the hot unplug occurs, the mechanism will detect the\nremoval event and then accordingly do the failure handling. In order to\ndo that, the below functionality will be required:\n - Add a new bus ops “memory_failure_handler” to handle bus read/write\n   errors.\n - Implement pci bus specific ops “pci_memory_failure_handler”. It will\n   be based on the failure address to remap memory for the corresponding\n   device that unplugged.\n\nFor the data path or other unexpected behaviors from the control path\nwhen a hot unplug occurs:\n - Add a new bus ops “sigbus_handler”, that is responsible for handling\n   the sigbus error which is either an original memory error, or a specific\n   memory error that is caused by a hot unplug. When a sigbus error is\n   captured, it will call this function to handle sigbus error.\n - Implement PCI bus specific ops “pci_sigbus_handler”. It will iterate all\n   device on PCI bus to find which device encounter the failure.\n - Implement a \"rte_bus_sigbus_handler\" to iterate all buses to find a bus\n   to handle the failure.\n - Add a couple of APIs “rte_dev_hotplug_handle_enable” and\n   “rte_dev_hotplug_handle_diable” to enable/disable hotplug handling.\n   It will monitor the sigbus error by a handler which is per-process.\n   Based on the signal event principle, the control path thread and the\n   data path thread will randomly receive the sigbus error, but will call the\n   common sigbus process. When sigbus be captured, it will call the above API\n   to find bus to handle it.\n\nThe mechanism could be used by app or PMDs. For example, the whole process\nof hotplug in testpmd is:\n - Enable device event monitor->Enable hotplug handle->Register event callback\n   ->attach port->start port->start forwarding->Device unplug->failure handle\n   ->stop forwarding->stop port->close port->detach port.\n\nThis patch set would not cover hotplug insert and binding, and it is only\nimplement the igb_uio failure handler, the vfio hotplug failure handler\nwill be in next coming patch set.\n\npatchset history:\nv10->v9:\nmodify the api name and exposure out for public use.\nadd hotplug handle enable/disable APIs\nrefine commit log\n\nv9->v8:\nrefine commit log to be more readable.\n\nv8->v7:\nrefine errno process in sigbus handler.\nrefine igb uio release process\n\nv7->v6:\ndelete some unused part\n\nv6->v5:\nrefine some description about bus ops\nrefine commit log\nadd some entry check.\n\nv5->v4:\nsplit patches to focus on the failure handle, remove the event usage\nby testpmd to another patch.\nchange the hotplug failure handler name.\nrefine the sigbus handle logic.\nadd lock for udev state in igb uio driver.\n\nv4->v3:\nsplit patches to be small and clear.\nchange to use new parameter \"--hotplug-mode\" in testpmd to identify\nthe eal hotplug and ethdev hotplug.\n\nv3->v2:\nchange bus ops name to bus_hotplug_handler.\nadd new API and bus ops of bus_signal_handler distingush handle generic.\nsigbus and hotplug sigbus.\n\nv2->v1(v21):\nrefine some doc and commit log.\nfix igb uio kernel issue for control path failure rebase testpmd code.\n\nSince the hot plug solution be discussed serval around in the public,\nthe scope be changed and the patch set be split into many times. Coming\nto the recently RFC and feature design, it just focus on the hot unplug\nfailure handler at this patch set, so in order let this topic more clear\nand focus, summarize privours patch set in history “v1(v21)”, the v2 here\ngo ahead for further track.\n\n\"v1(21)\" == v21 as below:\nv21->v20:\nsplit function in hot unplug ops.\nsync failure hanlde to fix multiple process issue fix attach port issue for multiple devices case.\ncombind rmv callback function to be only one.\n\nv20->v19:\nclean the code.\nrefine the remap logic for multiple device.\nremove the auto binding.\n\nv19->18:\nnote for limitation of multiple hotplug, fix some typo, sqeeze patch.\n\nv18->v15:\nadd document, add signal bus handler, refine the code to be more clear.\n\nthe prior patch history please check the patch set \"add device event monitor framework\".\n\nJeff Guo (8):\n  bus: add memory failure handler\n  bus/pci: implement memory failure handler ops\n  bus: add sigbus handler\n  bus/pci: implement sigbus handler ops\n  bus: add helper to handle sigbus\n  eal: add failure handle mechanism for hotplug\n  igb_uio: fix unexpected remove issue for hotplug\n  testpmd: use hotplug failure handle mechanism\n\n app/test-pmd/testpmd.c                  |  27 +++++-\n doc/guides/rel_notes/release_18_08.rst  |   5 +\n drivers/bus/pci/pci_common.c            |  81 ++++++++++++++++\n drivers/bus/pci/pci_common_uio.c        |  33 +++++++\n drivers/bus/pci/private.h               |  12 +++\n kernel/linux/igb_uio/igb_uio.c          |  69 +++++++++++---\n lib/librte_eal/bsdapp/eal/eal_dev.c     |  14 +++\n lib/librte_eal/common/eal_common_bus.c  |  43 +++++++++\n lib/librte_eal/common/eal_private.h     |  38 ++++++++\n lib/librte_eal/common/include/rte_bus.h |  34 +++++++\n lib/librte_eal/common/include/rte_dev.h |  26 ++++++\n lib/librte_eal/linuxapp/eal/eal_dev.c   | 159 +++++++++++++++++++++++++++++++-\n lib/librte_eal/rte_eal_version.map      |   2 +\n 13 files changed, 523 insertions(+), 20 deletions(-)"
}