[dpdk-dev] [PATCH 3/3] mk: rule help

2013-05-17 Thread Thomas Monjalon
17/05/2013 11:06, Olivier MATZ : > On 05/07/2013 05:17 PM, Thomas Monjalon wrote: > > The macro maketxt allow to interpret syntax "# make foo" while keeping > > indentation of the source document. > > Here, it is used to call "make showconfigs" to list available targets. > [..] > > +define maketxt

[dpdk-dev] [PATCH 2/3] doc: how to build

2013-05-17 Thread Thomas Monjalon
17/05/2013 11:06, Olivier MATZ : > On 05/07/2013 05:17 PM, Thomas Monjalon wrote: > > +Build options > > + T target template (install default: *) - used with config or install > > + O output directory (default: build/) - cannot be used with install > > + V verbose > > + D debug de

[dpdk-dev] [PATCH] lib: set version 1.2.3r2 and more helper macros

2013-05-17 Thread Olivier MATZ
Hi Thomas, > This patch introduce RTE_VER_PATCH_RELEASE so #ifdef RTE_VER_PATCH_RELEASE is > always true. I don't see why a commercial version of the DPDK would like to > apply part of this patch. ok this was just a comment in case of.

[dpdk-dev] [PATCH 1/3] mk: rule showconfigs for configs listing

2013-05-17 Thread Thomas Monjalon
17/05/2013 11:05, Olivier MATZ : > I wonder if this new rule is really needed: I think you can already > do a "make config" without argument to list the supported > configurations. Yes it is needed because "make config" adds an error comment. But I could refactor this patch to use "make showconfig

[dpdk-dev] [PATCH] lib: set version 1.2.3r2 and more helper macros

2013-05-17 Thread Thomas Monjalon
17/05/2013 11:12, Olivier MATZ : > > @@ -75,7 +87,8 @@ rte_version(void) { > > > > return RTE_VER_PREFIX" " > > > > RTE_STR(RTE_VER_MAJOR)"." > > RTE_STR(RTE_VER_MINOR)"." > > > > - RTE_STR(RTE_VER_PATCH_LEVEL); > > +

[dpdk-dev] DPDK apps are now working on CentOS 6.4

2013-05-17 Thread Thomas Monjalon
Hello, 17/05/2013 16:06, Patrick Mahan : > I don't know (yet) why Fedora 14 would not > work while CentOS does when it looks like they have the same fixes. It's strange. I've already seen DPDK running on Fedora 14. > The only error I was the inability to mmap the HPET even though the kernel > co

[dpdk-dev] [PATCH] lib: set version 1.2.3r2 and more helper macros

2013-05-17 Thread Olivier MATZ
Hi Thomas, > @@ -75,7 +87,8 @@ rte_version(void) { > return RTE_VER_PREFIX" " > RTE_STR(RTE_VER_MAJOR)"." > RTE_STR(RTE_VER_MINOR)"." > - RTE_STR(RTE_VER_PATCH_LEVEL); > + RTE_STR(RTE_VER_PATCH_LEVEL)"r" > +

[dpdk-dev] [PATCH 3/3] mk: rule help

2013-05-17 Thread Olivier MATZ
Thomas, I think the "maketxt" macro is overkill... why not just a "cat" ? Olivier On 05/07/2013 05:17 PM, Thomas Monjalon wrote: > The goal of this rule is to show which rules/options are available from a > "make" command. > Let's start by printing short doc about SDK building. > > The macro ma

[dpdk-dev] [PATCH 2/3] doc: how to build

2013-05-17 Thread Olivier MATZ
Hi Thomas, Good to see a "make help" ;) What do you think of replacing "Build options" by "Build variables" ? I think it is not obvious that we have to use V=1 for instance for a verbose build. About the "build targets", I suggest to include it in the help of T variable. What do you think ? Oli

[dpdk-dev] [PATCH 1/3] mk: rule showconfigs for configs listing

2013-05-17 Thread Olivier MATZ
Hi Thomas, I wonder if this new rule is really needed: I think you can already do a "make config" without argument to list the supported configurations. Olivier On 05/07/2013 05:17 PM, Thomas Monjalon wrote: > Allow to list configs in config/ directory with a simple make rule. > > Signed-off-by

[dpdk-dev] DPDK apps are now working on CentOS 6.4

2013-05-17 Thread Patrick Mahan
This is an update to my last email requesting help with the DPDK sources and Fedora 14. After talking with a colleague who was investigating DPDK as well, I removed the Fedora 14 install and installed instead CentOS 6.4. After installing the release, I did a 'yum update' on the kernel sources to