ovis icon indicating copy to clipboard operation
ovis copied to clipboard

Containerized arm64 cython --gdb work around

Open narategithub opened this issue 2 years ago • 7 comments

cython with --gdb option resulted in "segmentation fault" when make built ldms python module on arm64 (aarch64) containers. This issue does not appear on x86_64 containers. This patch adds a work around to only enable cython --gdb option for x86_64 architecture.

narategithub avatar Aug 23 '22 21:08 narategithub

@narate, can you please post this issue on the cython github page?

tom95858 avatar Sep 07 '22 01:09 tom95858

@tom95858 sure. Let me make a simple .pyx that reproduce the issue. I think ldms.pyx + ldms code is too complicated to submit to cython.

narategithub avatar Sep 07 '22 01:09 narategithub

This was all found on Raspberry Pi, correct? I would like to ensure that this is an issue on Stria, etc... before accepting this.

tom95858 avatar Sep 15 '22 18:09 tom95858

@tom95858 The problem is found in a containerized Pi4. On containerized x86_64, we don't have this issue. On bare metal machines, both x86_64 and Pi4 do NOT have the issue.

narategithub avatar Sep 15 '22 18:09 narategithub

Is there a simple-enough reproducer that we could just turn the reproducer into a configure.ac feature check and assume that code for unhappy containers is also going to be built in the unhappy containers?

baallan avatar Sep 15 '22 20:09 baallan

@narategithub I get it but this will disable --gdb for all platforms except x86_64. So I'd just like to understand what this does and why not just let it fail on those platforms that are broken. This is really a cython bug right?

tom95858 avatar Sep 15 '22 23:09 tom95858

@tom95858 I think I'll close this pull request. I think my Pi4 might be wonky. When using Pi4 (Ubuntu 22.04), the current https://github.com/ovis-hpc/ldms-containers cannot even successfully build ovishpc/ldms-dev. So, I switched to Jetson Nano (another arm64 small machine from Nvidia; the OS is based on Ubuntu 18.04), and everything works without this patch. I'll switch this PR to draft to keep the discussion open for now.

narategithub avatar Sep 16 '22 03:09 narategithub

@narategithub can you please close this?

tom95858 avatar Feb 25 '23 19:02 tom95858