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

Invoking papi counters give segfault #1890

Closed
pagrubel opened this issue Dec 1, 2015 · 9 comments
Closed

Invoking papi counters give segfault #1890

pagrubel opened this issue Dec 1, 2015 · 9 comments

Comments

@pagrubel
Copy link
Member

pagrubel commented Dec 1, 2015

bin/1d_stencil_4 --nx=1000 --np=1000 -t4 \
    --hpx:print-counter /arithmetics/add@/papi{locality#0/worker-thread#*}/PAPI_TOT_CYC
Segmentation fault

I tried this with current master and the 09.11 tag

@hkaiser hkaiser added this to the 0.9.12 milestone Dec 1, 2015
@hkaiser
Copy link
Member

hkaiser commented Dec 1, 2015

@maciekab Is that something you would have time to look into?

@pagrubel
Copy link
Member Author

pagrubel commented Dec 1, 2015

I'm doing a bisect see if I can narrow this

@pagrubel
Copy link
Member Author

pagrubel commented Dec 1, 2015

also segfaults when papi counter is not used with statistics

bin/1d_stencil_4 --nx=1000 --np=1000 -t4 \
    --hpx:print-counter /papi{locality#0/worker-thread#*}/PAPI_TOT_CYC  
Segmentation fault

and for hello_world

bin/hello_world --np=1000 -t4 \
    --hpx:print-counter /papi{locality#0/worker-thread#*}/PAPI_TOT_CYC
Segmentation fault

@pagrubel
Copy link
Member Author

pagrubel commented Dec 1, 2015

ecb63d4 is the first bad commit

@hkaiser
Copy link
Member

hkaiser commented Dec 1, 2015

ecb63d4 is the first bad commit

@pagrubel Ahh thanks, this makes sense. I will fix things.

@pagrubel
Copy link
Member Author

pagrubel commented Dec 4, 2015

tried fixing_1890 and now I get this:

 unknown counter type: /papi/PAPI_TOT_CYC, 

ah compiling errors:

output_build.txt

@hkaiser
Copy link
Member

hkaiser commented Dec 4, 2015

Please try again, I added the missing #include directives

@pagrubel
Copy link
Member Author

pagrubel commented Dec 4, 2015

I had to make one more change and now it works.

@pagrubel pagrubel mentioned this issue Dec 4, 2015
@hkaiser
Copy link
Member

hkaiser commented Dec 4, 2015

This was fixed by merging #1893

@hkaiser hkaiser closed this as completed Dec 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants