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

autotools build not working on current master #1320

Closed
edhartnett opened this issue Dec 31, 2018 · 8 comments · Fixed by #1327
Closed

autotools build not working on current master #1320

edhartnett opened this issue Dec 31, 2018 · 8 comments · Fixed by #1327

Comments

@edhartnett
Copy link
Collaborator

I just checked out master and tried to build, but got a failure:

autoreconf -i && CC=mpicc CPPFLAGS=-I/usr/local/netcdf-c-4.6.2_mpich-3.2/include LDFLAGS=-L/usr/local/netcdf-c-4.6.2_mpich-3.2/lib ./configure && make check
and I get:

FAIL: run_tests.sh

Running the test itself I see the problem is test_pioc:

mpiexec -n 4 ./test_pioc

===================================================================================
=   BAD TERMINATION OF ONE OF YOUR APPLICATION PROCESSES
=   PID 4448 RUNNING AT mikado
=   EXIT CODE: 139
=   CLEANING UP REMAINING PROCESSES
=   YOU CAN IGNORE THE BELOW CLEANUP MESSAGES
===================================================================================
YOUR APPLICATION TERMINATED WITH THE EXIT STRING: Segmentation fault (signal 11)
This typically refers to a problem with your application.
Please see the FAQ page for debugging suggestions
ed@mikado:~/tmp/ParallelIO/tests/cunit$ 
@jedwards4b
Copy link
Contributor

Can you provide more information? Compiler version, os, etc - typically this error (segfault with no other output) indicates a problem with mpi and not with the test itself.

@edhartnett
Copy link
Collaborator Author

This turns out to be a bug in netCDF 4.6.2. I am fixing now...

@edhartnett
Copy link
Collaborator Author

Well it's also a PIO issue. I now have to change the PIO code to not rely on NULL being a valid stride (standing for all 1's).

An annoyance, but required if PIO is to work with netCDF 4.6.2. So I will reopen this issue and submit a PR with those changes.

@edhartnett edhartnett reopened this Jan 2, 2019
@jedwards4b
Copy link
Contributor

I just want to make sure I'm understanding this - netcdf 4.6.2 is broken and this will be fixed in the next netcdf release but we still want to put a hack in pio so that it works with that version. Is that right?

@edhartnett
Copy link
Collaborator Author

Yes, otherwise we will have to constantly answer support questions about why PIO is broken with netcdf 4.6.2. ;-)

It's a simple hack that won't affect performance.

@jedwards4b
Copy link
Contributor

Hi Ed - what is the ETA on this change?

@edhartnett
Copy link
Collaborator Author

edhartnett commented Jan 18, 2019 via email

@edhartnett
Copy link
Collaborator Author

edhartnett commented Jan 28, 2019

Working on this now...

This is happening here:

0x0000555555569c53 in PIOc_put_vars_tc (ncid=<optimized out>, ncid@entry=52, varid=<o
    start=start@entry=0x555555882360, count=count@entry=0x555555a09e40, stride=stride@ent
    xtype=<optimized out>, xtype@entry=4, buf=0x7fffffffd7a0) at pio_getput_int.c:1158
1158                    ierr = nc_put_vars_int(file->fh, varid, (size_t *)start, (size_t

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

Successfully merging a pull request may close this issue.

2 participants