DPDK patches and discussions
 help / color / mirror / Atom feed
From: David Marchand <david.marchand@6wind.com>
To: "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH 0/7] fix build errors
Date: Mon, 18 Aug 2014 12:46:44 +0200	[thread overview]
Message-ID: <CALwxeUvFHGvSjGfsKJ9vnbP=6g81HRErgV1zioPr+1WUes47Wg@mail.gmail.com> (raw)
In-Reply-To: <1408353973-16663-1-git-send-email-david.marchand@6wind.com>

self-nack on the first patch ...
There is a typo in app/test/Makefile (RTE_CONFIG.. vs CONFIG_RTE..) that I
introduced while rebasing...

I will send a v2 shortly.

-- 
David Marchand



On Mon, Aug 18, 2014 at 11:26 AM, David Marchand <david.marchand@6wind.com>
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
>
>

  parent reply	other threads:[~2014-08-18 10:43 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 ` David Marchand [this message]
2014-08-18 14:07 ` [dpdk-dev] [PATCH 0/7] fix build errors Neil Horman

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='CALwxeUvFHGvSjGfsKJ9vnbP=6g81HRErgV1zioPr+1WUes47Wg@mail.gmail.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).