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 check fail on i386 #756

Open
frederic-loui opened this issue Apr 15, 2019 · 0 comments

Comments

@frederic-loui
Copy link

commented Apr 15, 2019

Hi,

On i386, make operation succeeds, but make check fails for bmv2.
Not sure if this is linked, but make succeeds for target simple_switch_grpc but also fail for make check.

On amd64, make & make check both succeeds for simple_switch and simple_switch_grpc targets.

As a test, on the same xenial i386 machine p4c make and 'make check` succeeds. (1301 successful tests )

How to reproduce:
--> install a fresh ubuntu xenial 32 bit image and p4lang software dependencies:
1- compile p4lang PI

git clone https://github.com/p4lang/PI.git
git submodule update --init --recursive
./autogen.sh
./configure --with-proto --without-internal-rpc --without-cli --without-bmv2 --without-sysrepo --prefix=/usr
make
make check
sudo make install

2- compile bmv2

git clone https://github.com/p4lang/behavioral-model.git
./autogen.sh
./configure --with-pdfixed --with-pi --prefix=/usr \
	--enable-debugger --with-thrift --with-nanomsg "CXXFLAGS=-O0 -g";
make
make check    <---- observe test_fields test failure
make install <---- for p4c tests

3- compile p4c

git clone https://github.com/p4lang/p4c.git
git submodule update --init --recursive
mkdir build && cd build ;
cmake .. -DCMAKE_BUILD_TYPE=DEBUG -DCMAKE_INSTALL_PREFIX=/usr 
make
make check  <---- observe that all the 1301 tests succeed)
make install 

As it can be time consuming to try to reproduce this issue, I build a 32 bit xenial ova you can download here: (I tried to minimise the size but it is still 3.7 Gb ...)
https://filesender.renater.fr/?s=download&token=9d3b2af9-51e9-200e-120d-99c9256da0db
username/password: p4lang/p4lang
(The link will stay valid till the 6th of may 2019)

PS: Most of the system are 64 bit today so this might not be a critical issue. It prevents just Launchpad/PPA to produce the package as by default launchpad consider i386 and amd64.
If it is decided to only consider amd64 architecture, I'll disable i386 build in debian/control file.

Thanks !
Frederic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.