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

Already on GitHub? Sign in to your account

Problems using mmv with Python #90

Closed
taintedkernel opened this Issue May 13, 2016 · 3 comments

Comments

Projects
None yet
3 participants

Registering this issue after discussing the problem in #pcp without any resolution.

After discovering the ability to instrument Python code with PCP, I attempted to do so with example code provided in the module - src/python/pcp/mmv.py. With a few modifications to get it to run, I had problems retrieving the data with pminfo. I also compiled the acme.c example, which worked just fine - pminfo -f mmv saw it without problems (and /var/lib/pcp/tmp/mmv/acme had data). Strace returned nothing for /var/lib/pcp/tmp/mmv on the Python code, so something appears to be broken on the binding.

This is on a RHEL6 box with latest packages (3.11.3 as of writing) built from scratch via Makepkgs and Python2.6. I can provide the full strace or example Python code if necessary.

Contributor

fche commented May 13, 2016

At least one bug evident in the python mmv bindings:

diff --git a/src/python/pcp/mmv.py b/src/python/pcp/mmv.py
-LIBPCP_MMV.mmv_lookup_value_desc.restype = pmAtomValue
+LIBPCP_MMV.mmv_lookup_value_desc.restype = POINTER(pmAtomValue)

which triggers a stack-smashing crash under glibc/ffi

Actually see a few other issues as well, going to continue to work on this.

@natoscott natoscott closed this May 19, 2016

Contributor

natoscott commented May 19, 2016

Oops, meant to comment on close: "all should be resolved in pcp git master branch now ... lemme know if issues persist"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment