DPDK patches and discussions
 help / color / mirror / Atom feed
From: David Marchand <david.marchand@6wind.com>
To: dev@dpdk.org
Subject: [dpdk-dev] [PATCH v2 0/7] fix build errors
Date: Mon, 18 Aug 2014 13:29:18 +0200	[thread overview]
Message-ID: <1408361365-12988-1-git-send-email-david.marchand@6wind.com> (raw)

This patch series resolves build errors I encountered after disabling some parts
of the dpdk.

- simplified app/*/Makefile,
- fixed some build error when disabling refcnt,
- removed dead code (reference to pm_autotest),
- reworked all the tests from app/test to use a dynamic register mechanism
  (this makes it easier to enable/disable them),
- reworked app/test Makefile to only enable the relevant tests.

Changes since v1:
- fixed typo in first patch (RTE_CONFIG.. in app/test/Makefile)

-- 
David Marchand

David Marchand (7):
  app: simplify makefiles
  app/test: use accessor to set refcnt field
  app/test: remove pm_autotest
  app/test: introduce dynamic commands list
  app/test: convert all tests to register system
  app/test: no more builtin commands
  app/test: only build what has been selected in config

 app/cmdline_test/Makefile        |    8 +-
 app/dump_cfg/Makefile            |    5 +-
 app/test-acl/Makefile            |    9 +-
 app/test-pipeline/Makefile       |   26 ++---
 app/test-pmd/Makefile            |   40 ++++----
 app/test/Makefile                |  145 ++++++++++++++++------------
 app/test/autotest_data.py        |    6 --
 app/test/commands.c              |  193 ++++++++------------------------------
 app/test/test.c                  |    5 +
 app/test/test.h                  |   74 +++++----------
 app/test/test_acl.c              |   18 ++--
 app/test/test_alarm.c            |    7 +-
 app/test/test_atomic.c           |    7 +-
 app/test/test_byteorder.c        |    8 +-
 app/test/test_cmdline.c          |   11 ++-
 app/test/test_common.c           |    8 +-
 app/test/test_cpuflags.c         |    8 +-
 app/test/test_cycles.c           |    8 +-
 app/test/test_debug.c            |    8 +-
 app/test/test_devargs.c          |    8 +-
 app/test/test_distributor.c      |   22 ++---
 app/test/test_distributor_perf.c |   20 ++--
 app/test/test_eal_flags.c        |   20 ++--
 app/test/test_eal_fs.c           |   17 ++--
 app/test/test_errno.c            |    8 +-
 app/test/test_func_reentrancy.c  |    8 +-
 app/test/test_hash.c             |   19 ++--
 app/test/test_hash_perf.c        |   19 ++--
 app/test/test_interrupts.c       |    7 +-
 app/test/test_ivshmem.c          |   15 +--
 app/test/test_kni.c              |   18 ++--
 app/test/test_kvargs.c           |    9 +-
 app/test/test_link_bonding.c     |    8 +-
 app/test/test_logs.c             |    8 +-
 app/test/test_lpm.c              |   19 ++--
 app/test/test_lpm6.c             |   19 ++--
 app/test/test_malloc.c           |    8 +-
 app/test/test_mbuf.c             |    8 +-
 app/test/test_memcpy.c           |    8 +-
 app/test/test_memcpy_perf.c      |    8 +-
 app/test/test_memory.c           |    8 +-
 app/test/test_mempool.c          |    8 +-
 app/test/test_mempool_perf.c     |    8 +-
 app/test/test_memzone.c          |    8 +-
 app/test/test_meter.c            |   18 ++--
 app/test/test_mp_secondary.c     |   19 +---
 app/test/test_pci.c              |    6 ++
 app/test/test_per_lcore.c        |    8 +-
 app/test/test_pmd_ring.c         |   19 ++--
 app/test/test_power.c            |   20 ++--
 app/test/test_prefetch.c         |    8 +-
 app/test/test_red.c              |   20 ++--
 app/test/test_ring.c             |    8 +-
 app/test/test_ring_perf.c        |    8 +-
 app/test/test_rwlock.c           |    8 +-
 app/test/test_sched.c            |   18 ++--
 app/test/test_spinlock.c         |    8 +-
 app/test/test_string_fns.c       |    8 +-
 app/test/test_table.c            |   21 ++---
 app/test/test_table.h            |    1 -
 app/test/test_table_acl.c        |    4 -
 app/test/test_table_combined.c   |    3 -
 app/test/test_table_pipeline.c   |    8 --
 app/test/test_table_ports.c      |    4 -
 app/test/test_table_tables.c     |    4 -
 app/test/test_tailq.c            |    8 +-
 app/test/test_timer.c            |   17 ++--
 app/test/test_timer_perf.c       |   17 ++--
 app/test/test_version.c          |    8 +-
 69 files changed, 550 insertions(+), 623 deletions(-)

-- 
1.7.10.4

             reply	other threads:[~2014-08-18 11:26 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-18 11:29 David Marchand [this message]
2014-08-18 11:29 ` [dpdk-dev] [PATCH v2 1/7] app: simplify makefiles David Marchand
2014-08-18 11:29 ` [dpdk-dev] [PATCH v2 2/7] app/test: use accessor to set refcnt field David Marchand
2014-08-18 11:29 ` [dpdk-dev] [PATCH v2 3/7] app/test: remove pm_autotest David Marchand
2014-08-18 11:29 ` [dpdk-dev] [PATCH v2 4/7] app/test: introduce dynamic commands list David Marchand
2014-08-18 11:29 ` [dpdk-dev] [PATCH v2 5/7] app/test: convert all tests to register system David Marchand
2014-08-18 11:29 ` [dpdk-dev] [PATCH v2 6/7] app/test: no more builtin commands David Marchand
2014-08-18 11:29 ` [dpdk-dev] [PATCH v2 7/7] app/test: only build what has been selected in config David Marchand
2014-08-18 12:57 ` [dpdk-dev] [PATCH v2 0/7] fix build errors Bruce Richardson
2014-08-26 15:58   ` 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=1408361365-12988-1-git-send-email-david.marchand@6wind.com \
    --to=david.marchand@6wind.com \
    --cc=dev@dpdk.org \
    /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).