Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/59686/?format=api
{ "id": 59686, "url": "https://patches.dpdk.org/api/patches/59686/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20190924173953.25644-1-stephen@networkplumber.org/", "project": { "id": 1, "url": "https://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": "<20190924173953.25644-1-stephen@networkplumber.org>", "list_archive_url": "https://inbox.dpdk.org/dev/20190924173953.25644-1-stephen@networkplumber.org", "date": "2019-09-24T17:39:53", "name": "[v6] eal: make lcore_config private", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "fe83c7c95e7b6495461900454dc49dc5b2729259", "submitter": { "id": 27, "url": "https://patches.dpdk.org/api/people/27/?format=api", "name": "Stephen Hemminger", "email": "stephen@networkplumber.org" }, "delegate": { "id": 24651, "url": "https://patches.dpdk.org/api/users/24651/?format=api", "username": "dmarchand", "first_name": "David", "last_name": "Marchand", "email": "david.marchand@redhat.com" }, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20190924173953.25644-1-stephen@networkplumber.org/mbox/", "series": [ { "id": 6508, "url": "https://patches.dpdk.org/api/series/6508/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/list/?series=6508", "date": "2019-09-24T17:39:53", "name": "[v6] eal: make lcore_config private", "version": 6, "mbox": "https://patches.dpdk.org/series/6508/mbox/" } ], "comments": "https://patches.dpdk.org/api/patches/59686/comments/", "check": "fail", "checks": "https://patches.dpdk.org/api/patches/59686/checks/", "tags": {}, "related": [], "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 947D02C16;\n\tTue, 24 Sep 2019 19:40:01 +0200 (CEST)", "from mail-pf1-f196.google.com (mail-pf1-f196.google.com\n\t[209.85.210.196]) by dpdk.org (Postfix) with ESMTP id 9AE772C12\n\tfor <dev@dpdk.org>; Tue, 24 Sep 2019 19:39:59 +0200 (CEST)", "by mail-pf1-f196.google.com with SMTP id q12so1777234pff.9\n\tfor <dev@dpdk.org>; Tue, 24 Sep 2019 10:39:59 -0700 (PDT)", "from hermes.lan (204-195-22-127.wavecable.com. [204.195.22.127])\n\tby smtp.gmail.com with ESMTPSA id\n\ti74sm4671714pfe.28.2019.09.24.10.39.56\n\t(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n\tTue, 24 Sep 2019 10:39:56 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=networkplumber-org.20150623.gappssmtp.com; s=20150623;\n\th=from:to:cc:subject:date:message-id:mime-version\n\t:content-transfer-encoding;\n\tbh=vp1hd10LyJ8ffkYDNnTm0cNEzSb5tqYKs6+hRGVpl8M=;\n\tb=ybii5ZPi/HqQYKe1gnxa3hgW6gIUDEIYthbp/eEu9T+w5yMtWrw7+ci7K6RFRYPJNy\n\tAlont94dDSjAYaDjLuBbYSDZPyc/I1bDdrKXyI0CFuHV1kCWIVHWP2Z9/XXuNKBlOaLU\n\tycYtrrE6xjiwO9XPj44kRZftTRLVLrggU2OUGoIN/bP3GMhw6kI3ePntCJVcnk0SbWtD\n\til1nsH4+Pn3EMYjfa29s7mPi/6hMAklRKXJRwM250pcwZUplyHtbmZD5ULFgTHsBQZ6O\n\tVRtNa5k1xSwHfqOh9deDTxf6mvguGIlhnXb+UddBmFbPCL10zRuLUpKpMfx52gi/IkXM\n\t7XXw==", "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:mime-version\n\t:content-transfer-encoding;\n\tbh=vp1hd10LyJ8ffkYDNnTm0cNEzSb5tqYKs6+hRGVpl8M=;\n\tb=E9pA/+lttKyTIqIqf5CNsYIE2VPa2yLv4rNZIJrxwluTz/NI13eFW1lNImAxR19xMz\n\ttXNAm+mwx7xeTshYivU5tdLSkihW/LJEiCZ7UIfTsaUgfr4o+OXDm2eyes93Fh641ZFF\n\ttlOmFeTlfIPJq/F9f3y9uuMdNs1KsA7uNC9rw3RJCfcRRxo3zVHDlfD61POzpzPsV8G8\n\tPmdkbUu6bD+lNSng6TNteZR8knjACpiGHlFw8FlIcL3JaV9gWHesT0IhOgHvzoWgf1j6\n\tmZP1s/Cua4laifeMR6THRXznIUYqIa7DeGng2/x/Oh1WaohycoO0IHQFLgh9iO/Hzv7T\n\tWU+A==", "X-Gm-Message-State": "APjAAAU1WKhPaN4yRlHBEQfFPw9c6q9LXTA7NKnV5BBC31KF+fyhF3a2\n\tLeZRTDP1Q2o/N+tiReYMKjURKn0aKlk=", "X-Google-Smtp-Source": "APXvYqywckOlzkhkYprilOsmcsQjda9OitxSYAvgovSs91WevHFPeCd5OYhMYzqdPS8Q90KxZ9TVXQ==", "X-Received": "by 2002:a17:90a:8c14:: with SMTP id\n\ta20mr1320780pjo.45.1569346798210; \n\tTue, 24 Sep 2019 10:39:58 -0700 (PDT)", "From": "Stephen Hemminger <stephen@networkplumber.org>", "To": "dev@dpdk.org", "Cc": "Stephen Hemminger <stephen@networkplumber.org>", "Date": "Tue, 24 Sep 2019 10:39:53 -0700", "Message-Id": "<20190924173953.25644-1-stephen@networkplumber.org>", "X-Mailer": "git-send-email 2.20.1", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v6] eal: make lcore_config private", "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": "The internal structure of lcore_config is no longer be part of\nvisible API/ABI. Make it private to EAL.\n\nRearrange and resize the fields in the structure so it takes\nless memory (and cache footprint).\n\nSigned-off-by: Stephen Hemminger <stephen@networkplumber.org>\n---\nv6 - rebase for 19.11\n follows my earlier efforts at hiding lcore_config\n which were then taken up by David Marchand.\n\n lib/librte_eal/common/eal_common_launch.c | 2 ++\n lib/librte_eal/common/eal_private.h | 22 +++++++++++++++++++++\n lib/librte_eal/common/include/rte_lcore.h | 24 -----------------------\n lib/librte_eal/common/rte_service.c | 2 ++\n lib/librte_eal/rte_eal_version.map | 1 -\n 5 files changed, 26 insertions(+), 25 deletions(-)", "diff": "diff --git a/lib/librte_eal/common/eal_common_launch.c b/lib/librte_eal/common/eal_common_launch.c\nindex fe0ba3f0d617..cf52d717f68e 100644\n--- a/lib/librte_eal/common/eal_common_launch.c\n+++ b/lib/librte_eal/common/eal_common_launch.c\n@@ -15,6 +15,8 @@\n #include <rte_per_lcore.h>\n #include <rte_lcore.h>\n \n+#include \"eal_private.h\"\n+\n /*\n * Wait until a lcore finished its job.\n */\ndiff --git a/lib/librte_eal/common/eal_private.h b/lib/librte_eal/common/eal_private.h\nindex 798ede553b21..25e80547904f 100644\n--- a/lib/librte_eal/common/eal_private.h\n+++ b/lib/librte_eal/common/eal_private.h\n@@ -10,6 +10,28 @@\n #include <stdio.h>\n \n #include <rte_dev.h>\n+#include <rte_lcore.h>\n+\n+/**\n+ * Structure storing internal configuration (per-lcore)\n+ */\n+struct lcore_config {\n+\tuint32_t core_id; /**< core number on socket for this lcore */\n+\tuint32_t core_index; /**< relative index, starting from 0 */\n+\tuint16_t socket_id; /**< physical socket id for this lcore */\n+\tuint8_t core_role; /**< role of core eg: OFF, RTE, SERVICE */\n+\tuint8_t detected; /**< true if lcore was detected */\n+\tvolatile enum rte_lcore_state_t state; /**< lcore state */\n+\trte_cpuset_t cpuset; /**< cpu set which the lcore affinity to */\n+\tpthread_t thread_id; /**< pthread identifier */\n+\tint pipe_master2slave[2]; /**< communication pipe with master */\n+\tint pipe_slave2master[2]; /**< communication pipe with master */\n+\tlcore_function_t * volatile f; /**< function to call */\n+\tvoid * volatile arg; /**< argument of function */\n+\tvolatile int ret; /**< return value of function */\n+};\n+\n+extern struct lcore_config lcore_config[RTE_MAX_LCORE];\n \n /**\n * Initialize the memzone subsystem (private to eal).\ndiff --git a/lib/librte_eal/common/include/rte_lcore.h b/lib/librte_eal/common/include/rte_lcore.h\nindex c86f72eb12a8..0c683919564e 100644\n--- a/lib/librte_eal/common/include/rte_lcore.h\n+++ b/lib/librte_eal/common/include/rte_lcore.h\n@@ -66,30 +66,6 @@ typedef cpuset_t rte_cpuset_t;\n } while (0)\n #endif\n \n-/**\n- * Structure storing internal configuration (per-lcore)\n- */\n-struct lcore_config {\n-\tunsigned detected; /**< true if lcore was detected */\n-\tpthread_t thread_id; /**< pthread identifier */\n-\tint pipe_master2slave[2]; /**< communication pipe with master */\n-\tint pipe_slave2master[2]; /**< communication pipe with master */\n-\tlcore_function_t * volatile f; /**< function to call */\n-\tvoid * volatile arg; /**< argument of function */\n-\tvolatile int ret; /**< return value of function */\n-\tvolatile enum rte_lcore_state_t state; /**< lcore state */\n-\tunsigned socket_id; /**< physical socket id for this lcore */\n-\tunsigned core_id; /**< core number on socket for this lcore */\n-\tint core_index; /**< relative index, starting from 0 */\n-\trte_cpuset_t cpuset; /**< cpu set which the lcore affinity to */\n-\tuint8_t core_role; /**< role of core eg: OFF, RTE, SERVICE */\n-};\n-\n-/**\n- * Internal configuration (per-lcore)\n- */\n-extern struct lcore_config lcore_config[RTE_MAX_LCORE];\n-\n RTE_DECLARE_PER_LCORE(unsigned, _lcore_id); /**< Per thread \"lcore id\". */\n RTE_DECLARE_PER_LCORE(rte_cpuset_t, _cpuset); /**< Per thread \"cpuset\". */\n \ndiff --git a/lib/librte_eal/common/rte_service.c b/lib/librte_eal/common/rte_service.c\nindex c3653ebae46c..6e21f549051b 100644\n--- a/lib/librte_eal/common/rte_service.c\n+++ b/lib/librte_eal/common/rte_service.c\n@@ -21,6 +21,8 @@\n #include <rte_memory.h>\n #include <rte_malloc.h>\n \n+#include \"eal_private.h\"\n+\n #define RTE_SERVICE_NUM_MAX 64\n \n #define SERVICE_F_REGISTERED (1 << 0)\ndiff --git a/lib/librte_eal/rte_eal_version.map b/lib/librte_eal/rte_eal_version.map\nindex 7cbf82d37b0a..aeedf397764f 100644\n--- a/lib/librte_eal/rte_eal_version.map\n+++ b/lib/librte_eal/rte_eal_version.map\n@@ -4,7 +4,6 @@ DPDK_2.0 {\n \t__rte_panic;\n \teal_parse_sysfs_value;\n \teal_timer_source;\n-\tlcore_config;\n \tper_lcore__lcore_id;\n \tper_lcore__rte_errno;\n \trte_calloc;\n", "prefixes": [ "v6" ] }