DPDK patches and discussions
 help / color / mirror / Atom feed
From: David Marchand <david.marchand@6wind.com>
To: "Qiu, Michael" <michael.qiu@intel.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v2] Fix rte_is_power_of_2
Date: Mon, 19 Jan 2015 10:49:19 +0100	[thread overview]
Message-ID: <CALwxeUvjjJV_vdPYfyCnQSM_m4x2YtizNW3-BXz8K0Tprd3Yhg@mail.gmail.com> (raw)
In-Reply-To: <533710CFB86FA344BFBF2D6802E60286CB74C5@SHSMSX101.ccr.corp.intel.com>

On Mon, Jan 19, 2015 at 10:30 AM, Qiu, Michael <michael.qiu@intel.com>
wrote:

>
> This could be, but I hope every contributor could do basic check like
> build and run of at lease one app like test-pmd, then it will reduce
> most of failure we faced.
>

testpmd is fine, but make test should be preferred as it requires no setup
and it requires no pmd to be configured.
Besides, testpmd is an application, it does not test libraries that it does
not use, while make test is supposed to test all libraries.


There is still the hugepages to configure when running make test so I would
say "make test" should tell you so.
This will avoid wasting 10 minutes to understand why all your tests fail ...

Can anyone help document this / make it more user friendly / easier ?


-- 
David Marchand

  reply	other threads:[~2015-01-19  9:49 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-27 15:28 Ravi Kerur
2014-12-27 15:30 ` Ravi Kerur
2014-12-27 20:49   ` Neil Horman
2015-01-15 13:12     ` Thomas Monjalon
2015-01-19  2:45     ` Zhang, Helin
2015-01-19  7:21       ` David Marchand
2015-01-19  9:13         ` Thomas Monjalon
2015-01-19  9:30           ` Qiu, Michael
2015-01-19  9:49             ` David Marchand [this message]
2015-01-19 10:02               ` Qiu, Michael
2015-01-19 10:07               ` Qiu, Michael
2015-01-19 17:07           ` Ravi Kerur
2015-01-19 22:07             ` Ravi Kerur
2015-01-19  9:14         ` De Lara Guarch, Pablo

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=CALwxeUvjjJV_vdPYfyCnQSM_m4x2YtizNW3-BXz8K0Tprd3Yhg@mail.gmail.com \
    --to=david.marchand@6wind.com \
    --cc=dev@dpdk.org \
    --cc=michael.qiu@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).