Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

make compile error with latest version of odp #2

Closed
jestan opened this issue Dec 25, 2015 · 5 comments
Closed

make compile error with latest version of odp #2

jestan opened this issue Dec 25, 2015 · 5 comments

Comments

@jestan
Copy link

jestan commented Dec 25, 2015

I am trying to build ofp and getting following compilation error.

It looks like pthread_create function has been changed in odp.

commit 1a7893cf034dcb0045c3f7a08559b0955b3093b5
Date: Wed Dec 16 15:14:14 2015 +0530

helper: linux: add thread type in pthread_create

The exisiting helper routine only create the worker threads.
However there is a need to use the same for creating the control
thread as well. e.g. CLI thread.

build@ubuntu:~/sources/ofp$ make
Making all in src
make[1]: Entering directory `/home/build/sources/ofp/src'
  CC       ofp_avl.lo
  CC       cli/ofp_cli.lo
cli/ofp_cli.c: In function 'ofp_start_cli_thread':
cli/ofp_cli.c:1724:6: error: too few arguments to function 'odph_linux_pthread_create'
  if (odph_linux_pthread_create(&ofp_global_cfg->cli_thread,
      ^
In file included from cli/ofp_cli.c:24:0:
/home/build/installs/odp/include/odp/helper/linux.h:67:5: note: declared here
 int odph_linux_pthread_create(odph_linux_pthread_t *thread_tbl,
     ^
make[1]: *** [cli/ofp_cli.lo] Error 1
make[1]: Leaving directory `/home/build/sources/ofp/src'
make: *** [all-recursive] Error 1
@bogdanPricope
Copy link
Contributor

Latest supported ODP version is 1.4.0
Adding support for ODP 1.5.0 and 1.6.0 is in progress.

@jestan
Copy link
Author

jestan commented Jan 6, 2016

Build environment preparation section in the readme doesn't mention about required ODP version.

@bogdanPricope
Copy link
Contributor

FAQ section:
"
Q: What versions of ODP does OFP support?

A: At the moment it supports all the way from 1.1.0 to 1.4.0"

ODP 1.5.0 patches are in.

@jestan jestan closed this as completed Jan 9, 2016
@jestan
Copy link
Author

jestan commented Jan 9, 2016

Build sections says,

Download ODP:
git clone git://git.linaro.org/lng/odp.git

When I tried it , I got the latest version which was not supported.
It will be better if you can say, clone the ODP tag 1.4.0, not the master.

Just my few cents.

@semihalf-berestovskyy-andriy
Copy link
Contributor

semihalf-berestovskyy-andriy commented Jan 24, 2017

Note: the comment was created automatically with bugzilla2github tool


Bugzilla bug ID: 2
Renumbered to issue: #125

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants