Cover Detail
Show a cover letter.
GET /api/covers/52800/?format=api
{ "id": 52800, "url": "http://patches.dpdk.org/api/covers/52800/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/1555364488-28207-1-git-send-email-erik.g.carrillo@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": "<1555364488-28207-1-git-send-email-erik.g.carrillo@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1555364488-28207-1-git-send-email-erik.g.carrillo@intel.com", "date": "2019-04-15T21:41:26", "name": "[v5,0/2] Timer library changes", "submitter": { "id": 762, "url": "http://patches.dpdk.org/api/people/762/?format=api", "name": "Carrillo, Erik G", "email": "erik.g.carrillo@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/cover/1555364488-28207-1-git-send-email-erik.g.carrillo@intel.com/mbox/", "series": [ { "id": 4319, "url": "http://patches.dpdk.org/api/series/4319/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=4319", "date": "2019-04-15T21:41:26", "name": "Timer library changes", "version": 5, "mbox": "http://patches.dpdk.org/series/4319/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/52800/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 CA4871B39F;\n\tMon, 15 Apr 2019 23:42:36 +0200 (CEST)", "from mga02.intel.com (mga02.intel.com [134.134.136.20])\n\tby dpdk.org (Postfix) with ESMTP id F08141B39F\n\tfor <dev@dpdk.org>; Mon, 15 Apr 2019 23:42:34 +0200 (CEST)", "from orsmga002.jf.intel.com ([10.7.209.21])\n\tby orsmga101.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t15 Apr 2019 14:42:33 -0700", "from txasoft-yocto.an.intel.com ([10.123.72.192])\n\tby orsmga002.jf.intel.com with ESMTP; 15 Apr 2019 14:42:33 -0700" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.60,355,1549958400\"; d=\"scan'208\";a=\"151130530\"", "From": "Erik Gabriel Carrillo <erik.g.carrillo@intel.com>", "To": "rsanford@akamai.com,\n\tthomas@monjalon.net", "Cc": "dev@dpdk.org", "Date": "Mon, 15 Apr 2019 16:41:26 -0500", "Message-Id": "<1555364488-28207-1-git-send-email-erik.g.carrillo@intel.com>", "X-Mailer": "git-send-email 1.7.10", "In-Reply-To": "<1551892822-23967-1-git-send-email-erik.g.carrillo@intel.com>", "References": "<1551892822-23967-1-git-send-email-erik.g.carrillo@intel.com>", "Subject": "[dpdk-dev] [PATCH v5 0/2] Timer library changes", "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": "This patch series modifies the timer library in such a way that\nstructures that used to be statically allocated in a process's data\nsegment are now allocated in shared memory. As these structures contain\nlists of timers, new APIs are introduced that allow a caller to specify\nthe particular structure instance into which a timer should be inserted\nor from which a timer should be removed. This enables primary and\nsecondary processes to modify the same timer list, which enables some\nmulti-process use cases that were not previously possible; e.g. a\nsecondary process can start a timer whose expiration is detected in a\nprimary process running a new flavor of timer_manage().\n\nThe original library API is mostly unchanged, though implementations are\nupdated to call into newly added functions with a default structure\ninstance ID that provides the original behavior. New functions are\nintroduced to enable applications to allocate structure instances to\nhouse timer lists, and to reference them with an identifier when\nstarting and stopping timers, and finally, to manage the timer lists\nreferenced with an identifier.\n\nMy initial performance testing with the \"timer_perf_autotest\" test shows\nno performance regression or improvement, and inspection of the\ngenerated optimized code shows that the extra function call gets inlined\nin the functions that now have an extra function call. \n\nChanges in v5:\n - define default_data_id as const (Robert)\n - modify for-loop control in rte_timer_alt_manage and\n rte_timer_stop_all (Robert)\n - change parameter type in rte_timer_alt_manage_cb_t from \"void *\" to\n \"struct rte_timer *\" (Robert)\n\nChanges in v4:\n - Updated versioned symbols so that they correspond to the next\n release. Checked ABI compatibility again with validate-abi.sh.\n\nChanges in v3:\n - remove C++ style comment in first patch in series (Stephen)\n\nChanges in v2:\n - split these changes out into their own series\n - version the symbols where the existing ABI was updated, and\n provide alternate implementation with behavior equivalent to original\n behavior. Validated ABI compatibility with validate-abi.sh\n - refactor changes to simplify patches\n\nErik Gabriel Carrillo (2):\n timer: allow timer management in shared memory\n timer: add function to stop all timers in a list\n\n lib/librte_timer/Makefile | 1 +\n lib/librte_timer/rte_timer.c | 557 ++++++++++++++++++++++++++++++---\n lib/librte_timer/rte_timer.h | 258 ++++++++++++++-\n lib/librte_timer/rte_timer_version.map | 23 ++\n 4 files changed, 794 insertions(+), 45 deletions(-)" }