Cover Detail
Show a cover letter.
GET /api/covers/50987/?format=api
{ "id": 50987, "url": "http://patches.dpdk.org/api/covers/50987/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/1552031797-146710-1-git-send-email-gavin.hu@arm.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": "<1552031797-146710-1-git-send-email-gavin.hu@arm.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1552031797-146710-1-git-send-email-gavin.hu@arm.com", "date": "2019-03-08T07:56:34", "name": "[v8,0/3] generic spinlock optimization and test case enhancements", "submitter": { "id": 1018, "url": "http://patches.dpdk.org/api/people/1018/?format=api", "name": "Gavin Hu", "email": "gavin.hu@arm.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/cover/1552031797-146710-1-git-send-email-gavin.hu@arm.com/mbox/", "series": [ { "id": 3676, "url": "http://patches.dpdk.org/api/series/3676/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=3676", "date": "2019-03-08T07:56:34", "name": "generic spinlock optimization and test case enhancements", "version": 8, "mbox": "http://patches.dpdk.org/series/3676/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/50987/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 2B186A3;\n\tFri, 8 Mar 2019 08:57:02 +0100 (CET)", "from foss.arm.com (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70])\n\tby dpdk.org (Postfix) with ESMTP id 5047127D\n\tfor <dev@dpdk.org>; Fri, 8 Mar 2019 08:57:00 +0100 (CET)", "from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249])\n\tby usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 965DEA78;\n\tThu, 7 Mar 2019 23:56:59 -0800 (PST)", "from net-arm-thunderx2.shanghai.arm.com\n\t(net-arm-thunderx2.shanghai.arm.com [10.169.40.121])\n\tby usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id\n\tE1AE93F71D; Thu, 7 Mar 2019 23:56:57 -0800 (PST)" ], "From": "Gavin Hu <gavin.hu@arm.com>", "To": "dev@dpdk.org", "Cc": "nd@arm.com, thomas@monjalon.net, jerinj@marvell.com,\n\themant.agrawal@nxp.com, nipun.gupta@nxp.com,\n\tHonnappa.Nagarahalli@arm.com, \n\tgavin.hu@arm.com, i.maximets@samsung.com, chaozhu@linux.vnet.ibm.com", "Date": "Fri, 8 Mar 2019 15:56:34 +0800", "Message-Id": "<1552031797-146710-1-git-send-email-gavin.hu@arm.com>", "X-Mailer": "git-send-email 2.7.4", "In-Reply-To": "<20181220104246.5590-1-gavin.hu@arm.com>", "References": "<20181220104246.5590-1-gavin.hu@arm.com>", "Subject": "[dpdk-dev] [PATCH v8 0/3] generic spinlock optimization and test\n\tcase enhancements", "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": "V8: Remove internal ChangeId\n\nV7: Update the 1/3 patch headline and commit message\n\nV6: Rebase and drop the first patch as a similar fix was already merged.\n\nV5: Remove ChangeId(sorry for that)\n\nV4:\n1. Drop one patch for the test case to get time precisely as the overhead\n of getting time is amortized already in another patch.\n2. Drop the ticket lock patch from this series as there are no dependency\n between them, the ticket lock patch was submitted separately: \n http://patchwork.dpdk.org/patch/49770/\n3. Define volatile variable in patch #3 to be more realistic for spinlock\n protection(avoid optimization be compiler).\n4. Fix typos.\n\nV3:\n1. Implemented the ticket lock to improve the fairness and predictability.\n The locks are obtained in the order of requested.\n\nV2:\n1. FORCE_INTRINCIS is still an option for ppc/x86, although not is use\n by default, so don't remove it from generic file.\n2. Fix the clang compiler error on x86 when the above FORCE_INTRINSICS\n is enabled.\n\nV1:\n1. Remove the 1us delay outside of the locked region to really benchmark\n the spinlock acquire/release performance, not the delay API.\n2. Use the precise version of getting timestamps for more precise\n benchmarking results.\n3. Amortize the overhead of getting the timestamp by 10000 loops.\n4. Move the arm specific implementation to arm folder to remove the\n hardcoded implementation.\n5. Use atomic primitives, which translate to one-way barriers, instead of\n two-way sync primitives, to optimize for performance.\n\nGavin Hu (3):\n test/spinlock: remove 1us delay for correct benchmarking\n test/spinlock: amortize the cost of getting time\n spinlock: reimplement with atomic one-way barrier builtins\n\n app/test/test_spinlock.c | 31 +++++++++++-----------\n .../common/include/generic/rte_spinlock.h | 18 +++++++++----\n 2 files changed, 29 insertions(+), 20 deletions(-)" }