From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga07.intel.com (mga07.intel.com [134.134.136.100]) by dpdk.org (Postfix) with ESMTP id 2E53B2BA5 for ; Thu, 8 Mar 2018 22:54:23 +0100 (CET) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by orsmga105.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 08 Mar 2018 13:54:19 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.47,442,1515484800"; d="scan'208";a="40388871" Received: from txasoft-yocto.an.intel.com (HELO txasoft-yocto.an.intel.com.) ([10.123.72.111]) by orsmga002.jf.intel.com with ESMTP; 08 Mar 2018 13:54:19 -0800 From: Erik Gabriel Carrillo To: pbhagavatula@caviumnetworks.com Cc: dev@dpdk.org, jerin.jacob@caviumnetworks.com, nipun.gupta@nxp.com, hemant.agrawal@nxp.com Date: Thu, 8 Mar 2018 15:53:59 -0600 Message-Id: <1520546046-6973-1-git-send-email-erik.g.carrillo@intel.com> X-Mailer: git-send-email 1.7.10 In-Reply-To: <1515630074-29020-1-git-send-email-erik.g.carrillo@intel.com> References: <1515630074-29020-1-git-send-email-erik.g.carrillo@intel.com> Subject: [dpdk-dev] [PATCH v7 0/7] eventtimer: introduce event timer adapter X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 08 Mar 2018 21:54:24 -0000 This patch series contains the next iteration of the Event Timer Adapter library, which abstracts timer mechanisms that are tightly coupled with event devices, and extends the event based programming model so that timer expirations are represented as an event. v7 - Addressed comments on previous patch series from Pavan: - Use SPDX license tags - Squash various commits to make series easier to review - Tag experimental functions as such - Use one mempool for messages and timers in sw driver - Limit service cores mapped to sw driver's service to one - Use smp memory barriers - In service function, invoke rte_timer_manage() with frequency matching the resolution the adapter was configured with - Reworked synchronization in sw driver between threads producing messages and service thread that consumes them. The new approach avoids a situation where event timers couldn't be armed/canceled from the same lcore the service was running on. - Updated logging facility - Added more unit tests - Added support for meson build v6 - Addressed comments on previous version from Jerin: - Added RTE_EVENT_TIMER_CANCELED event timer state back in - remove check for started adapter in timer arm/cancel functions - reuse CONFIG_RTE_LIBRTE_EVENTDEV_DEBUG instead of adding new config option - Added initial version of software driver implementation - Added stats APIs - Added API to retrieve adapter service ID - Added API to initialize event timer - Added entry to Programmer's Guide in documentation - Added new unit tests to auto-test v5 - Addressed comments on previous version from Pavan: - renamed rte_event_timer_adapter_driver.h to rte_event_timer_adapter_pmd.h - moved contents of sw_event_timer_adapter.c into rte_event_timer_adapter.c - added flags parameter to timer_adapter_caps_get() call - added DEBUG config variable to conditionally compile run-time checks on datapath - fixed license text and file description - Also added a config variable to enable/disable compilation of event timer adapter - feedback on whether this is desirable is appreciated v4 - Split changes into multiple patches for easier review v3 - Reworked allocation and ops organization in common code based on feedback received from Jerin and Pavan. This will allow fast-path function pointers to be dereferenced with one level of indirection with pointers valid in primary and secondary processes. - Moved default software implementation from sw_evdev directory to eventdev library directory, which will allow it to be used by any eventdev PMD as an alternative to providing its own definitions. - Reverted occurrences of id back to pointer to adapter struct in library API - Added rte_event_timer_adapter_lookup() function back in v2 - Added ops structure and stubbed out plugin for SW impl - Added unit test stubs - Replaced occurrences of "wheel" in API with "adapter" - Replaced occurrences of pointer to struct rte_event_timer_adapter with ids - Removed rte_event_timer_adapter_lookup() function - Replaced RTE_EVENT_TIMER_SUCCESS_{ARM,CANCEL} states with RTE_EVENT_TIMER_ARMED Erik Gabriel Carrillo (7): eventtimer: add event timer adapter API eventtimer: add common code eventtimer: add default software driver eventtimer: add support for meson build system test: add event timer adapter auto-test doc: add event timer adapter section to programmer's guide doc: add event timer adapter documentation MAINTAINERS | 7 + config/common_base | 1 + config/rte_config.h | 1 + doc/api/doxy-api-index.md | 32 +- doc/guides/prog_guide/event_timer_adapter.rst | 277 +++++ doc/guides/prog_guide/index.rst | 1 + doc/guides/rel_notes/release_18_05.rst | 5 + drivers/event/sw/sw_evdev.c | 18 + lib/Makefile | 2 +- lib/librte_eventdev/Makefile | 5 +- lib/librte_eventdev/meson.build | 9 +- lib/librte_eventdev/rte_event_timer_adapter.c | 1333 +++++++++++++++++++++ lib/librte_eventdev/rte_event_timer_adapter.h | 700 +++++++++++ lib/librte_eventdev/rte_event_timer_adapter_pmd.h | 150 +++ lib/librte_eventdev/rte_eventdev.h | 44 +- lib/librte_eventdev/rte_eventdev_pmd.h | 35 + lib/librte_eventdev/rte_eventdev_version.map | 20 + lib/meson.build | 3 +- mk/rte.app.mk | 2 +- test/test/Makefile | 1 + test/test/test_event_timer_adapter.c | 1234 +++++++++++++++++++ 21 files changed, 3810 insertions(+), 70 deletions(-) create mode 100644 doc/guides/prog_guide/event_timer_adapter.rst create mode 100644 lib/librte_eventdev/rte_event_timer_adapter.c create mode 100644 lib/librte_eventdev/rte_event_timer_adapter.h create mode 100644 lib/librte_eventdev/rte_event_timer_adapter_pmd.h create mode 100644 test/test/test_event_timer_adapter.c -- 2.6.4