From: Neil Horman <nhorman@tuxdriver.com>
To: David Marchand <david.marchand@6wind.com>
Cc: dev@dpdk.org
Subject: Re: [dpdk-dev] [PATCH 0/7] fix build errors
Date: Mon, 18 Aug 2014 10:07:35 -0400 [thread overview]
Message-ID: <20140818140735.GA31958@hmsreliant.think-freely.org> (raw)
In-Reply-To: <1408353973-16663-1-git-send-email-david.marchand@6wind.com>
On Mon, Aug 18, 2014 at 11:26:06AM +0200, David Marchand wrote:
> 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.
>
> --
> 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
>
>
Agree with Bruce here, this is a nice consolidation.
Acked-by: Neil Horman <nhorman@tuxdriver.com>
prev parent reply other threads:[~2014-08-18 14:04 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-18 9:26 David Marchand
2014-08-18 9:26 ` [dpdk-dev] [PATCH 1/7] app: simplify makefiles David Marchand
2014-08-18 9:26 ` [dpdk-dev] [PATCH 2/7] app/test: use accessor to set refcnt field David Marchand
2014-08-18 9:26 ` [dpdk-dev] [PATCH 3/7] app/test: remove pm_autotest David Marchand
2014-08-18 9:26 ` [dpdk-dev] [PATCH 4/7] app/test: introduce dynamic commands list David Marchand
2014-08-18 9:26 ` [dpdk-dev] [PATCH 5/7] app/test: convert all tests to register system David Marchand
2014-08-18 9:26 ` [dpdk-dev] [PATCH 6/7] app/test: no more builtin commands David Marchand
2014-08-18 9:26 ` [dpdk-dev] [PATCH 7/7] app/test: only build what has been selected in config David Marchand
2014-08-18 10:46 ` [dpdk-dev] [PATCH 0/7] fix build errors David Marchand
2014-08-18 14:07 ` Neil Horman [this message]
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=20140818140735.GA31958@hmsreliant.think-freely.org \
--to=nhorman@tuxdriver.com \
--cc=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).