From: "Liang, Ma" <liang.j.ma@intel.com>
To: Harry van Haaren <harry.van.haaren@intel.com>
Cc: dev@dpdk.org, peter.mccarthy@intel.com
Subject: Re: [dpdk-dev] [PATCH] event/opdl: rework loops to comply with dpdk style
Date: Mon, 22 Jan 2018 10:30:01 +0000 [thread overview]
Message-ID: <20180122103001.GA9466@sivswdev01.ir.intel.com> (raw)
In-Reply-To: <1516615450-57525-1-git-send-email-harry.van.haaren@intel.com>
On 22 Jan 10:04, Harry van Haaren wrote:
> This commit reworks the loop counter variable declarations
> to be in line with the DPDK source code.
>
> Fixes: 3c7f3dcfb099 ("event/opdl: add PMD main body and helper function")
> Fixes: 8ca8e3b48eff ("event/opdl: add event queue config get/set")
> Fixes: d548ef513cd7 ("event/opdl: add unit tests")
>
> Cc: liang.j.ma@intel.com
> Cc: peter.mccarthy@intel.com
>
> Signed-off-by: Harry van Haaren <harry.van.haaren@intel.com>
>
> ---
>
> Compile tested for loop declarations by adding "-std=gnu90" to the
> OPDL Makefile CFLAGS variable, which now passes here. Note that DPDK
> as a whole does not compile with gnu90, so don't pass it as EXTRA_CFLAGS
> ---
> drivers/event/opdl/Makefile | 2 --
> drivers/event/opdl/opdl_evdev.c | 9 +++++----
> drivers/event/opdl/opdl_evdev_init.c | 23 ++++++++++++++---------
> drivers/event/opdl/opdl_evdev_xstats.c | 9 ++++++---
> drivers/event/opdl/opdl_test.c | 6 ++++--
> 5 files changed, 29 insertions(+), 20 deletions(-)
>
> diff --git a/drivers/event/opdl/Makefile b/drivers/event/opdl/Makefile
> index a8aff2c..747ae5b 100644
> --- a/drivers/event/opdl/Makefile
> +++ b/drivers/event/opdl/Makefile
> @@ -7,8 +7,6 @@ include $(RTE_SDK)/mk/rte.vars.mk
> LIB = librte_pmd_opdl_event.a
>
> # build flags
> -CFLAGS += -std=c99
> -CFLAGS += -D_XOPEN_SOURCE=600
> CFLAGS += -O3
> CFLAGS += $(WERROR_FLAGS)
> # for older GCC versions, allow us to initialize an event using
> diff --git a/drivers/event/opdl/opdl_evdev.c b/drivers/event/opdl/opdl_evdev.c
> index dcbf404..d5e2b60 100644
> --- a/drivers/event/opdl/opdl_evdev.c
> +++ b/drivers/event/opdl/opdl_evdev.c
> @@ -288,7 +288,8 @@ opdl_queue_setup(struct rte_eventdev *dev,
> }
> }
> /* Check if queue id has been setup already */
> - for (uint32_t i = 0; i < device->nb_q_md; i++) {
> + uint32_t i;
> + for (i = 0; i < device->nb_q_md; i++) {
> if (device->q_md[i].ext_id == queue_id) {
> PMD_DRV_LOG(ERR, "DEV_ID:[%02d] : "
> "queue id %u already setup\n",
> @@ -390,8 +391,8 @@ opdl_dump(struct rte_eventdev *dev, FILE *f)
>
> fprintf(f,
> "\n\n -- RING STATISTICS --\n");
> -
> - for (uint32_t i = 0; i < device->nb_opdls; i++)
> + uint32_t i;
> + for (i = 0; i < device->nb_opdls; i++)
> opdl_ring_dump(device->opdl[i], f);
>
> fprintf(f,
> @@ -400,7 +401,7 @@ opdl_dump(struct rte_eventdev *dev, FILE *f)
> "Av. Grant Size Av. Cycles PP"
> " Empty DEQs Non Empty DEQs Pkts Processed\n");
>
> - for (uint32_t i = 0; i < device->max_port_nb; i++) {
> + for (i = 0; i < device->max_port_nb; i++) {
> char queue_id[64];
> char total_cyc[64];
> const char *p_type;
> diff --git a/drivers/event/opdl/opdl_evdev_init.c b/drivers/event/opdl/opdl_evdev_init.c
> index c37d8bc..84ab258 100644
> --- a/drivers/event/opdl/opdl_evdev_init.c
> +++ b/drivers/event/opdl/opdl_evdev_init.c
> @@ -314,8 +314,8 @@ static int opdl_add_deps(struct opdl_evdev *device,
> "Stages and dependents"
> " are not for same opdl ring",
> opdl_pmd_dev_id(device));
> - for (uint32_t k = 0;
> - k < device->nb_opdls; k++) {
> + uint32_t k;
> + for (k = 0; k < device->nb_opdls; k++) {
> opdl_ring_dump(device->opdl[k],
> stdout);
> }
> @@ -505,8 +505,9 @@ void
> destroy_queues_and_rings(struct rte_eventdev *dev)
> {
> struct opdl_evdev *device = opdl_pmd_priv(dev);
> + uint32_t i;
>
> - for (uint32_t i = 0; i < device->nb_opdls; i++) {
> + for (i = 0; i < device->nb_opdls; i++) {
> if (device->opdl[i])
> opdl_ring_free(device->opdl[i]);
> }
> @@ -639,7 +640,8 @@ create_queues_and_rings(struct rte_eventdev *dev)
> OPDL_Q_POS_START,
> -1);
>
> - for (uint32_t i = 0; i < device->nb_q_md; i++) {
> + uint32_t i;
> + for (i = 0; i < device->nb_q_md; i++) {
>
> /* Check */
> if (!device->q_md[i].setup) {
> @@ -702,7 +704,8 @@ initialise_all_other_ports(struct rte_eventdev *dev)
>
> struct opdl_evdev *device = opdl_pmd_priv(dev);
>
> - for (uint32_t i = 0; i < device->nb_ports; i++) {
> + uint32_t i;
> + for (i = 0; i < device->nb_ports; i++) {
> struct opdl_port *port = &device->ports[i];
> struct opdl_queue *queue = &device->queue[port->queue_id];
>
> @@ -827,7 +830,7 @@ initialise_all_other_ports(struct rte_eventdev *dev)
> * setup the last bit of stage md
> */
> if (!err) {
> - for (uint32_t i = 0; i < device->nb_ports; i++) {
> + for (i = 0; i < device->nb_ports; i++) {
> struct opdl_port *port = &device->ports[i];
> struct opdl_queue *queue =
> &device->queue[port->queue_id];
> @@ -872,7 +875,8 @@ initialise_queue_zero_ports(struct rte_eventdev *dev)
> struct opdl_evdev *device = opdl_pmd_priv(dev);
>
> /* Assign queue zero and figure out how many Q0 ports we have */
> - for (uint32_t i = 0; i < device->nb_ports; i++) {
> + uint32_t i;
> + for (i = 0; i < device->nb_ports; i++) {
> struct opdl_port *port = &device->ports[i];
> if (port->queue_id == OPDL_INVALID_QID) {
> port->queue_id = 0;
> @@ -889,7 +893,7 @@ initialise_queue_zero_ports(struct rte_eventdev *dev)
> if (stage_inst) {
>
> /* Assign the new created input stage to all relevant ports */
> - for (uint32_t i = 0; i < device->nb_ports; i++) {
> + for (i = 0; i < device->nb_ports; i++) {
> struct opdl_port *port = &device->ports[i];
> if (port->queue_id == 0) {
> queue = &device->queue[port->queue_id];
> @@ -914,8 +918,9 @@ assign_internal_queue_ids(struct rte_eventdev *dev)
> {
> int err = 0;
> struct opdl_evdev *device = opdl_pmd_priv(dev);
> + uint32_t i;
>
> - for (uint32_t i = 0; i < device->nb_ports; i++) {
> + for (i = 0; i < device->nb_ports; i++) {
> struct opdl_port *port = &device->ports[i];
> if (port->external_qid != OPDL_INVALID_QID) {
> port->queue_id =
> diff --git a/drivers/event/opdl/opdl_evdev_xstats.c b/drivers/event/opdl/opdl_evdev_xstats.c
> index 94dfeee..70b84d9 100644
> --- a/drivers/event/opdl/opdl_evdev_xstats.c
> +++ b/drivers/event/opdl/opdl_evdev_xstats.c
> @@ -86,7 +86,8 @@ opdl_xstats_get_names(const struct rte_eventdev *dev,
>
> uint32_t port_idx = queue_port_id * max_num_port_xstat;
>
> - for (uint32_t j = 0; j < max_num_port_xstat; j++) {
> + uint32_t j;
> + for (j = 0; j < max_num_port_xstat; j++) {
>
> strcpy(xstats_names[j].name,
> device->port_xstat[j + port_idx].stat.name);
> @@ -121,7 +122,8 @@ opdl_xstats_get(const struct rte_eventdev *dev,
> uint32_t p_start = queue_port_id * max_num_port_xstat;
> uint32_t p_finish = p_start + max_num_port_xstat;
>
> - for (uint32_t i = 0; i < n; i++) {
> + uint32_t i;
> + for (i = 0; i < n; i++) {
> if (ids[i] < p_start || ids[i] >= p_finish)
> return -EINVAL;
>
> @@ -142,7 +144,8 @@ opdl_xstats_get_by_name(const struct rte_eventdev *dev,
>
> uint32_t max_index = device->max_port_nb * max_num_port_xstat;
>
> - for (uint32_t i = 0; i < max_index; i++) {
> + uint32_t i;
> + for (i = 0; i < max_index; i++) {
>
> if (strncmp(name,
> device->port_xstat[i].stat.name,
> diff --git a/drivers/event/opdl/opdl_test.c b/drivers/event/opdl/opdl_test.c
> index 44a5cc5..13d4f6b 100644
> --- a/drivers/event/opdl/opdl_test.c
> +++ b/drivers/event/opdl/opdl_test.c
> @@ -433,7 +433,8 @@ atomic_basic(struct test *t)
> return -1;
> }
>
> - for (int j = 0; j < 3; j++) {
> + int j;
> + for (j = 0; j < 3; j++) {
> deq_ev[j].op = RTE_EVENT_OP_FORWARD;
> deq_ev[j].queue_id = t->qid[1];
> }
> @@ -495,8 +496,9 @@ static int
> check_statistics(void)
> {
> int num_ports = 3; /* Hard-coded for this app */
> + int i;
>
> - for (int i = 0; i < num_ports; i++) {
> + for (i = 0; i < num_ports; i++) {
> int num_stats, num_stats_returned;
>
> num_stats = rte_event_dev_xstats_names_get(0,
> --
> 2.7.4
>
Many thanks Harry.
Acked-by: Liang Ma <liang.j.ma@intel.com>
next prev parent reply other threads:[~2018-01-22 10:30 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-01-22 10:04 Harry van Haaren
2018-01-22 10:30 ` Liang, Ma [this message]
2018-01-24 17:56 ` Thomas Monjalon
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=20180122103001.GA9466@sivswdev01.ir.intel.com \
--to=liang.j.ma@intel.com \
--cc=dev@dpdk.org \
--cc=harry.van.haaren@intel.com \
--cc=peter.mccarthy@intel.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).