From: Erik Gabriel Carrillo <erik.g.carrillo@intel.com>
To: jerin.jacob@caviumnetworks.com
Cc: dev@dpdk.org, bruce.richardson@intel.com,
harry.van.haaren@intel.com, hemant.agrawal@nxp.com,
gage.ads@intel.com, nipun.gupta@nxp.com,
narender.vangati@intel.com, nikhil.rao@intel.com,
pbhagavatula@caviumnetworks.com, rsanford@akamai.com
Subject: [dpdk-dev] [RFC PATCH v2 0/1] eventtimer: introduce event timer adapter
Date: Fri, 22 Sep 2017 10:17:10 -0500 [thread overview]
Message-ID: <1506093431-57588-1-git-send-email-erik.g.carrillo@intel.com> (raw)
In-Reply-To: <20170817161104.24293-1-jerin.jacob@caviumnetworks.com>
Hi Jerin, et al.,
The following RFC patch contains enough of a skeleton implementation
for an event timer adapter to show how the API could call through an ops
structure to a plugin backend to provide different implementations.
In terms of differences from the original RFC, the most obvious one is
that I've replaced occurrences of "wheel" in the API with "adapter",
partly to reflect the similarity with the event_eth_rx_adapter, and
partly because "wheel" suggests an implementation that may not be
the one used (as in the case of the SW impl).
The second big change is to replace API parameters specifying pointer
to rte_event_timer_adapter with ids, which seems more common throughout
DPDK.
Other changes include:
- removed rte_event_timer_adapter_lookup() function, since APIs no longer
accept pointer to rte_event_timer_adapter
- Replaced RTE_EVENT_TIMER_SUCCESS_{ARM,CANCEL} states with
RTE_EVENT_TIMER_ARMED, since the purpose of the SUCCESS_CANCEL state was
unclear
Please have a look and let me know what you think.
Thanks,
Gabriel
Erik Gabriel Carrillo (1):
eventtimer: introduce event timer adapter
drivers/event/sw/Makefile | 1 +
drivers/event/sw/sw_evdev.c | 11 +
drivers/event/sw/sw_evdev_timer_adapter.c | 124 +++++++
lib/librte_eventdev/Makefile | 3 +
lib/librte_eventdev/rte_event_timer_adapter.c | 227 ++++++++++++
lib/librte_eventdev/rte_event_timer_adapter.h | 388 +++++++++++++++++++++
.../rte_event_timer_adapter_driver.h | 128 +++++++
lib/librte_eventdev/rte_eventdev.h | 2 +-
lib/librte_eventdev/rte_eventdev_pmd.h | 15 +
lib/librte_eventdev/rte_eventdev_version.map | 14 +
test/test/Makefile | 1 +
test/test/test_event_timer_adapter.c | 183 ++++++++++
12 files changed, 1096 insertions(+), 1 deletion(-)
create mode 100644 drivers/event/sw/sw_evdev_timer_adapter.c
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_driver.h
create mode 100644 test/test/test_event_timer_adapter.c
--
2.6.4
next prev parent reply other threads:[~2017-09-22 15:15 UTC|newest]
Thread overview: 33+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-17 16:11 [dpdk-dev] [RFC PATCH 0/1] eventtimer: introduce event timer wheel Jerin Jacob
2017-08-17 16:11 ` [dpdk-dev] [RFC PATCH 1/1] " Jerin Jacob
2017-08-23 22:57 ` [dpdk-dev] [RFC PATCH 0/1] " Carrillo, Erik G
2017-08-25 10:25 ` Jerin Jacob
2017-08-29 15:02 ` Thomas Monjalon
2017-08-29 15:41 ` Jerin Jacob
2017-08-29 15:48 ` Thomas Monjalon
2017-08-29 16:07 ` Jerin Jacob
2017-09-22 15:17 ` Erik Gabriel Carrillo [this message]
2017-09-22 15:17 ` [dpdk-dev] [RFC PATCH v2 1/1] eventtimer: introduce event timer adapter Erik Gabriel Carrillo
2017-10-03 14:37 ` [dpdk-dev] [RFC PATCH v2 0/1] " Jerin Jacob
2017-10-09 20:30 ` Carrillo, Erik G
2017-10-16 12:04 ` Pavan Nikhilesh Bhagavatula
2017-10-16 12:37 ` Pavan Nikhilesh Bhagavatula
2017-10-18 21:48 ` Carrillo, Erik G
2017-10-26 15:45 ` Pavan Nikhilesh Bhagavatula
2017-11-20 22:35 ` [dpdk-dev] [RFC PATCH v3 " Erik Gabriel Carrillo
2017-11-20 22:35 ` [dpdk-dev] [RFC PATCH v3 1/1] " Erik Gabriel Carrillo
2017-11-23 4:37 ` Pavan Nikhilesh Bhagavatula
2017-11-27 14:47 ` Carrillo, Erik G
2017-11-28 17:40 ` [dpdk-dev] [RFC PATCH v4 0/4] " Erik Gabriel Carrillo
2017-11-28 17:40 ` [dpdk-dev] [RFC PATCH v4 1/4] " Erik Gabriel Carrillo
2017-11-29 10:29 ` Pavan Nikhilesh Bhagavatula
2017-11-28 17:40 ` [dpdk-dev] [RFC PATCH v4 2/4] eventtimer: add common code Erik Gabriel Carrillo
2017-11-29 5:19 ` Pavan Nikhilesh Bhagavatula
2017-11-30 20:59 ` Carrillo, Erik G
2017-12-01 5:13 ` Pavan Nikhilesh Bhagavatula
2017-12-01 20:19 ` Carrillo, Erik G
2017-11-28 17:40 ` [dpdk-dev] [RFC PATCH v4 3/4] eventtimer: add default software implementation stub Erik Gabriel Carrillo
2017-11-29 10:34 ` Pavan Nikhilesh Bhagavatula
2017-11-30 23:56 ` Carrillo, Erik G
2017-12-01 5:15 ` Pavan Nikhilesh Bhagavatula
2017-11-28 17:40 ` [dpdk-dev] [RFC PATCH v4 4/4] test: add event timer adapter auto-test Erik Gabriel Carrillo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1506093431-57588-1-git-send-email-erik.g.carrillo@intel.com \
--to=erik.g.carrillo@intel.com \
--cc=bruce.richardson@intel.com \
--cc=dev@dpdk.org \
--cc=gage.ads@intel.com \
--cc=harry.van.haaren@intel.com \
--cc=hemant.agrawal@nxp.com \
--cc=jerin.jacob@caviumnetworks.com \
--cc=narender.vangati@intel.com \
--cc=nikhil.rao@intel.com \
--cc=nipun.gupta@nxp.com \
--cc=pbhagavatula@caviumnetworks.com \
--cc=rsanford@akamai.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).