Patch Detail

HTTP 200 OK
Allow: GET, PUT, PATCH, HEAD, OPTIONS
Content-Type: application/json
Vary: Accept

{
    "id": 78864,
    "url": "http://patchwork.dpdk.org/api/patches/78864/?format=api",
    "web_url": "http://patchwork.dpdk.org/project/dpdk/patch/20200925174340.10014-5-steven.lariau@arm.com/",
    "project": {
        "id": 1,
        "url": "http://patchwork.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": "<20200925174340.10014-5-steven.lariau@arm.com>",
    "list_archive_url": "https://inbox.dpdk.org/dev/20200925174340.10014-5-steven.lariau@arm.com",
    "date": "2020-09-25T17:43:38",
    "name": "[v2,4/5] lib/stack: reload head when pop fails",
    "commit_ref": null,
    "pull_url": null,
    "state": "accepted",
    "archived": true,
    "hash": "bd81aac8f9056c1a8834377e09adb2cda5076d94",
    "submitter": {
        "id": 1862,
        "url": "http://patchwork.dpdk.org/api/people/1862/?format=api",
        "name": "Steven Lariau",
        "email": "steven.lariau@arm.com"
    },
    "delegate": {
        "id": 24651,
        "url": "http://patchwork.dpdk.org/api/users/24651/?format=api",
        "username": "dmarchand",
        "first_name": "David",
        "last_name": "Marchand",
        "email": "david.marchand@redhat.com"
    },
    "mbox": "http://patchwork.dpdk.org/project/dpdk/patch/20200925174340.10014-5-steven.lariau@arm.com/mbox/",
    "series": [
        {
            "id": 12516,
            "url": "http://patchwork.dpdk.org/api/series/12516/?format=api",
            "web_url": "http://patchwork.dpdk.org/project/dpdk/list/?series=12516",
            "date": "2020-09-25T17:43:34",
            "name": "lib/stack: improve lockfree C11 implementation",
            "version": 2,
            "mbox": "http://patchwork.dpdk.org/series/12516/mbox/"
        }
    ],
    "comments": "http://patchwork.dpdk.org/api/patches/78864/comments/",
    "check": "success",
    "checks": "http://patchwork.dpdk.org/api/patches/78864/checks/",
    "tags": {},
    "related": [],
    "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 CFA0FA04C0;\n\tFri, 25 Sep 2020 19:45:32 +0200 (CEST)",
            "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id A73E81EA02;\n\tFri, 25 Sep 2020 19:44:29 +0200 (CEST)",
            "from foss.arm.com (foss.arm.com [217.140.110.172])\n by dpdk.org (Postfix) with ESMTP id 7F2521E9E9;\n Fri, 25 Sep 2020 19:44:17 +0200 (CEST)",
            "from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14])\n by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id CADDF101E;\n Fri, 25 Sep 2020 10:44:14 -0700 (PDT)",
            "from localhost.localdomain (unknown [10.57.54.55])\n by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id CD3F63F718;\n Fri, 25 Sep 2020 10:44:13 -0700 (PDT)"
        ],
        "From": "Steven Lariau <steven.lariau@arm.com>",
        "To": "Gage Eads <gage.eads@intel.com>, Olivier Matz <olivier.matz@6wind.com>,\n Honnappa Nagarahalli <honnappa.nagarahalli@arm.com>",
        "Cc": "dev@dpdk.org, nd@arm.com, Steven Lariau <steven.lariau@arm.com>,\n stable@dpdk.org",
        "Date": "Fri, 25 Sep 2020 18:43:38 +0100",
        "Message-Id": "<20200925174340.10014-5-steven.lariau@arm.com>",
        "X-Mailer": "git-send-email 2.17.1",
        "In-Reply-To": "<20200925174340.10014-1-steven.lariau@arm.com>",
        "References": "<20200911152938.8019-1-steven.lariau@arm.com>\n <20200925174340.10014-1-steven.lariau@arm.com>",
        "Subject": "[dpdk-dev] [PATCH v2 4/5] lib/stack: reload head when pop fails",
        "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": "List head must be loaded right before continue (when failed to\nfind the new head).\nWithout this, one thread might keep trying and failing to pop items\nwithout ever loading the new correct head.\n\nFixes: 7e6e609939a8 (\"stack: add C11 atomic implementation\")\nCc: gage.eads@intel.com\nCc: stable@dpdk.org\n\nSigned-off-by: Steven Lariau <steven.lariau@arm.com>\nReviewed-by: Dharmik Thakkar <dharmik.thakkar@arm.com>\nReviewed-by: Ruifeng Wang <ruifeng.wang@arm.com>\nAcked-by: Gage Eads <gage.eads@intel.com>\n---\n lib/librte_stack/rte_stack_lf_c11.h | 4 +++-\n 1 file changed, 3 insertions(+), 1 deletion(-)",
    "diff": "diff --git a/lib/librte_stack/rte_stack_lf_c11.h b/lib/librte_stack/rte_stack_lf_c11.h\nindex 2bc639419..adb9f590d 100644\n--- a/lib/librte_stack/rte_stack_lf_c11.h\n+++ b/lib/librte_stack/rte_stack_lf_c11.h\n@@ -133,8 +133,10 @@ __rte_stack_lf_pop_elems(struct rte_stack_lf_list *list,\n \t\t/* If NULL was encountered, the list was modified while\n \t\t * traversing it. Retry.\n \t\t */\n-\t\tif (i != num)\n+\t\tif (i != num) {\n+\t\t\told_head = list->head;\n \t\t\tcontinue;\n+\t\t}\n \n \t\tnew_head.top = tmp;\n \t\tnew_head.cnt = old_head.cnt + 1;\n",
    "prefixes": [
        "v2",
        "4/5"
    ]
}