Patch Detail

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

{
    "id": 40690,
    "url": "http://patches.dpdk.org/api/patches/40690/?format=api",
    "web_url": "http://patches.dpdk.org/project/dpdk/patch/20180606122627.18418-1-3chas3@gmail.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": "<20180606122627.18418-1-3chas3@gmail.com>",
    "list_archive_url": "https://inbox.dpdk.org/dev/20180606122627.18418-1-3chas3@gmail.com",
    "date": "2018-06-06T12:26:27",
    "name": "[dpdk-dev] net/bonding: don't clear active slave count",
    "commit_ref": null,
    "pull_url": null,
    "state": "superseded",
    "archived": true,
    "hash": "513a6c8685bac180e33eb92f14d9454b91dae5b7",
    "submitter": {
        "id": 341,
        "url": "http://patches.dpdk.org/api/people/341/?format=api",
        "name": "Chas Williams",
        "email": "3chas3@gmail.com"
    },
    "delegate": {
        "id": 319,
        "url": "http://patches.dpdk.org/api/users/319/?format=api",
        "username": "fyigit",
        "first_name": "Ferruh",
        "last_name": "Yigit",
        "email": "ferruh.yigit@amd.com"
    },
    "mbox": "http://patches.dpdk.org/project/dpdk/patch/20180606122627.18418-1-3chas3@gmail.com/mbox/",
    "series": [
        {
            "id": 19,
            "url": "http://patches.dpdk.org/api/series/19/?format=api",
            "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=19",
            "date": "2018-06-06T12:26:27",
            "name": "[dpdk-dev] net/bonding: don't clear active slave count",
            "version": 1,
            "mbox": "http://patches.dpdk.org/series/19/mbox/"
        }
    ],
    "comments": "http://patches.dpdk.org/api/patches/40690/comments/",
    "check": "success",
    "checks": "http://patches.dpdk.org/api/patches/40690/checks/",
    "tags": {},
    "related": [],
    "headers": {
        "Return-Path": "<3chas3@gmail.com>",
        "Received": [
            "from mail-qt0-f193.google.com (mail-qt0-f193.google.com\n\t[209.85.216.193]) by dpdk.org (Postfix) with ESMTP id EC5D4CF90\n\tfor <dev@dpdk.org>; Wed,  6 Jun 2018 14:26:33 +0200 (CEST)",
            "by mail-qt0-f193.google.com with SMTP id q6-v6so6070207qtn.7\n\tfor <dev@dpdk.org>; Wed, 06 Jun 2018 05:26:33 -0700 (PDT)",
            "from monolith.fios-router.home\n\t(pool-173-79-169-217.washdc.fios.verizon.net. [173.79.169.217])\n\tby smtp.gmail.com with ESMTPSA id\n\tm16-v6sm21185710qtn.92.2018.06.06.05.26.32\n\t(version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256);\n\tWed, 06 Jun 2018 05:26:32 -0700 (PDT)"
        ],
        "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=from:to:cc:subject:date:message-id;\n\tbh=3lmOUr2clF+0PDNuQUATAqpxQAiQ9sKfsnjL1F3yv8s=;\n\tb=rMFnvOaBs3Q7NguvVH7NUJ7neQpndMYwFks+QuIGNXl+qtdjO4WG8xHI3UZt+H7CI/\n\tylFkzpG2cfsOIz+Wu98I6KL4iO9yF/9q3R8cAvxpc4J1VuNmkrzyRKePp7+FRPmdvG2/\n\tFU55CHvw7+NSkaxGslayosFPR7J9i78ajjJhCU8sDlW3IoNMULzHuUEVKOXTs8yt+4ON\n\tb59GtCZ4jlIeCofbsIXXCf1Y0rq93Xb204mCRAlQdgYm+OCY0WvXffc/fcwyED5sT14s\n\tLDHEdl4hrOcM6SLXHOXpfZDUQGdPgqV9gtb/wqfIoZ06hJMssTcxNqHhS0BEiuIwY3gb\n\tEiSw==",
        "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id;\n\tbh=3lmOUr2clF+0PDNuQUATAqpxQAiQ9sKfsnjL1F3yv8s=;\n\tb=qKSab8S9wYrnCRtpdcH6ScyehpEG0tqGt032Av5gScT7621q9FfGFcPzG5wit5unMZ\n\t+gMKPNK1PKr3hv7MK3vNOJa1j2DuQHMrihK3NKFyuLw7rEX2TGzInNI0QlkRqzqTZUDg\n\trzuzJ/CvSDLYonAqC84mFtnWIjCXNgqmMvCoAT7wAx9bem52IT8F/7CojvQD6RzQ+coz\n\tdBiCdJZMTM68DZ4wcSKJlsVuUb6WbEzUhJefcedqWfY+DLUA90R2zG9a/Kxbr3MNkglZ\n\tIiI34s6qIrJmK2rek+ZMbMWF6anySGgCtrz5zZ95jrwqTHnicyMdQk+1aXbVCT2Lqpe7\n\tmJHQ==",
        "X-Gm-Message-State": "APt69E2tp+czpBuqnq1z+yoRG2awKAaV6mpWvL9LBucsiWN31aVjq2w5\n\tAQLBCfd6BG2dkXEzd5B0xPorAg==",
        "X-Google-Smtp-Source": "ADUXVKIHnOVwdXZhbMcns/Cxe7m+7qFGnFmFOSagltbAer4Gg/5DXmpxETUnmZHUYZAC5+t1Gp5PZw==",
        "X-Received": "by 2002:a0c:d911:: with SMTP id\n\tp17-v6mr2366715qvj.78.1528287993218; \n\tWed, 06 Jun 2018 05:26:33 -0700 (PDT)",
        "From": "Chas Williams <3chas3@gmail.com>",
        "To": "dev@dpdk.org",
        "Cc": "declan.doherty@intel.com,\n\t\"Charles (Chas) Williams\" <chas3@att.com>",
        "Date": "Wed,  6 Jun 2018 08:26:27 -0400",
        "Message-Id": "<20180606122627.18418-1-3chas3@gmail.com>",
        "X-Mailer": "git-send-email 2.14.3",
        "Subject": "[dpdk-dev] [PATCH] net/bonding: don't clear active slave count",
        "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://dpdk.org/ml/options/dev>,\n\t<mailto:dev-request@dpdk.org?subject=unsubscribe>",
        "List-Archive": "<http://dpdk.org/ml/archives/dev/>",
        "List-Post": "<mailto:dev@dpdk.org>",
        "List-Help": "<mailto:dev-request@dpdk.org?subject=help>",
        "List-Subscribe": "<https://dpdk.org/ml/listinfo/dev>,\n\t<mailto:dev-request@dpdk.org?subject=subscribe>",
        "X-List-Received-Date": "Wed, 06 Jun 2018 12:26:34 -0000"
    },
    "content": "From: \"Charles (Chas) Williams\" <chas3@att.com>\n\nWhen the bond PMD is stopped, the active slave count is reset.\nFor 802.3ad mode this potentially leaks memory and clears state since\na second sequential activate_slave() will occur when the bond PMD is\nrestarted and the LSC callback is triggered while the active slave\ncount is 0. To fix this, don't clear the active slave count when\nstopping. Only deactivate_slave() should be used to clear the slaves.\n\nSigned-off-by: Chas Williams <chas3@att.com>\n---\n drivers/net/bonding/rte_eth_bond_pmd.c | 1 -\n 1 file changed, 1 deletion(-)",
    "diff": "diff --git a/drivers/net/bonding/rte_eth_bond_pmd.c b/drivers/net/bonding/rte_eth_bond_pmd.c\nindex 02d94b1b1..4ae577078 100644\n--- a/drivers/net/bonding/rte_eth_bond_pmd.c\n+++ b/drivers/net/bonding/rte_eth_bond_pmd.c\n@@ -2173,7 +2173,6 @@ bond_ethdev_stop(struct rte_eth_dev *eth_dev)\n \t\t\ttlb_last_obytets[internals->active_slaves[i]] = 0;\n \t}\n \n-\tinternals->active_slave_count = 0;\n \tinternals->link_status_polling_enabled = 0;\n \tfor (i = 0; i < internals->slave_count; i++)\n \t\tinternals->slaves[i].last_link_status = 0;\n",
    "prefixes": [
        "dpdk-dev"
    ]
}