Fix for segmentation fault on exit (Python) #239
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Remove useless __del__ because the cleanup() function gets called by atexit(). Remove ws.ws2811_fini() call which causes the SEGFAULT on exit. Python does not complain about the memory leak in this way.
|
mind you, the code still segfaults if the event handler is registered to clear the LEDs before exiting (pass -c as an argument) in my own code segfault happens also if the code to clear the leds is registered as an atexit function. Clearing the leds happens, just I'm getting a segfault. |
|
Executing |
|
yes, also on a Pi3. |
|
@pietrodn Hello, and thanks for this fix man, I was getting segfault and now it's fixed. Could you link to some materials or briefly describe how you debug segfaults such as the one you fixed? Is it enabling core dump and then using |
brindosch
pushed a commit
to hyperion-project/rpi_ws281x
that referenced
this pull request
Apr 5, 2018
* Added delay to ensure reset time being met with low led counts (jgarff#157) * Fix includes as per jgarff#178 (jgarff#189) * clean up doco and strandtest (jgarff#187) * updated documentation on GPIO usage strandtest now defaults to GRB colour order added version.h to gitignore * updated python readme * more README cleanup * added notes on spi max transfer size * Fix freeze after running for long time (jgarff#197) * Whitespace cleanup * Move channel_protocol_time so it handles 32 bit Color LEDs. * Fix a freezing issue when running for a long time. When current_timestamp overflowed 2^64, rolling back to 0 and ws2811->render_wait_until had not, this caused an incredibly long usleep in ws2811_render. * Update README.md (jgarff#191) * Update README.md * updated spi buffer size location * Add -lrt link flag for systems with older glibc (jgarff#195) * Fix 32-bit overflow in get_microsecond_timestamp(). (jgarff#206) On Raspbian systems, the tv_sec member of struct timespec is 32 bits wide. get_microsecond_timestamp() was overflowing in 32-bit arithmetic on this member. This causes the time that it returns to jump ahead by many thousands of years at uptime = 35 minutes, 48 seconds. (While the starting point for CLOCK_MONOTONIC_RAW is unspecified, it seems to track system uptime in practice.) This overflow didn't cause noticeable problems for me--ws2811_render() skips the sleep when it thinks such a huge amount time has elapsed--but it could possibly result in rarely-occurring skipped frames if the WS2811 reset time isn't effectively honored by ws2811_render(). * Add TOOLCHAIN option for cross compilation (jgarff#213) Use this option if you want to cross-compile the library. Typical usage is: scons TOOLCHAIN=arm-linux-gnueabihf * Add -Wextra compile option and adapt code to suppress warnings (jgarff#214) The changes in code are mostly type casts between int ann unsigned. * Add shared library rasbpian package builder. The package is not built by default. To generate the package you need: - The dpkg-deb utility. - Use "scons deb" to generate the package. * Merge user-settable gamma correction into master (jgarff#216) Merge user-settable gamma correction into master * Bug fixes for Stretch and Compute Module support (jgarff#222) * Bug fixes Missing aligned(4) attribute in pcm.h makes the PCM case failing under Stretch, due to a newer version of gcc (6.x). Also fixed a bug in check_hwver_and_gpionum, so original Model A is recognised. * README, using SPI on RPi3 Added note to change the core frequency to 250 MHz when using SPI on RPi 3. * README fix * Add support for Compute Module 1 and 3 * Fix undefined clock_gettime symbol * add missing memory allocation for SPI mode (jgarff#231) * added Ctrl-C handler to turn off all LEDs on exit (jgarff#140) * added Ctrl-C handler to turn off all LEDs on exit * added option -c * Fix for segmentation fault on exit for Python API (jgarff#239) Remove useless __del__ because the cleanup() function gets called by atexit(). Remove ws.ws2811_fini() call which causes the SEGFAULT on exit. Python does not complain about the memory leak in this way. * Add support for new A+ hardware. * Change default DMA channel to safe choice (10) (jgarff#253) add warning on README.md * Add getBrightness for brightness feedback from the buffer (jgarff#248) * Instructions to install the Python library (jgarff#254) Solving issue jgarff#142 * Fix for non-root users & SPI (jgarff#257) * Fix for non-root users can use SPI * use DEFINE'd device strings also add a note to the readme * Basic Golang Examples (jgarff#256) * Added golang example * Added basic README for golang examples. Added os exits for errors during color wipes for golang examples * Fix the default DMA in python code (jgarff#266) DMA 5 causes file system errors on PI 3 * Fixed typo in README.md (jgarff#273) Was perusing recent commits and saw this. * clarify electrical notes (jgarff#264) * Fixed strandtest.py (jgarff#258) * Fixed keyboardinterrupt, indentation inconsistency, & a few other minor bugs. Added shebang. * Fixed KeyboardInterrupt. Again. * KeyboardInterrupt works now. For sure. * Tabs to spaces, removed redundant signal handling * Removed unused sys module import * Added Raspberry Pi 3 B+ (jgarff#276) * Added Pi 3 B+ * Remove unused import
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
NeoPixel Python examples, when quit with Ctrl+C, crash with a SEGFAULT on the Raspberry Pi 3 model B:
After some debugging I fixed the problem by removing the
ws.ws2811_fini(self._leds)call from the_cleanup()method in theAdafruit_NeoPixelPython class.I also removed the useless
__del__()because_cleanup()already gets called byatexit().Now the Python samples do not SEGFAULT anymore.
This problem is not present with the compiled C samples.